Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
When running an update via drush, the log messages aren't that useful.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
> [notice] Checked translation for devel.
Comment | File | Size | Author |
---|---|---|---|
#7 | locale-better_messaging-3119278-7.patch | 2.63 KB | nterbogt |
Comments
Comment #2
nterbogt CreditAttribution: nterbogt as a volunteer commentedComment #3
nterbogt CreditAttribution: nterbogt as a volunteer commentedComment #4
nterbogt CreditAttribution: nterbogt as a volunteer commentedComment #5
nterbogt CreditAttribution: nterbogt as a volunteer commentedComment #6
hansfn CreditAttribution: hansfn commentedYes, this is very nice improvement - in particular for Drush users. Patch tested with 8.9.0-beta2 (and Drush 10.2.2).
I would prefer the messages to be "Checked %langcode translation for %project." instead of "Checked translation %langcode for %project." - and similar for the other message. If your try to read the same messages with language name instead of code, it's clearly better.
Comment #7
nterbogt CreditAttribution: nterbogt as a volunteer commentedI agree that "Checked %langcode translation ..." has better readability than "Checked translation %langcode ...".
Here is an updated patch.
Comment #9
nterbogt CreditAttribution: nterbogt as a volunteer commentedTests failed due to server error.
Comment #10
quietone CreditAttribution: quietone as a volunteer commentedRetested patch
Comment #13
catchCommitted/pushed to 9.1.x/9.0.x/8.9.x, thanks!