-
Notifications
You must be signed in to change notification settings - Fork 17
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
Initial promotions implementation #1
Conversation
{ | ||
public function __construct($type) | ||
{ | ||
parent::__construct(sprintf('Promotion action of type "%s" already exist.', $type)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMHO messages should not be hardcoded in exceptions...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
at least it should be translated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Translated? This exception is thrown if you try to register action via container tag, and name is taken. :) (in most cases)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
agree in such case have no sense
YEAH! 👍 Nice work man! |
Initial promotions implementation
Update ActionChoiceType.php
* 1.11: Specify explicit priorities for all form type extensions Small CS fixes Line length fixes PHP 8 syntax in bundles vol.2 Typehint templating engines properly Fix line length issues PHP 8 syntax in bundles #1
No description provided.