INSTALL.txt: Oxymoron "Optional requirements"

webchick - June 16, 2008 - 20:54
Project:Drupal
Version:7.x-dev
Component:documentation
Category:bug report
Priority:minor
Assigned:Unassigned
Status:closed
Description

Are they optional, or are they requirements? ;)

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

#1

bdragon - June 16, 2008 - 20:54

Red Dwarf.

#2

keith.smith - June 22, 2008 - 02:59
Assigned to:Anonymous» keith.smith

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

#3

keith.smith - June 23, 2008 - 18:14
Assigned to:keith.smith» Anonymous
Status:active» needs review

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

AttachmentSize
INSTALL_tweak.patch 748 bytes

#4

Arancaytar - June 23, 2008 - 18:33
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

#5

webchick - June 23, 2008 - 20:34

LOL. :)

Thanks, Keith!

#6

Dries - June 24, 2008 - 17:05
Status:reviewed & tested by the community» fixed

Committed to CVS HEAD and to DRUPAL-6.

#7

Anonymous (not verified) - July 8, 2008 - 17:12
Status:fixed» closed

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

#8

xmacinfo - October 4, 2008 - 17:54
Status:closed» 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.

#9

webchick - October 4, 2008 - 18:01

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.

AttachmentSize
drupal-install-txt-consistencize-271326-9.patch 1.09 KB

#10

webchick - October 4, 2008 - 18:02
Status:active» needs review

#11

Ryan Palmer - October 4, 2008 - 18:04

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

Updated patch attached.

AttachmentSize
drupal-install-txt-fix-consistencize-271326-10.patch 657 bytes

#12

xmacinfo - October 4, 2008 - 18:08
Status:needs review» reviewed & tested by the community

That was quick. It's working perfectly.

#13

webchick - October 4, 2008 - 18:35
Status:reviewed & tested by the community» fixed

Great! Committed! Thanks!

#14

Anonymous (not verified) - October 18, 2008 - 18:42
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.