summaryrefslogtreecommitdiff
path: root/yarns/implementations.yarn
diff options
context:
space:
mode:
authorLars Wirzenius <lars.wirzenius@codethink.co.uk>2013-10-23 16:22:50 +0000
committerLars Wirzenius <lars.wirzenius@codethink.co.uk>2013-10-24 15:17:04 +0000
commit24e0e1ff64b1bcaede0366f9c0765bc740754da8 (patch)
tree8cd6f273d95870c20b732879c78c767917d9f3c0 /yarns/implementations.yarn
parentbcb725108c291611d125f8c5048a368dd5cdadb4 (diff)
downloadmorph-24e0e1ff64b1bcaede0366f9c0765bc740754da8.tar.gz
Fix app.status call in log_dict_diff
This bug was triggered by the fix to CachedRepo.resolve_ref and without this fix, the resolve_ref fix will break the test suite. The bug is that log_dict_diff calls the status method with an msg keyword argument that may contain percentage characters. status interprets the value of msg as a format string, and the percentage characters trigger formatting to happen. The fix for that is to not interpolate the value of key and dictA[key] and dictB[key] into msg before calling status, but letting status do that. Thus the msg values are changed to reference %(key)s instead and passing in a value for key as a separate argument. Ditto for dictA[key] and dictB[key].
Diffstat (limited to 'yarns/implementations.yarn')
0 files changed, 0 insertions, 0 deletions