)]}'
{"openstackclient/object/v1/account.py":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"0b083a48e16f2c1d919b9362eb6eb64808473424","unresolved":false,"context_lines":[{"line_number":37,"context_line":"        parser.add_argument("},{"line_number":38,"context_line":"            \"--no-property\","},{"line_number":39,"context_line":"            action\u003d\u0027store_true\u0027,"},{"line_number":40,"context_line":"            help\u003d_(\"Clear existing property on this account \""},{"line_number":41,"context_line":"                   \"Specify both --property and --no-property \""},{"line_number":42,"context_line":"                   \"to overwrite the current property.\")"},{"line_number":43,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_1aea2adb","line":40,"updated":"2017-01-24 16:52:21.000000000","message":"We shouldn\"t be able to pass \u0027--property\u0027 and \u0027--no-property\u0027 on the same command. It\u0027s counter intuitive... Do we do that already somewhere else in OSC ?","commit_id":"abfb34b6e75c64e0b180fae0381bc9345ab25bfb"},{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"797ca07d38180176b2ad7db38400d2d58e88344d","unresolved":false,"context_lines":[{"line_number":46,"context_line":"    def take_action(self, parsed_args):"},{"line_number":47,"context_line":"        if \u0027no-property\u0027 in parsed_args:"},{"line_number":48,"context_line":"            self.app.client_manager.object_store.account_unset("},{"line_number":49,"context_line":"                properties\u003dparsed_args.property[\u0027no-property\u0027]"},{"line_number":50,"context_line":"            )"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"        self.app.client_manager.object_store.account_set("}],"source_content_type":"text/x-python","patch_set":1,"id":"5a3905b3_dfd50fe5","line":49,"range":{"start_line":49,"start_character":38,"end_line":49,"end_character":61},"updated":"2017-01-24 09:03:44.000000000","message":"Hi rajivk,\nThanks for the patch.\nFor Overwrite options, we check if both property and --no-property are defined or not.\nIf only no-property is defined, then we call the account_unset on the currently defined properties of account, so maybe you can change it to \n\n if parsed_args.no_property:\n  key_list \u003d []\n  data \u003d self.app.client_manager.object_store.account_show()\n  if \u0027properties\u0027 in data:\n  try:\n   flavor.unset_keys(utils.format_dict(data.pop(\u0027properties\u0027)))","commit_id":"abfb34b6e75c64e0b180fae0381bc9345ab25bfb"}]}