Having the clone link always read as "Clone content" seems a little unfriendly to me. Here's a patch that offers an admin setting to include the node type name in the clone link, so if set, it will read something like "Clone article" rather than "Clone content".

CommentFileSizeAuthor
#6 1137178-5.patch2.13 KBpwolanin
#4 node_clone-node-type-on-clone-link-1137178.patch2.18 KBmstef
node_clone-node-type-on-clone-link.patch2.16 KBmstef
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented

nice idea. to lower-case probably should use:

http://api.drupal.org/api/drupal/includes--unicode.inc/function/drupal_s...

mstef’s picture

mstef CreditAttribution: mstef commented

No objections to that. Actually didn't know that wrapper existed.

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented

can you re-roll with that?

mstef’s picture

mstef CreditAttribution: mstef commented
FileSize
node_clone-node-type-on-clone-link-1137178.patch2.18 KB

This should do it

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Status: Needs review » Fixed
FileSize
1137178-5.patch2.13 KB

Sorry to have left this for so long.

re-rolled and committed.

Status: Fixed » Closed (fixed)

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

  • Commit ef9377b on 7.x-1.x, 8.x-1.x by pwolanin: 
    Issue #1137178 by mikestefff: Optionally add the node type name to the...
  • Commit 528318b on 7.x-1.x, 8.x-1.x by pwolanin:
    Issue #1137178 typo follow-up