mirror of
https://github.com/opf/openproject.git
synced 2026-06-14 03:30:14 +00:00
40aff02f11
This is making the interface of BaseCallable more consistent. Previously it was unclear, whether the params passed to perform should be used to reference parameters or whether the attribute accessor should be used. Different code used different approaches. To apply this change more consistently, BaseContracted also removed params from the methods called inside its own perform method, e.g. before_perform or after_validate. At a later point we might consider removing this argument from other large inheritors as well, for example `BaseServices::SetAttributes#set_attributes`.