- Issue created by @Liam Morland
- Merge request !10156Issue #3487014: Fix documentation for optional params in MessengerInterface → (Closed) created by Liam Morland
- 🇺🇸United States smustgrave
Seems pretty straight forward and looks like all $message have been updated.
- Status changed to RTBC
4 months ago 2:12am 22 January 2025 - 🇳🇿New Zealand quietone
This was added in #2278383: Create an injectible service for drupal_set_message() → where the (optional) was valid for drupal_set_message. Should this be restoring the default value instead? I have asked in committer slack about this.
- First commit to issue fork.
-
longwave →
committed 957c16fc on 10.4.x
Issue #3487014 by liam morland, nod_, smustgrave, quietone: Fix...
-
longwave →
committed 957c16fc on 10.4.x
-
longwave →
committed 46cf264b on 10.5.x
Issue #3487014 by liam morland, nod_, smustgrave, quietone: Fix...
-
longwave →
committed 46cf264b on 10.5.x
-
longwave →
committed 8877da3b on 11.1.x
Issue #3487014 by liam morland, nod_, smustgrave, quietone: Fix...
-
longwave →
committed 8877da3b on 11.1.x
-
longwave →
committed 4899db93 on 11.x
Issue #3487014 by liam morland, nod_, smustgrave, quietone: Fix...
-
longwave →
committed 4899db93 on 11.x
- 🇬🇧United Kingdom longwave UK
This is valid, the message is no longer optional.
drupal_set_message()with no arguments returned the static set of messages, now that is handled by a separate method in the messenger service.Backported down to 10.4.x as an eligible documentation fix.
Committed and pushed 4899db93d99 to 11.x and 8877da3b01b to 11.1.x and 46cf264bf32 to 10.5.x and 957c16fc138 to 10.4.x. Thanks!
Automatically closed - issue fixed for 2 weeks with no activity.