...as per title. Module's name should be 'Entity translation ', not 'Entity Translation'.

CommentFileSizeAuthor
#2 et-1096028-2.patch844 bytesplach

Comments

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented

Works for me, let's hear a word from sun. Can we have a patch meanwhile?

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Status: Active » Needs review
Issue tags: -Coding standards, -do not capitalize name +Quick fix
StatusFileSize
new et-1096028-2.patch844 bytes
sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community
klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented
Issue tags: +Coding standards, +do not capitalize name

...mind if we keep the extra tags?

Also, remember to update the project's page in d.o ;)

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD and changed project title.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

...thanx for the prompt actions guys ;)

Status: Fixed » Closed (fixed)

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

michelle’s picture

michelle
she/her
Primary language English
Location Wisconsin, USA
commented

See #1346158: Decide on coding standard for capitalization of module names [policy, no patch] and #1430452: Use Proper Name Case for core modules for current information on this policy.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented
Title: Do not capitalize first letters in project's name » Capitalize first letters in project's name

...yep, the standard has changed and it now allows (more likely suggests) Proper Name Case instead of the Sentence case for module/theme names. The Sentence case policy remains for UI text like widget labels for example.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented
Status: Closed (fixed) » Active

;)

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Component: Miscellaneous » Base system
Status: Active » Fixed

Reverted the patch in #2.

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix, -Coding standards, -do not capitalize name

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

Issue tags: +Quick fix, +Coding standards, +do not capitalize name

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.

  • Commit 581f10d on master, et-permissions-1829630, factory, et-fc, revisions by plach: 
    Issue #1095390 by plach, klonos, sun: Changed Do not capitalize first...
  • Commit 242d16e on master, et-permissions-1829630, factory, et-fc, revisions by plach:
    Issue #1095390 by plach | klonos: Capitalize first letters in project's...

  • Commit 581f10d on master, et-permissions-1829630, factory, et-fc, revisions, workbench by plach: 
    Issue #1095390 by plach, klonos, sun: Changed Do not capitalize first...
  • Commit 242d16e on master, et-permissions-1829630, factory, et-fc, revisions, workbench by plach:
    Issue #1095390 by plach | klonos: Capitalize first letters in project's...