This merge request was automatically created by the Project Update Bot. It contains the changes from run 11-109123.
Thanks, please check the feedback.
I would not do that.
* The number of times the current user has submitted the form.
Hmm, not sure.
Let's remove this empty line.
I am not sure if order of property names has any code styles, but I would put it after $defaultTheme
to have alphabet order.
Also not sure if t() is necessary.
* {@inheritdoc}
$this->submitForm($message, 'Send message');
Please use sprintf().
No need t() in test.
$this->submitForm([], 'Apply to selected items');
What is the reason for changing the logic?
StringTranslationTrait is missing.
Closes #3403408
The article before number of times is correct.
I would rather change the description, than removing a word that is necessary. A method description should not say what the method returns, since that is already documented with a specific documentation tag.
There are six parameters. Is there any reason for adding type hinting to only a single parameter, considering that $message
is a string?
I believe that is because it is defined as array.
Updated.
$this->assertCount(1, $elements, 'All contact message operations are found.');
Please use assertCount()