Closed ghost closed 9 years ago
I removed status and canEdit from submitting, approving, and returning. CanEdit is purely dependant on the status and shouldn't be set seperately. We could refactor the model, but that's fairly intensive so I don't know if anyone wants to do that.
When submitting, approving or returning a claim, it might be better if the canEdit flag isn't passed as a parameter from the activity through the controller and finally to the claim model. Since whether a claim is editable is ultimately a model issue, the claim itself should make that decision.