)]}'
{"horizon/tables/actions.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":218,"context_line":"                                                    self.name.title())"},{"line_number":219,"context_line":"        self.verbose_name_plural \u003d verbose_name_plural or \\"},{"line_number":220,"context_line":"            getattr(self, \u0027verbose_name_plural\u0027,"},{"line_number":221,"context_line":"                   \"%ss\" % self.verbose_name)"},{"line_number":222,"context_line":"        self.handles_multiple \u003d getattr(self,"},{"line_number":223,"context_line":"                                        \"handles_multiple\","},{"line_number":224,"context_line":"                                        handles_multiple)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPmk%3D","line":221,"updated":"2013-10-01 17:14:18.000000000","message":"IMO new line with a backslash should be removed.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"horizon/tables/base.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":750,"context_line":"    def __init__(self, options):"},{"line_number":751,"context_line":"        self.name \u003d getattr(options, \u0027name\u0027, self.__class__.__name__)"},{"line_number":752,"context_line":"        verbose_name \u003d getattr(options, \u0027verbose_name\u0027, None) \\"},{"line_number":753,"context_line":"            or self.name.title()"},{"line_number":754,"context_line":"        self.verbose_name \u003d verbose_name"},{"line_number":755,"context_line":"        self.columns \u003d getattr(options, \u0027columns\u0027, None)"},{"line_number":756,"context_line":"        self.status_columns \u003d getattr(options, \u0027status_columns\u0027, [])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPmM%3D","line":753,"updated":"2013-10-01 17:14:18.000000000","message":"It is better to add a opening parenthesis before getattr.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"horizon/test/helpers":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"ii"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAS3%2F%2FPl4%3D","line":1,"updated":"2013-10-01 17:14:18.000000000","message":"This file seems to be added by mistake.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":6914,"name":"Tatiana Ovchinnikova","email":"t.v.ovtchinnikova@gmail.com","username":"tmazur"},"change_message_id":"592f67ac812a2b2288358577ba7adec92afa62de","unresolved":false,"context_lines":[{"line_number":1,"context_line":"ii"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAATn%2F91bA%3D","line":1,"updated":"2013-12-05 13:11:55.000000000","message":"Could you please specify what is this?","commit_id":"a30d4ab2605641830ea352a9b78fb95d5574e3c0"}],"horizon/utils/fields.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":115,"context_line":"    def render_option(self, selected_choices, option_value, option_label):"},{"line_number":116,"context_line":"        option_value \u003d force_unicode(option_value)"},{"line_number":117,"context_line":"        other_html \u003d (option_value in selected_choices) and \\"},{"line_number":118,"context_line":"            u\u0027 selected\u003d\"selected\"\u0027 or \u0027\u0027"},{"line_number":119,"context_line":"        if not isinstance(option_label, (basestring, Promise)):"},{"line_number":120,"context_line":"            for data_attr in self.data_attrs:"},{"line_number":121,"context_line":"                data_value \u003d conditional_escape("}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPlY%3D","line":118,"updated":"2013-10-01 17:14:18.000000000","message":"it is better to avoid a line break with backslash rather than this fix. This indentation change makes it difficult to understand the code. Consider:\n\n  other_html \u003d u\u0027 selected\u003d\"selected\"\u0027 if option_value in selected_choices else \u0027\u0027","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/admin/domains/workflows.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":74,"context_line":"                default \u003d getattr(settings,"},{"line_number":75,"context_line":"                                  \"OPENSTACK_KEYSTONE_DEFAULT_ROLE\", None)"},{"line_number":76,"context_line":"                msg \u003d _(\u0027Could not find default role \"%s\" in Keystone\u0027) % \\"},{"line_number":77,"context_line":"                    default"},{"line_number":78,"context_line":"                raise exceptions.NotFound(msg)"},{"line_number":79,"context_line":"        except Exception:"},{"line_number":80,"context_line":"            exceptions.handle(self.request,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPlk%3D","line":77,"updated":"2013-10-01 17:14:18.000000000","message":"it is better to avoid a line break with backslash rather than this fix.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/admin/instances/tables.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":109,"context_line":"                           status\u003dTrue,"},{"line_number":110,"context_line":"                           status_choices\u003dSTATUS_CHOICES,"},{"line_number":111,"context_line":"                           display_choices\u003d"},{"line_number":112,"context_line":"                           project_tables.STATUS_DISPLAY_CHOICES)"},{"line_number":113,"context_line":"    task \u003d tables.Column(\"OS-EXT-STS:task_state\","},{"line_number":114,"context_line":"                         verbose_name\u003d_(\"Task\"),"},{"line_number":115,"context_line":"                         filters\u003d(title, filters.replace_underscores),"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPkc%3D","line":112,"updated":"2013-10-01 17:14:18.000000000","message":"Hmm.. difficult to read the code...","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/admin/metering/tables.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":31,"context_line":"        return [resource for resource in resources"},{"line_number":32,"context_line":"                if q in resource.resource.lower() or"},{"line_number":33,"context_line":"                q in resource.tenant.lower() or"},{"line_number":34,"context_line":"                q in resource.user.lower()]"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"def get_status(fields):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPj0%3D","line":34,"updated":"2013-10-01 17:14:18.000000000","message":"Please consider adding ( after \"if\" at L.32. By doing it all logic have the same indent level and it makes easier to read.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/admin/metering/tests.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":268,"context_line":"                         \u0027\"x\": \"2012-12-21T11:00:55\"}], \u0027"},{"line_number":269,"context_line":"                         \u0027\"name\": \"\\\\u4e91\\\\u89c4\\\\u5219\", \u0027"},{"line_number":270,"context_line":"                         \u0027\"unit\": \"\"}], \u0027"},{"line_number":271,"context_line":"                         \u0027\"settings\": {}}\u0027])"},{"line_number":272,"context_line":""},{"line_number":273,"context_line":"    @test.create_stubs({api.keystone: (\u0027tenant_list\u0027,)})"},{"line_number":274,"context_line":"    def test_stats_for_line_chart_attr_max(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPjU%3D","line":271,"updated":"2013-10-01 17:14:18.000000000","message":"Please consider changing the position of line breaks.\nThe proposed indentation looks difficult to understand.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":307,"context_line":"                          \u0027{\"data\": [{\"y\": 9, \u0027"},{"line_number":308,"context_line":"                          \u0027\"x\": \"2012-12-21T11:00:55\"}], \u0027"},{"line_number":309,"context_line":"                          \u0027\"name\": \"\\\\u4e91\\\\u89c4\\\\u5219\", \u0027"},{"line_number":310,"context_line":"                          \u0027\"unit\": \"\"}], \u0027"},{"line_number":311,"context_line":"                          \u0027\"settings\": {}}\u0027])"},{"line_number":312,"context_line":""},{"line_number":313,"context_line":"    def test_stats_for_line_chart_no_group_by(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPjQ%3D","line":310,"updated":"2013-10-01 17:14:18.000000000","message":"Ditto","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":341,"context_line":"                          \u0027{\"data\": [{\"y\": 4, \u0027"},{"line_number":342,"context_line":"                          \u0027\"x\": \"2012-12-21T11:00:55\"}], \u0027"},{"line_number":343,"context_line":"                          \u0027\"name\": \"fake_resource_id2\", \u0027"},{"line_number":344,"context_line":"                          \u0027\"unit\": \"\"}], \u0027"},{"line_number":345,"context_line":"                          \u0027\"settings\": {}}\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPjE%3D","line":344,"updated":"2013-10-01 17:14:18.000000000","message":"Ditto","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/admin/projects/workflows.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":409,"context_line":"                                         \u0027project members%(group_msg)s and \u0027"},{"line_number":410,"context_line":"                                         \u0027set project quotas.\u0027)"},{"line_number":411,"context_line":"                              % {\u0027users_to_add\u0027: users_to_add,"},{"line_number":412,"context_line":"                                 \u0027group_msg\u0027: group_msg})"},{"line_number":413,"context_line":""},{"line_number":414,"context_line":"        if PROJECT_GROUP_ENABLED:"},{"line_number":415,"context_line":"            # update project groups"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPiU%3D","line":412,"updated":"2013-10-01 17:14:18.000000000","message":"These lines is better to align with _().","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":626,"context_line":"                                         \u0027 project members%(group_msg)s and \u0027"},{"line_number":627,"context_line":"                                         \u0027update project quotas.\u0027)"},{"line_number":628,"context_line":"                              % {\u0027users_to_modify\u0027: users_to_modify,"},{"line_number":629,"context_line":"                                 \u0027group_msg\u0027: group_msg})"},{"line_number":630,"context_line":"            return True"},{"line_number":631,"context_line":""},{"line_number":632,"context_line":"        if PROJECT_GROUP_ENABLED:"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPiQ%3D","line":629,"updated":"2013-10-01 17:14:18.000000000","message":"Ditto","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/admin/volumes/forms.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":36,"context_line":"            volume_type \u003d cinder.volume_type_create(request,"},{"line_number":37,"context_line":"                                                    data[\u0027name\u0027])"},{"line_number":38,"context_line":"            messages.success(request, _(\u0027Successfully created volume type: %s\u0027)"},{"line_number":39,"context_line":"                             % data[\u0027name\u0027])"},{"line_number":40,"context_line":"            return volume_type"},{"line_number":41,"context_line":"        except Exception:"},{"line_number":42,"context_line":"            exceptions.handle(request,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPhk%3D","line":39,"updated":"2013-10-01 17:14:18.000000000","message":"This line is better to be aligned with _() rather than \u0027request\u0027. It is a part of second argument.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/project/access_and_security/keypairs/forms.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":58,"context_line":"                                              data[\u0027name\u0027],"},{"line_number":59,"context_line":"                                              data[\u0027public_key\u0027])"},{"line_number":60,"context_line":"            messages.success(request, _(\u0027Successfully imported public key: %s\u0027)"},{"line_number":61,"context_line":"                             % data[\u0027name\u0027])"},{"line_number":62,"context_line":"            return keypair"},{"line_number":63,"context_line":"        except Exception:"},{"line_number":64,"context_line":"            exceptions.handle(request, ignore\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPk4%3D","line":61,"updated":"2013-10-01 17:14:18.000000000","message":"Honestly i don\u0027t like this type of indentation :-(","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/project/instances/tests.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":845,"context_line":"                                                \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":846,"context_line":"            .AndReturn([self.images.list(), False])"},{"line_number":847,"context_line":"        api.glance.image_list_detailed(IsA(http.HttpRequest),"},{"line_number":848,"context_line":"            filters\u003d{\u0027property-owner_id\u0027: self.tenant.id,"},{"line_number":849,"context_line":"                     \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":850,"context_line":"            .AndReturn([[], False])"},{"line_number":851,"context_line":"        api.neutron.network_list(IsA(http.HttpRequest),"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPgw%3D","line":848,"updated":"2013-10-01 17:14:18.000000000","message":"Usual indentation for such case is to move IsA() into a new line. Like L.1630 below.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/project/loadbalancers/forms.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":146,"context_line":"                            \u0027description\u0027: context[\u0027description\u0027],"},{"line_number":147,"context_line":"                            \u0027pool_id\u0027: context[\u0027pool_id\u0027],"},{"line_number":148,"context_line":"                            \u0027session_persistence\u0027:"},{"line_number":149,"context_line":"                            context[\u0027session_persistence\u0027],"},{"line_number":150,"context_line":"                            \u0027connection_limit\u0027: context[\u0027connection_limit\u0027],"},{"line_number":151,"context_line":"                            \u0027admin_state_up\u0027: context[\u0027admin_state_up\u0027],"},{"line_number":152,"context_line":"                            }}"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPgo%3D","line":149,"updated":"2013-10-01 17:14:18.000000000","message":"My prefernce is the old one.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":243,"context_line":"                    \u0027max_retries\u0027: context[\u0027max_retries\u0027],"},{"line_number":244,"context_line":"                    \u0027admin_state_up\u0027: context[\u0027admin_state_up\u0027]}}"},{"line_number":245,"context_line":"            monitor \u003d api.lbaas.pool_health_monitor_update(request,"},{"line_number":246,"context_line":"                context[\u0027monitor_id\u0027], **data)"},{"line_number":247,"context_line":"            msg \u003d _(\u0027Health monitor %s was successfully updated.\u0027)\\"},{"line_number":248,"context_line":"                % context[\u0027monitor_id\u0027]"},{"line_number":249,"context_line":"            LOG.debug(msg)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPgg%3D","line":246,"updated":"2013-10-01 17:14:18.000000000","message":"Pleaase consider inserting  a new line before \"request\".","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/project/volumes/tests.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":62,"context_line":"                                       filters\u003d{\u0027is_public\u0027: True,"},{"line_number":63,"context_line":"                                                \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":64,"context_line":"                  .AndReturn([self.images.list(), False])"},{"line_number":65,"context_line":"        api.glance.image_list_detailed(IsA(http.HttpRequest),"},{"line_number":66,"context_line":"            filters\u003d{\u0027property-owner_id\u0027: self.tenant.id,"},{"line_number":67,"context_line":"                     \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":68,"context_line":"            .AndReturn([[], False])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPf0%3D","line":65,"updated":"2013-10-01 17:14:18.000000000","message":"A line break before IaA()","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":111,"context_line":"                                       filters\u003d{\u0027is_public\u0027: True,"},{"line_number":112,"context_line":"                                                \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":113,"context_line":"                  .AndReturn([self.images.list(), False])"},{"line_number":114,"context_line":"        api.glance.image_list_detailed(IsA(http.HttpRequest),"},{"line_number":115,"context_line":"            filters\u003d{\u0027property-owner_id\u0027: self.tenant.id,"},{"line_number":116,"context_line":"                     \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":117,"context_line":"            .AndReturn([[], False])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPfs%3D","line":114,"updated":"2013-10-01 17:14:18.000000000","message":"it is better to avoid a line break with backslash rather than this fix.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":212,"context_line":"                                       filters\u003d{\u0027is_public\u0027: True,"},{"line_number":213,"context_line":"                                                \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":214,"context_line":"                  .AndReturn([self.images.list(), False])"},{"line_number":215,"context_line":"        api.glance.image_list_detailed(IsA(http.HttpRequest),"},{"line_number":216,"context_line":"            filters\u003d{\u0027property-owner_id\u0027: self.tenant.id,"},{"line_number":217,"context_line":"                     \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":218,"context_line":"            .AndReturn([[], False])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPfo%3D","line":215,"updated":"2013-10-01 17:14:18.000000000","message":"it is better to avoid a line break with backslash rather than this fix.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":352,"context_line":"                                       filters\u003d{\u0027is_public\u0027: True,"},{"line_number":353,"context_line":"                                                \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":354,"context_line":"                  .AndReturn([self.images.list(), False])"},{"line_number":355,"context_line":"        api.glance.image_list_detailed(IsA(http.HttpRequest),"},{"line_number":356,"context_line":"            filters\u003d{\u0027property-owner_id\u0027: self.tenant.id,"},{"line_number":357,"context_line":"                     \u0027status\u0027: \u0027active\u0027}) \\"},{"line_number":358,"context_line":"            .AndReturn([[], False])"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPfk%3D","line":355,"updated":"2013-10-01 17:14:18.000000000","message":"it is better to avoid a line break with backslash rather than this fix.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":647,"context_line":""},{"line_number":648,"context_line":"        create_link \u003d tables.CreateVolume()"},{"line_number":649,"context_line":"        url \u003d create_link.get_link_url()"},{"line_number":650,"context_line":"        classes \u003d list(create_link.get_default_classes())\\"},{"line_number":651,"context_line":"            + list(create_link.classes)"},{"line_number":652,"context_line":"        link_name \u003d \"%s (%s)\" % (unicode(create_link.verbose_name),"},{"line_number":653,"context_line":"                                 \"Quota exceeded\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPfE%3D","line":650,"updated":"2013-10-01 17:14:18.000000000","message":"Insert a opening parenthesis before \"list\".","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":651,"context_line":"            + list(create_link.classes)"},{"line_number":652,"context_line":"        link_name \u003d \"%s (%s)\" % (unicode(create_link.verbose_name),"},{"line_number":653,"context_line":"                                 \"Quota exceeded\")"},{"line_number":654,"context_line":"        expected_string \u003d \"\u003ca href\u003d\u0027%s\u0027 title\u003d\u0027%s\u0027  class\u003d\u0027%s disabled\u0027 \"\\"},{"line_number":655,"context_line":"                          \"id\u003d\u0027volumes__action_create\u0027\u003e%s\u003c/a\u003e\" \\"},{"line_number":656,"context_line":"            % (url, link_name, \" \".join(classes), link_name)"},{"line_number":657,"context_line":"        self.assertContains(res, expected_string, html\u003dTrue,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPew%3D","line":654,"updated":"2013-10-01 17:14:18.000000000","message":"Insert a opening parenthesis before \"\u003ca href\"...","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}],"openstack_dashboard/dashboards/router/nexus1000v/forms.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":70,"context_line":"                                                 \u0027data-switch-on\u0027:"},{"line_number":71,"context_line":"                                                 \u0027segtype\u0027,"},{"line_number":72,"context_line":"                                                 \u0027data-segtype-vxlan\u0027:"},{"line_number":73,"context_line":"                                                 _(\"Multicast IP Range\")}))"},{"line_number":74,"context_line":"    physical_network \u003d forms.CharField(max_length\u003d255,"},{"line_number":75,"context_line":"                                       label\u003d_(\"Physical Network\"),"},{"line_number":76,"context_line":"                                       required\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPgA%3D","line":73,"updated":"2013-10-01 17:14:18.000000000","message":"A bit difficult to read. Please consider a line break at \"max_length\u003d30\" at L.65.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":78,"context_line":"                                       (attrs\u003d{\u0027class\u0027: \u0027switched\u0027,"},{"line_number":79,"context_line":"                                               \u0027data-switch-on\u0027: \u0027segtype\u0027,"},{"line_number":80,"context_line":"                                               \u0027data-segtype-vlan\u0027:"},{"line_number":81,"context_line":"                                               _(\"Physical Network\")}))"},{"line_number":82,"context_line":"    project_id \u003d forms.ChoiceField(label\u003d_(\"Project\"),"},{"line_number":83,"context_line":"                                  required\u003dFalse)"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPf8%3D","line":81,"updated":"2013-10-01 17:14:18.000000000","message":"Same as the above.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":9571,"name":"Gabriel Pettier","email":"gabriel.pettier@gmail.com","username":"GabrielPettier"},"change_message_id":"42bf0fc57beb8ec90c92352d72086f5d644c6698","unresolved":false,"context_lines":[{"line_number":68,"context_line":"                                         widget\u003dforms.TextInput"},{"line_number":69,"context_line":"                                         (attrs\u003d{\u0027class\u0027: \u0027switched\u0027,"},{"line_number":70,"context_line":"                                                 \u0027data-switch-on\u0027:"},{"line_number":71,"context_line":"                                                 \u0027segtype\u0027,"},{"line_number":72,"context_line":"                                                 \u0027data-segtype-vxlan\u0027:"},{"line_number":73,"context_line":"                                                 _(\"Multicast IP Range\")}))"},{"line_number":74,"context_line":"    physical_network \u003d forms.CharField(max_length\u003d255,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F96No%3D","line":71,"updated":"2013-12-06 10:01:47.000000000","message":"would be nicer to have the value on the same line as the key, no? if there is not enough space, adding a \\n before max_lenth and unindenting the whole block could be a better solution.","commit_id":"a30d4ab2605641830ea352a9b78fb95d5574e3c0"},{"author":{"_account_id":9571,"name":"Gabriel Pettier","email":"gabriel.pettier@gmail.com","username":"GabrielPettier"},"change_message_id":"42bf0fc57beb8ec90c92352d72086f5d644c6698","unresolved":false,"context_lines":[{"line_number":77,"context_line":"                                       widget\u003dforms.TextInput"},{"line_number":78,"context_line":"                                       (attrs\u003d{\u0027class\u0027: \u0027switched\u0027,"},{"line_number":79,"context_line":"                                               \u0027data-switch-on\u0027: \u0027segtype\u0027,"},{"line_number":80,"context_line":"                                               \u0027data-segtype-vlan\u0027:"},{"line_number":81,"context_line":"                                               _(\"Physical Network\")}))"},{"line_number":82,"context_line":"    project_id \u003d forms.ChoiceField(label\u003d_(\"Project\"),"},{"line_number":83,"context_line":"                                  required\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F96Nk%3D","line":80,"updated":"2013-12-06 10:01:47.000000000","message":"ditto","commit_id":"a30d4ab2605641830ea352a9b78fb95d5574e3c0"}],"openstack_dashboard/test/test_data/neutron_data.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":280,"context_line":"    router_dict \u003d {\u0027id\u0027: \u0027279989f7-54bb-41d9-ba42-0d61f12fda61\u0027,"},{"line_number":281,"context_line":"                   \u0027name\u0027: \u0027router1\u0027,"},{"line_number":282,"context_line":"                   \u0027external_gateway_info\u0027:"},{"line_number":283,"context_line":"                   {\u0027network_id\u0027: ext_net[\u0027id\u0027]},"},{"line_number":284,"context_line":"                   \u0027tenant_id\u0027: \u00271\u0027}"},{"line_number":285,"context_line":"    TEST.api_routers.add(router_dict)"},{"line_number":286,"context_line":"    TEST.routers.add(neutron.Router(router_dict))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPnY%3D","line":283,"updated":"2013-10-01 17:14:18.000000000","message":"Hmm.. this type of indentation level change looks negative effect.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"a6162c03a3dcbb4ea3721d2f61ef2c7749fbd7b6","unresolved":false,"context_lines":[{"line_number":790,"context_line":"    firewall_dict \u003d {\u0027id\u0027: \u00278913dde8-4915-4b90-8d3e-b95eeedb0d49\u0027,"},{"line_number":791,"context_line":"                     \u0027tenant_id\u0027: \u00271\u0027,"},{"line_number":792,"context_line":"                     \u0027firewall_policy_id\u0027:"},{"line_number":793,"context_line":"                     \u0027abcdef-c3eb-4fee-9763-12de3338041e\u0027,"},{"line_number":794,"context_line":"                     \u0027name\u0027: \u0027firewall1\u0027,"},{"line_number":795,"context_line":"                     \u0027description\u0027: \u0027firewall description\u0027,"},{"line_number":796,"context_line":"                     \u0027status\u0027: \u0027PENDING_CREATE\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAS3%2F%2FPnM%3D","line":793,"updated":"2013-10-01 17:14:18.000000000","message":"I seems a bit difficult to understand. Instead it is better to add a new line before \u0027id\u0027 at L.790.","commit_id":"4939a648cca98d89188c4d2bd98de4242a146326"}]}