From a541050abfb1e2b43968921eec6b99fe921a5cd3 Mon Sep 17 00:00:00 2001
From: jcleeland <jason@cleeland.org>
Date: Thu, 16 Aug 2012 11:31:02 +1000
Subject: [PATCH] Fixed issue #6491 - Modal dialogs in attribute control do not
 close after use or reload grid Dev - some new translations required

---
 .../participants/attributeControl_view.php    |  4 ++
 scripts/admin/attributeControl.js             | 45 ++++++++++++-------
 2 files changed, 34 insertions(+), 15 deletions(-)

diff --git a/application/views/admin/participants/attributeControl_view.php b/application/views/admin/participants/attributeControl_view.php
index 7b49132362c..05aa5a7dd08 100644
--- a/application/views/admin/participants/attributeControl_view.php
+++ b/application/views/admin/participants/attributeControl_view.php
@@ -1,4 +1,8 @@
 <script type="text/javascript">
+    var deleteCaption = "<?php $clang->eT("Delete attribute") ?>";
+    var deleteMsg = "<?php $clang->eT("Delete selected attribute(s) and it's associated data?") ?>";
+    var addCaption = "<?php $clang->eT("Add attribute") ?>";
+
     var attributeInfoUrl = "<?php echo Yii::app()->getController()->createUrl("admin/participants/getAttributeInfo_json"); ?>";
     var editAttributeUrl = "<?php echo Yii::app()->getController()->createUrl("admin/participants/editAttributeInfo"); ?>";
     var attributeControlCols = '["<?php $clang->eT('Actions'); ?>", "<?php $clang->eT('Attribute name'); ?>", "<?php $clang->eT('Attribute type'); ?>", "<?php $clang->eT('Visible in participants panel'); ?>"]';
diff --git a/scripts/admin/attributeControl.js b/scripts/admin/attributeControl.js
index 7877a8551e6..7378806f3c2 100644
--- a/scripts/admin/attributeControl.js
+++ b/scripts/admin/attributeControl.js
@@ -30,32 +30,47 @@ $(document).ready(function() {
     });
 
     jQuery.extend($.fn.fmatter , {
-        rowactions : function(rid,gid,act) {
+        rowactions : function(rid,gid,act, pos) {
+            var delOptions = {
+                caption: deleteCaption,
+                msg: deleteMsg,
+                reloadAfterSubmit: true,
+                width: 400
+            };
             switch(act)
             {
                 case 'edit' :
                     window.open(attributeEditUrl + '/' + rid, '_top');
                     break;
                 case 'del':
-                    $('#'+gid).jqGrid('delGridRow', rid);
+                    $('#'+gid).jqGrid('delGridRow', rid, delOptions);
                     break;
             }
         }
     });
 
-    jQuery('#attributeControl').jqGrid('navGrid',
-                                       '#pager',
-                                       {add:true, del:true, edit:true},
-                                       {closeAfterAdd: true
-                                       }, //Add options
-                                       {    width:400,
-                                            reloadAfterSubmit: true,
-                                            afterSubmit: function (response) {
-                                                return [true, '', response.responseText];
-                                            }
-                                       }, //Del options
-                                       {}, //Edit options
-                                       {multipleSearch:true, width:600},
+    jQuery('#attributeControl').jqGrid('navGrid', '#pager',
+                                       { add:true,
+                                         edit:false,
+                                         del:true},
+                                       {}, //Default settings for edit
+                                       { addCaption: addCaption,
+                                         closeAfterAdd: true,
+                                         width: 400,
+                                         afterSubmit: function () {
+                                             $(this).jqGrid('setGridParam', {datatype: 'json'});
+                                             return [true,'',false]; //no error and no new rowid
+                                         },
+                                           }, //default settings for add
+                                       {    reloadAfterSubmit: true,
+                                            caption: deleteCaption,
+                                            msg: deleteMsg,
+                                            width: 500
+                                           }, //Default settings for delete
+                                       { multipleSearch:true,
+                                         width:600,
+                                         closeAfterSearch: true,
+                                         closeAfterReset: true}, //Default settings for search
                                        {closeAfterAdd:true}
                                       );