)]}'
{"keystone/logic/types/auth.py":[{"author":{"_account_id":8,"name":"Yogeshwar Srikrishnan","email":"yoga80@yahoo.com","username":"yogesh-srikrishnan"},"change_message_id":"fce3a73cb321a71ef6449b2b09e0a6e7811e0445","unresolved":false,"context_lines":[{"line_number":340,"context_line":"        if self.user.role_refs !\u003d None:"},{"line_number":341,"context_line":"            user.append(self.user.role_refs.to_dom())"},{"line_number":342,"context_line":""},{"line_number":343,"context_line":"        if self.base_urls is not None or len(self.base_urls) \u003e 0:"},{"line_number":344,"context_line":"            service_catalog \u003d etree.Element(\"serviceCatalog\")"},{"line_number":345,"context_line":"            for key, key_base_urls in self.d.items():"},{"line_number":346,"context_line":"                dservice \u003d db_api.SERVICE.get(key)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAC3%2F%2F6qc%3D","line":343,"updated":"2011-12-24 00:24:39.000000000","message":"should this not be if self.base_urls is not None and len(self.base_urls) \u003e 0:","commit_id":"b725ad943eac1326429fa2a4f00f4f8104a6870c"},{"author":{"_account_id":8,"name":"Yogeshwar Srikrishnan","email":"yoga80@yahoo.com","username":"yogesh-srikrishnan"},"change_message_id":"bf2ebea72be3470599bf20a584a87cd7c091577b","unresolved":false,"context_lines":[{"line_number":401,"context_line":"                        base_url_item \u003d getattr(base_url, url_kind + \"_url\")"},{"line_number":402,"context_line":"                        if base_url_item:"},{"line_number":403,"context_line":"                            if \u0027%tenant_id%\u0027 in base_url_item:"},{"line_number":404,"context_line":"                                if self.token.tenant:"},{"line_number":405,"context_line":"                                    # Don\u0027t return tenant endpoints if token"},{"line_number":406,"context_line":"                                    # not scoped to a tenant"},{"line_number":407,"context_line":"                                    endpoint[url_kind + \"URL\"] \u003d \\"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAC3%2F%2F6p0%3D","line":404,"updated":"2011-12-24 01:12:39.000000000","message":"method seems to be really complex.I cud refactor if u want?","commit_id":"b725ad943eac1326429fa2a4f00f4f8104a6870c"},{"author":{"_account_id":8,"name":"Yogeshwar Srikrishnan","email":"yoga80@yahoo.com","username":"yogesh-srikrishnan"},"change_message_id":"bf2ebea72be3470599bf20a584a87cd7c091577b","unresolved":false,"context_lines":[{"line_number":410,"context_line":"                                    include_this_endpoint \u003d True"},{"line_number":411,"context_line":"                            else:"},{"line_number":412,"context_line":"                                endpoint[url_kind + \"URL\"] \u003d base_url_item"},{"line_number":413,"context_line":"                                include_this_endpoint \u003d True"},{"line_number":414,"context_line":"                    if include_this_endpoint:"},{"line_number":415,"context_line":"                        endpoints.append(endpoint)"},{"line_number":416,"context_line":"                        dservice \u003d db_api.SERVICE.get(key)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAC3%2F%2F6p4%3D","line":413,"updated":"2011-12-24 01:12:39.000000000","message":"when would  include_this_endpoint be False?It seems to be true for both if and else.","commit_id":"b725ad943eac1326429fa2a4f00f4f8104a6870c"}]}
