)]}'
{"keystone/assignment/backends/sql.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"390ced4c7a39aa0a4d179413b13e560af78054ea","unresolved":false,"context_lines":[{"line_number":330,"context_line":""},{"line_number":331,"context_line":"        def _get_roles_for_groups_on_project(group_ids, project_id):"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":"            def _role_ids_to_dicts(session, ids):"},{"line_number":334,"context_line":"                if not ids:"},{"line_number":335,"context_line":"                    return []"},{"line_number":336,"context_line":"                else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_5996bbf1","line":333,"updated":"2014-11-12 07:51:05.000000000","message":"the fact that this is a function within a function within a function, kinda bothers me","commit_id":"bfbe1ee96c871163e4af0eca7568e13be72b8fde"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"3365e36adab41a718beddf06d50ba89e75dedd0a","unresolved":false,"context_lines":[{"line_number":330,"context_line":""},{"line_number":331,"context_line":"        def _get_roles_for_groups_on_project(group_ids, project_id):"},{"line_number":332,"context_line":""},{"line_number":333,"context_line":"            def _role_ids_to_dicts(session, ids):"},{"line_number":334,"context_line":"                if not ids:"},{"line_number":335,"context_line":"                    return []"},{"line_number":336,"context_line":"                else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_ee95d640","line":333,"in_reply_to":"5a890539_5996bbf1","updated":"2014-11-12 10:10:22.000000000","message":"I know...balance between keeping as many code blocks as they are vs re-factoring too much.  If it\u0027s any consolation, this gets removed further up the patch change once we split resources and assignments.","commit_id":"bfbe1ee96c871163e4af0eca7568e13be72b8fde"}],"keystone/tests/test_backend.py":[{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"bedebc83b5c869373ea7a320851cb790365e432d","unresolved":false,"context_lines":[{"line_number":3282,"context_line":"                                         project_id\u003dproject2[\u0027id\u0027],"},{"line_number":3283,"context_line":"                                         role_id\u003drole_list[3][\u0027id\u0027])"},{"line_number":3284,"context_line":""},{"line_number":3285,"context_line":"        # ...and a few of spoiler assignments to Domain2/Project4"},{"line_number":3286,"context_line":"        self.assignment_api.create_grant(group_id\u003dgroup_list[4][\u0027id\u0027],"},{"line_number":3287,"context_line":"                                         domain_id\u003ddomain2[\u0027id\u0027],"},{"line_number":3288,"context_line":"                                         role_id\u003drole_list[4][\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_210a9f21","line":3285,"updated":"2014-11-11 20:11:18.000000000","message":"Project4 or Domain4?","commit_id":"bfbe1ee96c871163e4af0eca7568e13be72b8fde"},{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"3365e36adab41a718beddf06d50ba89e75dedd0a","unresolved":false,"context_lines":[{"line_number":3282,"context_line":"                                         project_id\u003dproject2[\u0027id\u0027],"},{"line_number":3283,"context_line":"                                         role_id\u003drole_list[3][\u0027id\u0027])"},{"line_number":3284,"context_line":""},{"line_number":3285,"context_line":"        # ...and a few of spoiler assignments to Domain2/Project4"},{"line_number":3286,"context_line":"        self.assignment_api.create_grant(group_id\u003dgroup_list[4][\u0027id\u0027],"},{"line_number":3287,"context_line":"                                         domain_id\u003ddomain2[\u0027id\u0027],"},{"line_number":3288,"context_line":"                                         role_id\u003drole_list[4][\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_2ec99e78","line":3285,"in_reply_to":"5a890539_210a9f21","updated":"2014-11-12 10:10:22.000000000","message":"I think it is correct as is...we only have 2 two domains...domain1 has projects1-3, and domain2 has project4","commit_id":"bfbe1ee96c871163e4af0eca7568e13be72b8fde"}]}