Changeset 5856

Timestamp:

11/04/07 18:24:22 (1 year ago)

Author:

fabien

Message:

fixed through_class in admin_*_list tags (closes #1612 - patch from Pascal Borreli)

Files:

• branches/1.0/lib/helper/ObjectAdminHelper.php (modified) (2 diffs)
• branches/1.0/test/functional/backend/m2mTest.php (modified) (3 diffs)

branches/1.0/lib/helper/ObjectAdminHelper.php

@@ -73 +73 @@
-  }
-
+  // remove non html option
+  unset($options['through_class']);
-  // override field name
-  unset($options['control_name']);
@@ -127 +129 @@
-  // get the lists of objects
-  list($objects, $objects_associated, $ids) = _get_object_list($object, $method, $options, $callback);
+  // remove non html option
+  unset($options['through_class']);
-  // override field name
-  unset($options['control_name']);

branches/1.0/test/functional/backend/m2mTest.php

@@ -37 +37 @@
-    checkResponseElement('link[href][media]', 2)->
-    checkResponseElement('div.form-row label', 'Author article:', array('position' => 2))->
+    checkResponseElement('div.form-row select[name="unassociated_author_article[]"][through_class]', false)->
-    checkResponseElement('div.form-row select[name="unassociated_author_article[]"]', true, array('position' => 2))->
-    checkResponseElement('div.form-row select[name="unassociated_author_article[]"] option', 1)->
+    checkResponseElement('div.form-row select[name="associated_author_article[]"][through_class]', false)->
-    checkResponseElement('div.form-row select[name="associated_author_article[]"]', true, array('position' => 2))->
-    checkResponseElement('div.form-row select[name="associated_author_article[]"] option', 2)->
@@ -46 +48 @@
-    checkResponseElement('script[src]', false)->
-    checkResponseElement('div.form-row label', 'Author article:', array('position' => 2))->
+    checkResponseElement('div.form-row select[name="associated_author_article[]"][through_class]', false)->
-    checkResponseElement('div.form-row select[name="associated_author_article[]"][multiple="multiple"]', true)->
-    checkResponseElement('div.form-row select[name="associated_author_article[]"] option', 3)->
@@ -56 +59 @@
-    checkResponseElement('div.form-row input[type="checkbox"][name="associated_author_article[]"][checked="checked"]', 2)->
-    checkResponseElement('div.form-row input[type="checkbox"][name="associated_author_article[]"]', 3)->
+    checkResponseElement('div.form-row input[type="checkbox"][name="associated_author_article[]"][through_class]', false)->
-    checkResponseElement('div.form-row label[for="associated_author_article_1"]')->
-    checkResponseElement('div.form-row label[for="associated_author_article_2"]')->