Are they optional, or are they requirements? ;)

And yes, I *am* gunning once again for "most trivial issue of the week." ;)

CommentFileSizeAuthor
#11 drupal-install-txt-fix-consistencize-271326-10.patch657 bytesRyan Palmer
#9 drupal-install-txt-consistencize-271326-9.patch1.09 KBwebchick
#3 INSTALL_tweak.patch748 byteskeith.smith
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bdragon’s picture

bdragon CreditAttribution: bdragon commented

Red Dwarf.

keith.smith’s picture

keith.smith CreditAttribution: keith.smith commented
Assigned: Unassigned » keith.smith

Webchick, how do I love thee; let me count the issues. I'll roll a patch for this.

keith.smith’s picture

keith.smith CreditAttribution: keith.smith commented
Assigned: keith.smith » Unassigned
Status: Active » Needs review
FileSize
INSTALL_tweak.patch748 bytes

Changes "Optional Requirements" to "Optional Tasks". Other suggestions welcome, of course.

cburschka’s picture

cburschka
they
CreditAttribution: cburschka commented
Status: Needs review » Reviewed & tested by the community

Patch applies. I have done in-depth unit-testing and found that it does not significantly affect functionality, hence RTBC. =P

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

LOL. :)

Thanks, Keith!

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD and to DRUPAL-6.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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

xmacinfo’s picture

xmacinfo
he/him
Primary language French
Location Canada
CreditAttribution: xmacinfo commented
Status: Closed (fixed) » Active

Hi from DrupalCamp in Montréal.

The content section mention "Optional requirements" while in the content area the sub-title is "Optional tasks". This needs to be fixed.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
FileSize
drupal-install-txt-consistencize-271326-9.patch1.09 KB

While we were fixing this, we also noticed that the case of the other stuff was inconsistent, and so this patch addresses both of those problems.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Active » Needs review
Ryan Palmer’s picture

Ryan Palmer CreditAttribution: Ryan Palmer commented
FileSize
drupal-install-txt-fix-consistencize-271326-10.patch657 bytes

Um Angie... what's with "WHEEEEEEEE"? :)

Updated patch attached.

xmacinfo’s picture

xmacinfo
he/him
Primary language French
Location Canada
CreditAttribution: xmacinfo commented
Status: Needs review » Reviewed & tested by the community

That was quick. It's working perfectly.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Great! Committed! Thanks!

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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