Christophe Coevoet

Results 2811 comments of Christophe Coevoet

depends how you want to handle the "bonus point".

@phansys could you please tag a new release ?

Done in https://github.com/nelmio/NelmioSecurityBundle/pull/386 but not released yet.

> You will also be able to use enums instead of string. this is not true. Enums are objects in PHP, and so cannot be used in places expecting strings....

@Jean-Beru `Status::Published->value` is still a string, not an enum (and so you are just using a more verbose syntax for string constants. You are not actually using enums)

the API proposed by @GromNaN works for state machines where each transition has a single place for `from` and `to`. but it does not work for generic workflows where transitions...

@thomassebert 2 transitions wit the same name and a different `from` place are not equivalent at all to a single transition with 2 `from` places. Note that transition names are...

> How can you call the "can('transition_name')" and "apply('transition_name')" functions if there is multiple transitions with the same name ? Those `can` and `apply` method know the current marking. That's...

@lyrixx a class meant to define the workflow metadata through attributes should probably not extend the Workflow class (which is the class used for the runtime service). It would create...

> Is transparent. It just works. I swap the service created by symfony. Not true. We have different dumping for state machines vs full featured workflows, which you lost here.