-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Action after run result #19870
base: master
Are you sure you want to change the base?
Action after run result #19870
Conversation
PR Summary
|
This is a big BC break, not sure if we want to change it like that. |
@rhertogh do you have a use-case for it or is it based mainly on the contradiction of current code with phpdoc? |
@samdark I've an actual use case for it where I need to run some code in the |
@rhertogh maybe. I've posted there in a discussion. |
The
\yii\base\Action::afterRun()
documentation states:Currently post-processing is not possible since the
$result
is not passed to theafterRun()
function in the\yii\base\Action::runWithParams()
function.This was discussed earlier, but since Yii 2.1 is cancelled I would like to re-submit this feature.
ToDo: