Download & Extend
Issues

INSTALL.txt: Oxymoron "Optional requirements"

Posted by webchick on June 16, 2008 at 8:54pm
Project:Drupal core
Version:7.x-dev
Component:documentation
Category:bug report
Priority:minor
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Are they optional, or are they requirements? ;)

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

Login or register to post comments

Comments

#1

Posted by bdragon on June 16, 2008 at 8:54pm

Red Dwarf.

#2

Posted by keith.smith on June 22, 2008 at 2:59am
Assigned to:Anonymous» keith.smith

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

#3

Posted by keith.smith on June 23, 2008 at 6:14pm
Assigned to:keith.smith» Anonymous
Status:active» needs review

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

AttachmentSizeStatusTest resultOperations
INSTALL_tweak.patch748 bytesIgnored: Check issue status.NoneNone

#4

Posted by cburschka on June 23, 2008 at 6:33pm
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

Posted by webchick on June 23, 2008 at 8:34pm

LOL. :)

Thanks, Keith!

#6

Posted by Dries on June 24, 2008 at 5:05pm
Status:reviewed & tested by the community» fixed

Committed to CVS HEAD and to DRUPAL-6.

#7

Posted by Anonymous (not verified) on July 8, 2008 at 5:12pm
Status:fixed» closed (fixed)

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

#8

Posted by xmacinfo on October 4, 2008 at 5:54pm
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.

#9

Posted by webchick on October 4, 2008 at 6:01pm

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.

AttachmentSizeStatusTest resultOperations
drupal-install-txt-consistencize-271326-9.patch1.09 KBIgnored: Check issue status.NoneNone

#10

Posted by webchick on October 4, 2008 at 6:02pm
Status:active» needs review

#11

Posted by Ryan Palmer on October 4, 2008 at 6:04pm

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

Updated patch attached.

AttachmentSizeStatusTest resultOperations
drupal-install-txt-fix-consistencize-271326-10.patch657 bytesIgnored: Check issue status.NoneNone

#12

Posted by xmacinfo on October 4, 2008 at 6:08pm
Status:needs review» reviewed & tested by the community

That was quick. It's working perfectly.

#13

Posted by webchick on October 4, 2008 at 6:35pm
Status:reviewed & tested by the community» fixed

Great! Committed! Thanks!

#14

Posted by Anonymous (not verified) on October 18, 2008 at 6:42pm
Status:fixed» closed (fixed)

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