While working on the pt-pt translation, I detected two small typos in node.install.

On line 35, the string
'description' => t('The title of this node, always treated a non-markup plain text.'),

should be
'description' => t('The title of this node, always treated as non-markup plain text.'),

and on line 66, the string
'description' => t('Boolean indicating whether the node should displayed on the front page.'),

should be
'description' => t('Boolean indicating whether the node should be displayed on the front page.'),

I attached a very simple diff with the necessary changes.

Raul

CommentFileSizeAuthor
#4 node.install-typos-2.patch915 bytesjpoesen
#1 node.install-typos.patch993 bytesGarrett Albright
node.install.diff428 bytesborfast

Comments

Garrett Albright’s picture

Garrett Albright commented
Version: 6.1 » 7.x-dev
StatusFileSize
new node.install-typos.patch993 bytes

I suspect it's too late to fix them in 6.x, but they're both still in place in d7's node.install. Patch attached.

david strauss’s picture

david strauss
he/him
commented

The procedure is to fix it in Drupal 7 and backport it to previous versions with the same typo.

jpoesen’s picture

jpoesen commented
Status: Needs review » Needs work

Unable to apply the patch. It should have been created from the Drupal root.

can't find file to patch at input line 3
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|--- /Users/Albright/Desktop/drupal-7.x-dev/modules/node/node.install copy      2008-03-15 05:31:28.000000000 -0700
|+++ /Users/Albright/Desktop/drupal-7.x-dev/modules/node/node.install   2008-04-06 11:58:08.000000000 -0700
--------------------------
jpoesen’s picture

jpoesen commented
Status: Needs work » Needs review
StatusFileSize
new node.install-typos-2.patch915 bytes

Re-rolled the patch from Drupal root. Applied it and works as advertised.

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs review » Reviewed & tested by the community

Looks good, RTBC.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD and DRUPAL-6. Thanks! :)

Anonymous’s picture

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

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