)]}'
{"keystone/trust/controllers.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"50177514217b7bea6ab269653380765923ceaf37","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        trust \u003d self.trust_api.get_trust(trust_id)"},{"line_number":53,"context_line":"        _trustor_trustee_only(trust, request.context.user_id)"},{"line_number":54,"context_line":"        self._fill_in_roles(request.context_dict, trust,"},{"line_number":55,"context_line":"                            self.role_api.list_roles())"},{"line_number":56,"context_line":"        return TrustV3.wrap_member(request.context_dict, trust)"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    def _fill_in_roles(self, context, trust, all_roles):"}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_e9887124","side":"PARENT","line":55,"range":{"start_line":55,"start_character":28,"end_line":55,"end_character":55},"updated":"2016-08-05 05:02:40.000000000","message":"yay for removing this","commit_id":"5628e24de3dfb0b5c6bd9776d3dfa24ff68fb66d"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"50177514217b7bea6ab269653380765923ceaf37","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        self.identity_api.get_user(trust[\u0027trustee_user_id\u0027])"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"        # Normalize roles"},{"line_number":134,"context_line":"        normalized_roles \u003d self._normalize_role_list(trust.get(\u0027roles\u0027, []))"},{"line_number":135,"context_line":"        trust[\u0027roles\u0027] \u003d normalized_roles"},{"line_number":136,"context_line":"        self._require_trustor_has_role_in_project(trust)"},{"line_number":137,"context_line":"        trust[\u0027expires_at\u0027] \u003d self._parse_expiration_date("}],"source_content_type":"text/x-python","patch_set":1,"id":"bacf61ea_e9c7d10f","line":134,"range":{"start_line":134,"start_character":53,"end_line":134,"end_character":75},"updated":"2016-08-05 05:02:40.000000000","message":"may be easier to read if you just pass in the trust","commit_id":"9838e542af9bc69140b9036af7a9c4a09e86365b"},{"author":{"_account_id":13055,"name":"Alexander Makarov","email":"ajiekcahdp.makapob@gmail.com","username":"amakarov"},"change_message_id":"65894587aebd740018ca975649ef603a71b2287e","unresolved":false,"context_lines":[{"line_number":131,"context_line":"        self.identity_api.get_user(trust[\u0027trustee_user_id\u0027])"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"        # Normalize roles"},{"line_number":134,"context_line":"        normalized_roles \u003d self._normalize_role_list(trust.get(\u0027roles\u0027, []))"},{"line_number":135,"context_line":"        trust[\u0027roles\u0027] \u003d normalized_roles"},{"line_number":136,"context_line":"        self._require_trustor_has_role_in_project(trust)"},{"line_number":137,"context_line":"        trust[\u0027expires_at\u0027] \u003d self._parse_expiration_date("}],"source_content_type":"text/x-python","patch_set":1,"id":"9ad45d7e_dcf21f6f","line":134,"range":{"start_line":134,"start_character":53,"end_line":134,"end_character":75},"in_reply_to":"bacf61ea_e9c7d10f","updated":"2016-08-08 13:57:14.000000000","message":"I\u0027ve done it this way considering 2 things:\n\n1. It\u0027s better seen what\u0027s actually handled in called func.\n\n2. Called func is more focused on logic than on data extraction.","commit_id":"9838e542af9bc69140b9036af7a9c4a09e86365b"}]}