Skip to content
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

ChangeStatusAction.php #55

Open
makroxyz opened this issue Jun 25, 2018 · 1 comment
Open

ChangeStatusAction.php #55

makroxyz opened this issue Jun 25, 2018 · 1 comment

Comments

@makroxyz
Copy link

Just a question.
Why in this action you are using sendToStatus and then $model->save()?
In this way if model has WorkflowValidator it doesn't validate... transition has already done so validator can't reach any scenarios.

Thanks

@raoul2000
Copy link
Owner

hi @makroxyz,
yes, of course you are right ! I This action was added by a PR and I didn't anticipate this behavior... maybe I should re-implement it or just remove it (or just highlight this behavior in the doc)
Until I do something about it use it taking this limitation into account.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants