From 84589e7b899172d165aaac856b96f22cbaf968d0 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Mar 06 2017 19:12:37 +0000 Subject: When resetting the value of a custom field, indicate what the old value was Fixes https://pagure.io/pagure/issue/2041 --- diff --git a/pagure/lib/__init__.py b/pagure/lib/__init__.py index 5aa6ccb..49407c6 100644 --- a/pagure/lib/__init__.py +++ b/pagure/lib/__init__.py @@ -3838,7 +3838,7 @@ def set_custom_key_value(session, issue, key, value): output += ' (was: %s)' % old_value return output elif updated and old_value: - return 'Custom field %s reset' % key.name + return 'Custom field %s reset (from %s)' % (key.name, old_value) def get_yearly_stats_user(session, user, date): diff --git a/tests/test_pagure_flask_api_issue.py b/tests/test_pagure_flask_api_issue.py index 2fc5eca..9ac5544 100644 --- a/tests/test_pagure_flask_api_issue.py +++ b/tests/test_pagure_flask_api_issue.py @@ -2037,7 +2037,8 @@ class PagureFlaskApiIssuetests(tests.Modeltests): self.assertDictEqual( data, { - "message": "Custom field bugzilla reset" + "message": "Custom field bugzilla reset " + "(from https://bugzilla.redhat.com/1234)" } )