It would be wonderful if your module can trigger a 'clone' event to the rules module and the drupal own actions instrumenting. For the Rules integration, it would be enough to include the following:

1) create clone.rules.inc with the following content:

// implementation of 'hook_rules_event_info'
// required to let Rules know about a new 'clone node' event
function clone_rules_event_info() {
$items = array(
'node_clone' => array(
'label' => t('After cloning a node'),
'group' => t('Node'),
'variables' => array(
'cloned_node' => array('type' => 'node', 'label' => t('The cloned node')),
'original_node' => array('type' => 'node', 'label' => t('The original node')),
),
),
);
return $items;
}

2) add the following after your node_save($node):

if(module_exists('rules')) {
rules_invoke_event('node_clone', $node, $original_node);
}

It works like a charm for me!

CommentFileSizeAuthor
#6 node_clone-add_Rules_integration_for_node_clone-1720330-6.patch1.09 KBtripper54
#5 node_clone-add_rules_to_node_clone-1720330-5.patch1.05 KBjordan8037310
#3 node_clone-add_Rules_integration_for_node_clone-1720330-3.patch0 bytesjordan8037310
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Version: 7.x-1.0-beta1 » 7.x-1.x-dev

Can you create a patch for that?

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented

Also, the event should probably be "clone_node", not the other way around.

jordan8037310’s picture

jordan8037310 CreditAttribution: jordan8037310 commented
Status: Needs review » Active
FileSize
node_clone-add_Rules_integration_for_node_clone-1720330-3.patch0 bytes

This is my first time trying to create a patch. Hope it works! Any constructive criticism on formatting of the patch would be appreciated.

Thanks for the great module!

Edit: Might help if the attached file actually contained something... Going to go through my steps again and upload it.

jordan8037310’s picture

jordan8037310 CreditAttribution: jordan8037310 commented
Assigned: Unassigned » jordan8037310
Status: Active » Needs review

Updating status

jordan8037310’s picture

jordan8037310 CreditAttribution: jordan8037310 commented
Status: Active » Needs review
FileSize
node_clone-add_rules_to_node_clone-1720330-5.patch1.05 KB

okay let's try this again

Edit: Looks like that worked.

Now I'd like to try to configure a Rules action that will execute the Node_Clone operation programmatically. Any ideas or direction on this?

I'll post progress as I make some more.

tripper54’s picture

tripper54 CreditAttribution: tripper54 commented
FileSize
node_clone-add_Rules_integration_for_node_clone-1720330-6.patch1.09 KB

Patch applied well and works great! I re-rolled with some small changes in the comments format to conform to coding standards, also removed some whitespace.

arkjoseph’s picture

arkjoseph CreditAttribution: arkjoseph commented

any chance this can work in D6?

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented

Looks like a pretty simple change

@arkjoseph - I don't know what the rules APi looks like in 6, but it should be something similar if you want to create the patch.

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Needs review » Patch (to be ported)

committed to 7.x

skyredwang’s picture

skyredwang
Location Shanghai
CreditAttribution: skyredwang commented
Title: add Rules integration for node clone » add Rules integration (event) for node clone

update the title to be more specific

pwolanin’s picture

pwolanin CreditAttribution: pwolanin commented
Issue summary: View changes
Status: Patch (to be ported) » Closed (won't fix)

closing is the absence of a 6.x patch

  • Commit f41f777 on 7.x-1.x, 8.x-1.x by pwolanin: 
    Issue #1720330 by jordan8037310, tripper54: add Rules integration for...