I have replaced changed the line postion while changing the 'this' selector

$.browser.msie == true ?  $('#'+selectId).click(cckSelectOther.switchField) : $(this).change(cckSelectOther.switchField);

See Attachment

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mradcliffe’s picture

Status: Patch (to be ported) » Needs review
FileSize
1.16 KB

Does this patch work for you?

bakr’s picture

I have tested it, and the patch did not work on my machine (64-bit IE8)

Rather again, I have reverted to using: ('#'+selectId)

 $.browser.msie == true ? $(this).click(cckSelectOther.switchField) ..... // Did not work
 $.browser.msie == true ? $('#'+selectId).click(cckSelectOther.switchField) ..... // This works for Different Browsers

mradcliffe’s picture

Status: Needs review » Fixed
FileSize
1.4 KB

Thanks for your response. I've corrected it back to the way you had it, and I tested it in WinXP IE8 + IE7 compatibility mode.

Patch contains a couple of minor comment removals.

bakr’s picture

FileSize
70.29 KB

Very well done.

Just installed the latest dev release today and it includes the patch already. thanks

However, I have a further comment on the comments (:-) ... see attachment

Basically, line 19 and line 40 reads:

//@todo selectId and inputId may be modified, as seen in flexifield

My Comment is: Add a space before the '@' character, as IE JS compiler confuses this line being an actual statement rather than a gray commented statement.

Thats it.

With Best Regards

mradcliffe’s picture

Thank you.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.