- Issue created by @joachim
- First commit to issue fork.
- Status changed to Needs review
11 months ago 4:29am 11 January 2024 - Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7last update
11 months ago Waiting for branch to pass - 🇮🇳India viren18febS
I have modified the doc comment and added a patch for this.
- Status changed to Needs work
11 months ago 11:25am 11 January 2024 - 🇬🇧United Kingdom joachim
Thanks for the patch!
-
+++ b/modules/tome_static/src/Event/ModifyDestinationEvent.php @@ -5,7 +5,9 @@ namespace Drupal\tome_static\Event; + * Defines the ModifyDestinationEvent class.We don't need this line, it just repeats the class name.
-
+++ b/modules/tome_static/src/Event/ModifyDestinationEvent.php @@ -5,7 +5,9 @@ namespace Drupal\tome_static\Event; + * Allows modules to provide the opportunity to get and set the destination path.This is a good start, but it goes over the line limit.
We can shorten it easily though - we don't need to say 'provide the opportunity'.
So something like:
'Allows modules to change the destination path of a static path.'
-
- Status changed to Needs review
11 months ago 12:04pm 11 January 2024 - Open on Drupal.org →Core: 9.5.x + Environment: PHP 7.3 & MySQL 5.7last update
11 months ago Waiting for branch to pass - Status changed to RTBC
11 months ago 2:52pm 11 January 2024