As the title says:

Cloning a Rules Link doesn't seem to clone the components also, thus rendering the clone option pretty useless...

CommentFileSizeAuthor
#3 rules_link-fix-clone-action-1868016-3.patch2.09 KBmoiseh

Comments

cornelyus’s picture

cornelyus commented

subscribing..

Anonymous’s picture

Anonymous (not verified) commented
Version: 7.x-1.0-beta4 » 7.x-1.x-dev
Category: bug » task
Priority: Normal » Major
moiseh’s picture

moiseh commented
Issue summary: View changes
StatusFileSize
new rules_link-fix-clone-action-1868016-3.patch2.09 KB

Here's my patch...

sepgil’s picture

sepgil commented

Your patch doesn't work quite correct, since it doesn't changes the path. Rules link will show an error, when trying to save the cloned rules link.
There are also some minor code style issues:

  1. +++ b/rules_link.admin.inc
@@ -399,6 +408,23 @@ function rules_link_form_submit(&$form, &$form_state) {
+    ¶

    Trailing space 

  2. +++ b/rules_link.admin.inc
@@ -399,6 +408,23 @@ function rules_link_form_submit(&$form, &$form_state) {
+    $condition_set = rules_link_load_condition_set($rules_link_load);  ¶

    Trailing space 

  3. +++ b/rules_link.admin.inc
@@ -399,6 +408,23 @@ function rules_link_form_submit(&$form, &$form_state) {
+    drupal_set_message(t('Your Rules Link has been cloned.'));  ¶

    Trailing space

sepgil’s picture

sepgil commented
Status: Active » Needs work