)]}'
{"watcher/applier/default.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"ecb24efd3af72403bfaa6e2a56281ff2c515de40","unresolved":false,"context_lines":[{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def execute(self, action_plan_uuid):"},{"line_number":58,"context_line":"        LOG.debug(\"Executing action plan %s \", action_plan_uuid)"},{"line_number":59,"context_line":"        actions \u003d objects.ActionPlan.get_action_list_by_uuid("},{"line_number":60,"context_line":"            self.context,"},{"line_number":61,"context_line":"            action_plan_uuid)"},{"line_number":62,"context_line":"        return self.engine.execute(actions)"}],"source_content_type":"text/x-python","patch_set":3,"id":"dab17558_655489c9","line":61,"range":{"start_line":59,"start_character":18,"end_line":61,"end_character":29},"updated":"2016-05-17 07:47:34.000000000","message":"How about simply using:\n\nactions \u003d objects.Action.list(self.context, filters\u003d{\"action_plan_uuid\": action_plan_uuid})\n\n(didn\u0027t test it)","commit_id":"16364001024939f23917e5fc5cc1e7d08118cfaa"}],"watcher/objects/action_plan.py":[{"author":{"_account_id":18971,"name":"Vincent Françoise","email":"Vincent.FRANCOISE@b-com.com","username":"v-francoise"},"change_message_id":"ecb24efd3af72403bfaa6e2a56281ff2c515de40","unresolved":false,"context_lines":[{"line_number":143,"context_line":"        return action_plan"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"    @classmethod"},{"line_number":146,"context_line":"    def get_action_list_by_uuid(cls, context, uuid):"},{"line_number":147,"context_line":"        \"\"\"Get an action list based on an uuid."},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"        :param context: Security context"}],"source_content_type":"text/x-python","patch_set":3,"id":"dab17558_227a77bd","line":146,"range":{"start_line":146,"start_character":4,"end_line":146,"end_character":52},"updated":"2016-05-17 07:47:34.000000000","message":"IMHO, the is not the best way to do it. We should do this using a filter mapping to keep it generic.\nThis way, we also do not add a new method to the action_plan object.\n\nBesides, this method is returning a list of Action objects although we should only return with ActionPlan objects in this class. Hence, I would also suggest to move this piece of logic to watcher/objects/action.py although I strongly suspect that you can use the \"filters\" parameter over there to get waht you want.","commit_id":"16364001024939f23917e5fc5cc1e7d08118cfaa"}]}