Dev Version works well

Modified patch committed to DRUPAL-5 and HEAD.

http://drupal.org/cvs?commit=63386
http://drupal.org/cvs?commit=63387

Pushing it works on all browsers, right? Seemed like a simpler solution.

Please reopen if my changes to your patch broke anything.

Thanks,
--Brandon

Hi Brandon,
i've installed the latest cvs version for Drupal 5.1. Firefox works great under XP SP2. Opera and IE7 fail. Both remain blank.

Here's the Opera error log:
JavaScript - http://domain/node/add/page
Unknown thread
Error:
name: TypeError
message: Statement on line 152: Type mismatch (usually a non-object value used where an object is required)
Backtrace:
Line 152 of linked script http://domain/sites/all/modules/activeselect/activeselect.js
this.targets[targetIndex].options.push(new Option(text, value, false, selected));
Line 246 of linked script http://domain/sites/all/modules/activeselect/activeselect.js
db.owner.populate(targets);
Line 1 of eval script
if (xml && (xml.readyState == 4 || isTimeout == "timeout"))
At unknown location
[statement source code not available]

OK, rolling back to the 4.7 version of the line, which accomplishes the same thing as the patch without using a temporary variable.

Thanks for applying a version of the patch. Or at least rolling back to a version that had the same effect. I can confirm that the Apr 9, 2007 development version of this module now works great with content_taxonomy under Firefox and IE 6. (I did not test with IE 7 but expect the same result.)

As far as I am concerned this bug report can be closed.

The following works for me:

Change line 156 (approx) from activeselect.js

-- $(this.targets[targetIndex]).append(new Option(text, value, false, selected));

++ if (navigator.appName == "Microsoft Internet Explorer") {
++ this.targets[targetIndex].add(new Option(text, value, false, selected));
++ }else{
++ $(this.targets[targetIndex]).append(new Option(text, value, false, selected));
++ }

Thanks to olemsa: http://drupal.org/node/147076#comment-561190