)]}'
{"openstack_dashboard/dashboards/admin/users/forms.py":[{"author":{"_account_id":1941,"name":"Lin Hua Cheng","email":"os.lcheng@gmail.com","username":"lin-hua-cheng"},"change_message_id":"e740dc5e2a3f5fb8b95772bf418a24f9e39f7875","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        # if user is already set (update action), list only projects which"},{"line_number":45,"context_line":"        # the user has access to"},{"line_number":46,"context_line":"        user_id \u003d kwargs[\u0027initial\u0027].get(\u0027id\u0027, None)"},{"line_number":47,"context_line":"        projects, has_more \u003d api.keystone.tenant_list(request, user\u003duser_id)"},{"line_number":48,"context_line":"        for project in projects:"},{"line_number":49,"context_line":"            if project.enabled:"},{"line_number":50,"context_line":"                project_choices.append((project.id, project.name))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2Fts4%3D","line":47,"updated":"2013-06-12 04:54:09.000000000","message":"Isn\u0027t the change should only be applied for Update User? This also affects Create User.","commit_id":"15e938b4212b5ead85f385573e92b9246c22fe58"},{"author":{"_account_id":5733,"name":"Kieran Spear","email":"kispear@gmail.com","username":"kspear"},"change_message_id":"0d6a7321e9835b35542742577b50b9e92ee6e5a2","unresolved":false,"context_lines":[{"line_number":44,"context_line":"        # if user is already set (update action), list only projects which"},{"line_number":45,"context_line":"        # the user has access to"},{"line_number":46,"context_line":"        user_id \u003d kwargs[\u0027initial\u0027].get(\u0027id\u0027, None)"},{"line_number":47,"context_line":"        projects, has_more \u003d api.keystone.tenant_list(request, user\u003duser_id)"},{"line_number":48,"context_line":"        for project in projects:"},{"line_number":49,"context_line":"            if project.enabled:"},{"line_number":50,"context_line":"                project_choices.append((project.id, project.name))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2Ftnc%3D","line":47,"in_reply_to":"AAAAPX%2F%2Fts4%3D","updated":"2013-06-12 06:28:32.000000000","message":"In the Create User case user_id will be None here, so the behaviour should be the same as before.","commit_id":"15e938b4212b5ead85f385573e92b9246c22fe58"}],"openstack_dashboard/dashboards/admin/users/tests.py":[{"author":{"_account_id":5733,"name":"Kieran Spear","email":"kispear@gmail.com","username":"kspear"},"change_message_id":"0d6a7321e9835b35542742577b50b9e92ee6e5a2","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        user \u003d self.users.get(id\u003d\"1\")"},{"line_number":56,"context_line":"        role \u003d self.roles.first()"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        api.keystone.tenant_list(IgnoreArg(), user\u003dIgnoreArg()) \\"},{"line_number":59,"context_line":"            .AndReturn([self.tenants.list(), False])"},{"line_number":60,"context_line":"        api.keystone.user_create(IgnoreArg(),"},{"line_number":61,"context_line":"                                 user.name,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPX%2F%2FtnQ%3D","line":58,"updated":"2013-06-12 06:28:32.000000000","message":"You should be putting in the expected id here, since it\u0027s quite important to the correctness of the form. It also makes it easy to prove that user create is not affected by your change.","commit_id":"15e938b4212b5ead85f385573e92b9246c22fe58"}]}