From c109be6c5af1459db40e8680696acf3c1b2b09b1 Mon Sep 17 00:00:00 2001
From: Coleman Watts <coleman@civicrm.org>
Date: Sat, 21 Dec 2013 22:46:29 -0800
Subject: [PATCH] CRM-13863 - Only load real links not js buttons

---
 CRM/Admin/Page/Tag.php           |  2 +-
 js/Common.js                     |  2 +-
 js/crm.livePage.js               |  8 ++++++--
 templates/CRM/Admin/Page/Tag.tpl | 15 ++++++---------
 4 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/CRM/Admin/Page/Tag.php b/CRM/Admin/Page/Tag.php
index fa1cccadfa..46ae2f3095 100644
--- a/CRM/Admin/Page/Tag.php
+++ b/CRM/Admin/Page/Tag.php
@@ -78,7 +78,6 @@ class CRM_Admin_Page_Tag extends CRM_Core_Page_Basic {
         CRM_Core_Action::FOLLOWUP => array(
           'name' => ts('Merge'),
           'class' => 'merge_tag',
-          'url' => 'javascript:',
           'title' => ts('Merge Tag'),
         ),
       );
@@ -135,6 +134,7 @@ class CRM_Admin_Page_Tag extends CRM_Core_Page_Basic {
    * override function browse()
    */
   function browse($action = NULL, $sort = NULL) {
+    CRM_Core_Resources::singleton()->addScriptFile('civicrm', 'js/crm.livePage.js');
     $adminTagSet = FALSE;
     if (CRM_Core_Permission::check('administer Tagsets')) {
       $adminTagSet = TRUE;
diff --git a/js/Common.js b/js/Common.js
index 1735f1b63d..0976395d80 100644
--- a/js/Common.js
+++ b/js/Common.js
@@ -937,7 +937,7 @@ CRM.validate = CRM.validate || {
         validate: true,
         refreshAction: ['next_new', 'submit_savenext'],
         cancelButton: '.cancel.form-submit',
-        openInline: 'a.button',
+        openInline: 'a.button:not([href="#"])',
         onCancel: function(event) {},
         onError: function(data) {
           var $el = $(this);
diff --git a/js/crm.livePage.js b/js/crm.livePage.js
index 722ebabc15..4c0e8c9791 100644
--- a/js/crm.livePage.js
+++ b/js/crm.livePage.js
@@ -5,9 +5,13 @@ cj(function($) {
     // Widgetize the content area
     .crmSnippet()
     // Open action links in a popup
-    .on('click', 'a.button, a.action-item:not(".crm-enable-disable")', function() {
+    .on('click', 'a.button, a.action-item', function() {
+      // only follow real links not javascript buttons
+      if ($(this).attr('href') === '#' || $(this).attr('onclick')) {console.log('bogus', this);
+        return;
+      }
       CRM.loadForm($(this).attr('href'), {
-        openInline: 'a:not(".crm-enable-disable")'
+        openInline: 'a:not([href="#"])'
       }).on('crmFormSuccess', function(e, data) {
         // Refresh page when form completes
         $('#crm-main-content-wrapper').crmSnippet('refresh');
diff --git a/templates/CRM/Admin/Page/Tag.tpl b/templates/CRM/Admin/Page/Tag.tpl
index d721f127cb..d74f916229 100644
--- a/templates/CRM/Admin/Page/Tag.tpl
+++ b/templates/CRM/Admin/Page/Tag.tpl
@@ -96,7 +96,7 @@
         </div>
     {/if}
 
-<div id="mergeTagDialog">
+<div id="mergeTagDialog" style="display:none;">
     {ts}Begin typing name of tag to merge into.{/ts}<br/>
     <input type="text" id="tag_name"/>
     <input type="hidden" id="tag_name_id" value="">
@@ -108,11 +108,11 @@
 {literal}
 <script type="text/javascript">
 cj("#mergeTagDialog").hide( );
-cj( function() {
-    cj('.merge_tag').click(function(){
-        var row_id = cj(this).closest('tr').attr('id');
-        var tagId = row_id.split('-');
+cj( function($) {
+    cj('.merge_tag').click(function(e){
+        var tagId = cj(this).closest('tr').attr('id').split('-');
         mergeTag(tagId[1]);
+        return false;
     });
 });
 
@@ -120,7 +120,6 @@ function mergeTag( fromId ) {
     var fromTag = cj('#tag-' + fromId).children('td.crm-tag-name').text();
     cj('#used_for_warning').html('');
 
-    cj("#mergeTagDialog").show( );
   cj("#mergeTagDialog").dialog({
     title: "Merge tag '" + fromTag + "' into:",
     modal: true,
@@ -182,13 +181,11 @@ function mergeTag( fromId ) {
                       }
                 });
 
-                cj(this).dialog("close");
-        cj(this).dialog("destroy");
+        cj(this).dialog("close");
        },
 
       "Cancel": function() {
         cj(this).dialog("close");
-        cj(this).dialog("destroy");
       }
           }
   });
-- 
2.25.1