Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[Workflow] add get available on workflow component#26042

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed
ronfroy wants to merge1 commit intosymfony:masterfromronfroy:add-get-available-transitions-on-workflow
Closed

[Workflow] add get available on workflow component#26042

ronfroy wants to merge1 commit intosymfony:masterfromronfroy:add-get-available-transitions-on-workflow

Conversation

@ronfroy
Copy link
Contributor

QA
Branch?master
Bug fix?no
New feature?yes
BC breaks?no
Deprecations?no
Tests pass?yes
LicenseMIT

Returns all available transitions (without taking care of guard handlers). It's may be really useful in the case we have to get the available and not juste enabled.

@nicolas-grekasnicolas-grekas added this to the4.1 milestoneFeb 5, 2018
@lyrixx
Copy link
Member

Hello@ronfroy

Could you describe your use case for this feature? For now, I'm a bit reluctant to add this feature to Symfony.

@ronfroy
Copy link
ContributorAuthor

@lyrixx I want to validate a transition with multiple error depending of rich condition (sub entity, author...) after an user action. In my idea i get all available transition and call "->can" to generate error with guard listener.

Maybe you have a better solution?

@ronfroy
Copy link
ContributorAuthor

@lyrixx Maybe we can just add a "validate" method to the workflow that call only a throw validation event.

@ronfroy
Copy link
ContributorAuthor

@lyrixxhttps://github.com/symfony/symfony/pull/26076/files this is maybe a better way to achieve this?

@lyrixx
Copy link
Member

With#26076 you will be able to know why a transition is blocked. I think it's the way to go indeed.

I'm closing this PR, but don't hesitate to re-open it if something is missing.

@lyrixxlyrixx closed thisFeb 7, 2018
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

4.1

Development

Successfully merging this pull request may close these issues.

4 participants

@ronfroy@lyrixx@nicolas-grekas@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp