Fix modules batch processing

joomla · Jan 2, 2012 · d97df21 · d97df21
1 parent f29aa82
commit d97df21
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 9 deletions.
diff --git a/administrator/components/com_modules/helpers/html/modules.php b/administrator/components/com_modules/helpers/html/modules.php
@@ -129,7 +129,7 @@ public static function positions($clientId)
 			'</label>',
 			'<fieldset id="batch-choose-action" class="combo">',
 			'<select name="batch[position_id]" class="inputbox" id="batch-position-id">',
-			'<option value="nochange">'.JText::_('COM_MODULES_BATCH_POSITION_NOCHANGE').'</option>',
+			'<option value="">'.JText::_('COM_MODULES_BATCH_POSITION_NOCHANGE').'</option>',
 			'<option value="noposition">'.JText::_('COM_MODULES_BATCH_POSITION_NOPOSITION').'</option>',
 			JHtml::_('select.options',	self::positionList($clientId)),
 			'</select>',

diff --git a/administrator/components/com_modules/models/module.php b/administrator/components/com_modules/models/module.php
@@ -186,10 +186,6 @@ protected function batchCopy($value, $pks, $contexts)
 				{
 					$position = '';
 				}
-				elseif ($value == 'nochange')
-				{
-					$position = $table->position;
-				}
 				else
 				{
 					$position = $value;
@@ -282,10 +278,6 @@ protected function batchMove($value, $pks, $contexts)
 				{
 					$position = '';
 				}
-				elseif ($value == 'nochange')
-				{
-					$position = $table->position;
-				}
 				else
 				{
 					$position = $value;