search

drush-ops / drush

Drush is a command-line shell and scripting interface for Drupal, a veritable Swiss Army knife designed to make life easier for those who spend their working hours hacking away at the command prompt.
https://www.drush.org
2.33k stars 1.08k forks source link

Slashes in URLs in doc comments of hook_post_update_NAME get stripped #6087

Open jurgenhaas opened 1 month ago

jurgenhaas commented 1 month ago

Here is an example code from the ECA module:

/**
 * Re-run the 2.0.0 post update hook.
 *
 * @see https://www.drupal.org/project/eca/issues/3460491
 */
function eca_post_update_rename_tokens_2_0_1(): void {
  eca_post_update_rename_tokens_2_0_0();
}

The output on the console when running drush updatedb strips the slashes from the URL, and it then looks like this:

 -------- --------------------- ------------- --------------------------------- 
  Module   Update ID             Type          Description                      
 -------- --------------------- ------------- --------------------------------- 
  eca      rename_tokens_2_0_2   post-update   Re-run the 2.0.0 post update     
                                               hook.   @see                     
                                               https:www.drupal.orgprojectecai  
                                               ssues3460491                     
 -------- --------------------- ------------- ---------------------------------

The same can be found in modules like encrypt, restui, simple_sitemap and certainly others.

This has been tested in Drush 12.5 and 13.0 with the same result.

Not sure where exactly the slashes get removed. I guess this could be Drush itself or maybe Laravel?

weitzman commented 1 month ago

I don't know either. The sub-command that is showing that table is drush updatedb:status