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.
While evaluating this module I noticed that on the Extend page, the project name is in lower case and should be in title case to be consistent with the rest of the project names on the page.
"Capitalize the name of the module, because module names are proper nouns."
https://www.drupal.org/docs/develop/documenting-your-project/help-text-s...
Patch to follow, thanks!
Comment | File | Size | Author |
---|---|---|---|
#3 | Term_Merge_2969607_after.png | 236.97 KB | moghingold |
#3 | Term_Merge_2969607_before.png | 236.45 KB | moghingold |
#2 | term_merge-capitalize-name-issue-2.patch | 298 bytes | volkswagenchick |
Comments
Comment #2
volkswagenchickI uploaded a patch that capitalizes the name of the module in the .info file to align with Drupal standards, thanks.
Comment #3
moghingold CreditAttribution: moghingold at Hook 42 commentedI have tested this patch and uploaded 2 files. Term_Merge_2969607_before.png shows this module loaded unpatched on simplytest.me and shows that the issue is not resolved and the module title is not in title case. Term_Merge_2969607_after.png shows this module loaded with the patch on simplytest.me and shows that the module title is in title case and the patch fixes the issue. Marking RTBC.
Comment #4
legolasboCommitted and pushed to 8.x-1.x thanks.
Comment #6
legolasbo