Author: sevein
Date: Mon Nov 8 11:49:18 2010
New Revision: 8802
Log:
Use hide() instead of remove(), duplicating events will work correctly
regardless or whether js is enabled.
Modified:
trunk/js/multiDelete.js
trunk/plugins/sfIsadPlugin/modules/sfIsadPlugin/actions/eventComponent.class.php
Modified: trunk/js/multiDelete.js
==============================================================================
--- trunk/js/multiDelete.js Mon Nov 8 10:48:49 2010 (r8801)
+++ trunk/js/multiDelete.js Mon Nov 8 11:49:18 2010 (r8802)
@@ -17,7 +17,7 @@
{
event.stopPropagation();
- $input.attr('checked',
'checked').appendTo($input.closest('table'));
+ $input.attr('checked', 'checked');
// Hide element
var $parentRows = $(this).closest('tr');
@@ -34,7 +34,7 @@
$('div:visible', $parentRows).hide('normal', function ()
{
- $parentRows.remove();
+ $parentRows.hide();
});
});
})
Modified:
trunk/plugins/sfIsadPlugin/modules/sfIsadPlugin/actions/eventComponent.class.php
==============================================================================
---
trunk/plugins/sfIsadPlugin/modules/sfIsadPlugin/actions/eventComponent.class.php
Mon Nov 8 10:48:49 2010 (r8801)
+++
trunk/plugins/sfIsadPlugin/modules/sfIsadPlugin/actions/eventComponent.class.php
Mon Nov 8 11:49:18 2010 (r8802)
@@ -75,6 +75,13 @@
continue;
}
+ // Ignore item if it was removed and it was being duplicated
+ if (isset($item['id'])
+ && false !== array_search($item['id'],
(array)$this->request->deleteEvents))
+ {
+ continue;
+ }
+
$this->form->bind($item);
if ($this->form->isValid())
{
--
You received this message because you are subscribed to the Google Groups
"Qubit Toolkit Commits" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/qubit-commits?hl=en.
