Active Select doesn't work properly with Category's (Category Module) Parent and Container propeties selection. "Container" field reduced to one choice (current container that category belongs to). "Parents" field stays absolutely empty with no choice. Category module version 5.x-1.1

To reproduce:
1. Create Container1.
2. Create Category1 in Container1.
3. Create Container2 as child of Container1.
4. Set Container2 "Hierarchy" to multiple.
5. Set Container2's distant parents to both "this container" and "Container1" (without asterisk)
6. Create Category2 in Container2, try to set "Container" and "Parents" fields with Active Select enabled and disabled, feel the difference.

CommentFileSizeAuthor
#13 category_activeselect_4_7.patch897 bytesknseibert
#10 category_activeselect.patch850 bytesknseibert

Comments

vm’s picture

vm commented

I can confirm this. I was just walking myself through the tutorial found here: http://drupal.org/node/74747

and cannot get the parent drop down to populate.

vm’s picture

vm commented

everyone once in a while I can get the parent drop down to disappear though.

vm’s picture

vm commented
Priority: Normal » Critical

marking this as critical since it doesn't seem to work under any circumstances. Just tried with newest release of category and still cannot get the parent drop down to populate.

bdragon’s picture

bdragon commented
Assigned: Unassigned » bdragon

(assigning to self)

I'll be looking into the category+activeselect problems soon. Something's definately wrong with multiple hierarchy and Category...

vm’s picture

vm commented

Thanks bdragon!

slybanshee’s picture

slybanshee commented

This issue is also submitted in category's bugs:
http://drupal.org/node/115582 - made that duplicate because bdragon already assigned this here.

bdragon’s picture

bdragon commented

Hmm, the code is trying to {for ... in} an array. Bad bad bad...

Still working on it..

slybanshee’s picture

slybanshee commented

Thank you bdragon! Your work is very appreciated!
I've submitted one more issue on Active select: http://drupal.org/node/121047 It is concerning Internet Explorer behavior with Active Select. May be it would be suitable if you looked at this too (since you are digging the code by now). Number of users with IE still remain very large, so this bug becomes critical.

shane birley’s picture

shane birley commented

http://drupal.org/node/129027

knseibert’s picture

knseibert commented
Status: Active » Needs review
StatusFileSize
new category_activeselect.patch850 bytes

Hi there,
i found a solution which is for now working for me.
In the category_activeselect() function line round about 1400:

$options = _category_category_select_options(...);
foreach ($options as $key => $value) {
  ...
  $options[$key] = ...
}
$output = array(... $options ...);

These lines of code result in a mix of the result of _category_category_select_options and the additional $options provided within the foreach statement.
IMHO the $output should only contain the options from within foreach.
I have just replaced $options with $select_options in the first 2 lines (_category_category_select_options(...) results and foreach) and what can i say. For the time beeing it works for me.

Patch attached. Feedback appreciated.

inforeto’s picture

inforeto commented

Patch worked. Thanks for solving one of my problems!
Since it is a small patch i didn't find anything else to review and hope it gets commited soon.

summit’s picture

summit commented

Hi,
THis behaviour is also in 4.7.6.
Is this patch also working for 4.7.6?

greetings,
Martijn

knseibert’s picture

knseibert commented
StatusFileSize
new category_activeselect_4_7.patch897 bytes

Hi Martijn,
here's the patch for version 4.7. I just took a quick look at the code and it seems to be the same problem. However i did not test this patch.

bdragon’s picture

bdragon commented
Status: Needs review » Active

Patch from #10 committed to DRUPAL-4-7, DRUPAL-5, and HEAD.

Thanks!

--Brandon

knseibert’s picture

knseibert commented
Version: 5.x-1.0 » 5.x-1.x-dev

Problem still exists in current 5.x-1.x-dev CVS. Patch still valid.

liquidcms’s picture

liquidcms commented

I am not using ASelect but i get the same issue when editing a container.

When i edit, the "Parents" field is unset. So when i submit i get "Parents field is required".

I only have the one option under Parents: root.

Pretty sure this was fixed back when i used cat module in Dr4.7. But it is back now for Dr5.7 and lastest dev version of cat module.

haza’s picture

haza
Primary language French
commented
Status: Active » Closed (won't fix)

This version of Active Select is not supported anymore. The issue is closed for this reason.
Please upgrade to a supported version and feel free to reopen the issue on the new version if applicable.