)]}'
{"openstack_dashboard/dashboards/project/loadbalancers/workflows.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e89172bc7ff70ce2ffa7a2edb1642f0dd3ef0cd0","unresolved":false,"context_lines":[{"line_number":107,"context_line":"    def handle(self, request, context):"},{"line_number":108,"context_line":"        try:"},{"line_number":109,"context_line":"            pool \u003d api.lbaas.pool_create(request, **context)"},{"line_number":110,"context_line":"            context[\u0027name\u0027]"},{"line_number":111,"context_line":"            return True"},{"line_number":112,"context_line":"        except:"},{"line_number":113,"context_line":"            msg \u003d self.format_status_message(self.failure_message)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F1f8%3D","line":110,"updated":"2013-03-10 09:55:16.000000000","message":"This line is unnecessary. Could you remove it?","commit_id":"3af51c92bd1404601b49af5c51e231a2bcf87028"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e89172bc7ff70ce2ffa7a2edb1642f0dd3ef0cd0","unresolved":false,"context_lines":[{"line_number":354,"context_line":"                context[\u0027member_id\u0027] \u003d api.lbaas.member_create("},{"line_number":355,"context_line":"                    request, **context).id"},{"line_number":356,"context_line":"            except:"},{"line_number":357,"context_line":"                msg \u003d self.format_status_message(self.failure_message)"},{"line_number":358,"context_line":"                exceptions.handle(request, msg)"},{"line_number":359,"context_line":"                return False"},{"line_number":360,"context_line":"        return True"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F1f0%3D","line":357,"updated":"2013-03-10 09:55:16.000000000","message":"At this point, context[\u0027member_id\u0027] is undefined since an exception typically occurs during the API call.\nI think it would be better to change the message just to \"Unable to add member\" without member_id and remove format_status_message().","commit_id":"3af51c92bd1404601b49af5c51e231a2bcf87028"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e89172bc7ff70ce2ffa7a2edb1642f0dd3ef0cd0","unresolved":false,"context_lines":[{"line_number":443,"context_line":"                request, **context).get(\u0027id\u0027)"},{"line_number":444,"context_line":"            return True"},{"line_number":445,"context_line":"        except:"},{"line_number":446,"context_line":"            msg \u003d self.format_status_message(self.failure_message)"},{"line_number":447,"context_line":"            exceptions.handle(request, msg)"},{"line_number":448,"context_line":"        return False"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAN3%2F%2F1gA%3D","line":446,"updated":"2013-03-10 09:55:16.000000000","message":"Same as the above.","commit_id":"3af51c92bd1404601b49af5c51e231a2bcf87028"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"83fdb6d8ac71e5fc1c9ff608dcb51a2c5933fab9","unresolved":false,"context_lines":[{"line_number":329,"context_line":"    default_steps \u003d (AddMemberStep,)"},{"line_number":330,"context_line":""},{"line_number":331,"context_line":"    def format_status_message(self, message):"},{"line_number":332,"context_line":"        member_id \u003d self.context.get(\u0027member_id\u0027)"},{"line_number":333,"context_line":"        return message % member_id"},{"line_number":334,"context_line":""},{"line_number":335,"context_line":"    def handle(self, request, context):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAN3%2F%2FzwM%3D","line":332,"updated":"2013-03-11 20:46:53.000000000","message":"Please remove format_status_message(). It is no longer used.","commit_id":"6da55cce90452bcccf66672fe185f9794e2b6e52"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"83fdb6d8ac71e5fc1c9ff608dcb51a2c5933fab9","unresolved":false,"context_lines":[{"line_number":434,"context_line":""},{"line_number":435,"context_line":"    def format_status_message(self, message):"},{"line_number":436,"context_line":"        monitor_id \u003d self.context.get(\u0027monitor_id\u0027)"},{"line_number":437,"context_line":"        return message % monitor_id"},{"line_number":438,"context_line":""},{"line_number":439,"context_line":"    def handle(self, request, context):"},{"line_number":440,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAN3%2F%2FzwI%3D","line":437,"updated":"2013-03-11 20:46:53.000000000","message":"Same as the above.","commit_id":"6da55cce90452bcccf66672fe185f9794e2b6e52"}]}