From 19f1eec6d69fa428d17b9d10a24084e4bb90ee34 Mon Sep 17 00:00:00 2001
From: vChabot <vincent.chabot@prestashop.com>
Date: Mon, 31 Oct 2011 11:01:01 +0000
Subject: [PATCH] // correct drag and drop for position carriers

---
 admin-dev/themes/template/helper/list/list_content.tpl | 3 ++-
 admin-dev/themes/template/helper/list/list_header.tpl  | 2 +-
 classes/HelperList.php                                 | 6 +++---
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/admin-dev/themes/template/helper/list/list_content.tpl b/admin-dev/themes/template/helper/list/list_content.tpl
index 0b2a33594..ba03efd70 100644
--- a/admin-dev/themes/template/helper/list/list_content.tpl
+++ b/admin-dev/themes/template/helper/list/list_content.tpl
@@ -23,7 +23,7 @@
 *  @license    http://opensource.org/licenses/afl-3.0.php  Academic Free License (AFL 3.0)
 *  International Registered Trademark & Property of PrestaShop SA
 *}
-
+<tbody>
 {foreach $list AS $index => $tr}
 	<tr
 	{if $is_dnd_identifier}id="tr_{$id_category}_{$tr.$identifier}_{$tr.position}"{/if}
@@ -111,3 +111,4 @@
 	{/if}
 	</tr>
 {/foreach}
+</tbody>
\ No newline at end of file
diff --git a/admin-dev/themes/template/helper/list/list_header.tpl b/admin-dev/themes/template/helper/list/list_header.tpl
index bb9626532..601cf22aa 100644
--- a/admin-dev/themes/template/helper/list/list_header.tpl
+++ b/admin-dev/themes/template/helper/list/list_header.tpl
@@ -110,7 +110,7 @@
 			<td{if $simple_header} style="border:none;"{/if}>
 				<table
 				{if $table_id} id={$table_id}{/if}
-				class="table {if $table_dnd}tableDnd{/if} {$table}"
+				class="table {if $table_dnd}tableDnD{/if} {$table}"
 				cellpadding="0" cellspacing="0"
 				style="width: 100%; margin-bottom:10px;"
 				>
diff --git a/classes/HelperList.php b/classes/HelperList.php
index 4adf72c8b..5b3fa847e 100644
--- a/classes/HelperList.php
+++ b/classes/HelperList.php
@@ -445,8 +445,8 @@ class HelperListCore extends Helper
 			isset($this->context->cookie->{$this->table.'_pagination'}) ? $this->context->cookie->{$this->table.'_pagination'} : null
 		);
 
-		/*$is_dnd_identifier = array_key_exists($this->identifier,$this->identifiersDnd);
-
+		$this->is_dnd_identifier = array_key_exists($this->identifier,$this->identifiersDnd);
+/*
 		if ($is_dnd_identifier)
 		{
 			' id="'.
@@ -465,7 +465,7 @@ class HelperListCore extends Helper
 		if (array_key_exists($this->identifier, $this->identifiersDnd) && (int)Tools::getValue($this->identifiersDnd[$this->identifier], 1))
 			$table_id = substr($this->identifier, 3, strlen($this->identifier));
 
-		if (array_key_exists($this->identifier, $this->identifiersDnd) && ($this->orderBy != 'position' && $this->orderWay != 'DESC'))
+		if (array_key_exists($this->identifier, $this->identifiersDnd) && ($this->orderBy == 'position' && $this->orderWay != 'DESC'))
 			$table_dnd = true;
 
 		foreach ($this->fieldsDisplay as $key => $params)