)]}'
{"openstack_dashboard/dashboards/admin/flavors/workflows.py":[{"author":{"_account_id":18494,"name":"Ravi Shekhar Jethani","email":"rsjethani@gmail.com","username":"rsjethani"},"change_message_id":"5865f12f690b0d2ff4f044f78e8db5c525b4d4ff","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        name \u003d cleaned_data.get(\u0027name\u0027)"},{"line_number":74,"context_line":"        flavor_id \u003d cleaned_data.get(\u0027flavor_id\u0027)"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        if name and name.isspace():"},{"line_number":77,"context_line":"            msg \u003d _(\u0027Flavor name cannot be empty.\u0027)"},{"line_number":78,"context_line":"            self._errors[\u0027name\u0027] \u003d self.error_class([msg])"},{"line_number":79,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_ccca791a","line":76,"range":{"start_line":76,"start_character":11,"end_line":76,"end_character":34},"updated":"2016-07-08 08:58:47.000000000","message":"We are changing behavior. For e.g.:\nisspace(\u0027\u0027) -\u003e False whereas\nlen(\u0027\u0027.strip()) \u003d\u003d 0 -\u003e True","commit_id":"7fdebb62f3d99ccdefa362fc0222fb5015de7b17"},{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"63e78466cdd7543f7cad0014f66bd3c52c48be14","unresolved":false,"context_lines":[{"line_number":73,"context_line":"        name \u003d cleaned_data.get(\u0027name\u0027)"},{"line_number":74,"context_line":"        flavor_id \u003d cleaned_data.get(\u0027flavor_id\u0027)"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"        if name and name.isspace():"},{"line_number":77,"context_line":"            msg \u003d _(\u0027Flavor name cannot be empty.\u0027)"},{"line_number":78,"context_line":"            self._errors[\u0027name\u0027] \u003d self.error_class([msg])"},{"line_number":79,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_660ba17e","line":76,"range":{"start_line":76,"start_character":11,"end_line":76,"end_character":34},"in_reply_to":"1aa78d24_ccca791a","updated":"2016-07-08 09:42:54.000000000","message":"In this particular case, ``name and name.isspace()`` works as expected -- the empty string is caught by the first part. However, you are right for the other cases.","commit_id":"7fdebb62f3d99ccdefa362fc0222fb5015de7b17"}]}
