If you clone a node called "Bacon & Eggs" with the Node Clone module, the title field will be "Clone of Bacon & Eggs"

Quite a nuisance when you're copying a bunch of nodes with ampersands.

CommentFileSizeAuthor
#4 title-fix-5-1-188292-4.patch840 bytespwolanin
#3 title-fix-5-2-188292-3.patch750 bytespwolanin

Comments

bradweikel’s picture

bradweikel commented

Sneaky filters...

To clarify, it will give me a title field of "Clone of Bacon & Eggs"

pwolanin’s picture

pwolanin commented

hmmm, that's odd - ah I see why - the way this string is generated invokes check_plain():

 $node->title = t('Clone of @title', array('@title' => $node->title));

For 5.x it should be:

 $node->title = t('Clone of !title', array('!title' => $node->title));

I'll try to fix it this week.

pwolanin’s picture

pwolanin commented
StatusFileSize
new title-fix-5-2-188292-3.patch750 bytes

ok, here's a patch for HEAD and the 5.x-2 branch

pwolanin’s picture

pwolanin commented
Status: Active » Patch (to be ported)
StatusFileSize
new title-fix-5-1-188292-4.patch840 bytes

here's a patch for the 5.x-1 branch

these are trivial so I am committing them now.

pwolanin’s picture

pwolanin commented
Status: Patch (to be ported) » Fixed
Anonymous’s picture

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

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

  • Commit f9f3304 on 6.x-1.x, 7.x-1.x, master, 8.x-1.x by pwolanin: 
    #188292 use ! placeholder to prevent title from getting subject to...