Problem/Motivation

Looking through the list of module names: admin/modules#edit-modules-example-modules

...we see that the capitalization of module names is inconsistent. For module names they should all be fully capitalized.

Proposed resolution

For each module we need the following:

Ensure that the module's .info.yml file name: field is fully capitalized, such as "Content Entity Example".

Ensure that the @defgroup tag in the module's .module file is fully capitalized, such as @defgroup content_entity_example Example: Content Entity

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#18 interdiff.txt944 bytesMaskyS
#18 Capitalise_all_module_names-2897096-18.patch9.01 KBMaskyS
#13 interdiff.txt1.77 KBMaskyS
#13 Capitalise_all_module_names-2897096-13.patch9.01 KBMaskyS
#10 Capitalise-all-module-names_2897096_10.patch7.08 KBMaskyS
#7 interdiff.txt1.07 KBMile23
#7 2897096_7.patch6.73 KBMile23
#4 2897096.patch5.66 KBLal_
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mile23 created an issue. See original summary.

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 commented
Parent issue: » #2831282: [meta] Inconsistent capitalization in module name / menu links
Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 commented
Issue summary: View changes
Lal_’s picture

Lal_
He\Him
CreditAttribution: Lal_ as a volunteer and at Google Summer of Code commented
FileSize
2897096.patch5.66 KB

I hope this is what you are looking for.

Lal_’s picture

Lal_
He\Him
CreditAttribution: Lal_ as a volunteer and at Google Summer of Code commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 4: 2897096.patch, failed testing. View results

Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 commented
Status: Needs work » Needs review
FileSize
2897096_7.patch6.73 KB
interdiff.txt1.07 KB

Looks like one test needs updating...

Status: Needs review » Needs work

The last submitted patch, 7: 2897096_7.patch, failed testing. View results

MaskyS’s picture

MaskyS CreditAttribution: MaskyS at Google Code-In commented
Assigned: Unassigned » MaskyS
MaskyS’s picture

MaskyS CreditAttribution: MaskyS at Google Code-In commented
FileSize
Capitalise-all-module-names_2897096_10.patch7.08 KB

Fixed all the issues mentioned in issues description. I noticed that there is also an inconsistency concerning the use of single quotes for module names in info.yml files. Only one or two files have the module names enclosed in quotes. @maintainers Should we fix that in a separate issue?

MaskyS’s picture

MaskyS CreditAttribution: MaskyS at Google Code-In commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 10: Capitalise-all-module-names_2897096_10.patch, failed testing. View results

MaskyS’s picture

MaskyS CreditAttribution: MaskyS at Google Code-In commented
FileSize
Capitalise_all_module_names-2897096-13.patch9.01 KB
interdiff.txt1.77 KB

Made a few changes to tests that needed updating. Should pass now.

MaskyS’s picture

MaskyS CreditAttribution: MaskyS at Google Code-In commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 13: Capitalise_all_module_names-2897096-13.patch, failed testing. View results

navneet0693’s picture

navneet0693 CreditAttribution: navneet0693 as a volunteer and at QED42 commented 
+++ b/src/Tests/ExamplesTest.php
@@ -48,7 +48,8 @@ class ExamplesTest extends WebTestBase {
+    $phpunit_link = 'PHPUnit example';

Replace with 'PHPUnit Example'

MaskyS’s picture

MaskyS CreditAttribution: MaskyS at Google Code-In commented

Gotcha.

MaskyS’s picture

MaskyS CreditAttribution: MaskyS at Google Code-In commented
FileSize
Capitalise_all_module_names-2897096-18.patch9.01 KB
interdiff.txt944 bytes

Alright, let's test this again.

MaskyS’s picture

MaskyS CreditAttribution: MaskyS at Google Code-In commented
Status: Needs work » Needs review
navneet0693’s picture

navneet0693 CreditAttribution: navneet0693 as a volunteer and at QED42 commented
Status: Needs review » Reviewed & tested by the community

That should be all.

  • Mile23 committed 1adab71 on 8.x-1.x authored by AbhishekLal 
    Issue #2897096 by Kifah Meeran, Mile23, AbhishekLal, navneet0693:...
Mile23’s picture

Mile23
Primary language English
Location Seattle, WA
CreditAttribution: Mile23 commented
Status: Reviewed & tested by the community » Fixed

Done. Thanks everyone!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.