since upgrading to alpha 7, i've been getting this error: http://drupal.org/node/370110

I'm not sure if its a filefield issue or jquery media issue.

Comments

akolahi’s picture

akolahi commented

FYI: I've just downgraded to filefield alpha 6 and it "fixed" the problem

quicksketch’s picture

quicksketch
he/him
commented

I'm almost certain this is caused by FileField's aggressive addition of settings into the Drupal.settings array. This chunk of code may adversely affect other modules that use Drupal settings.

The problem code:

  // AHAH is not being nice to us and doesn't know the "other" button (that is,
  // either "Upload" or "Delete") yet. Which in turn causes it not to attach
  // AHAH behaviours after replacing the element. So we need to tell it first.
  $javascript = drupal_add_js(NULL, NULL);
  if (isset($javascript['setting'])) {
    $output .= '<script type="text/javascript">jQuery.extend(Drupal.settings, '. drupal_to_js(call_user_func_array('array_merge_recursive', $javascript['setting'])) .');</script>';
  }

Instead of merging in all settings, we should somehow only merge in the AHAH settings for the "Delete" button that is added.

quicksketch’s picture

quicksketch
he/him
commented
Status: Active » Fixed

This should be fixed with #372401: Merge only needed FileField Drupal.settings.

Arban’s picture

Arban commented

After upgrading filefield from alpha 7 to 6.x-3.x-dev, I still have the message 'a script on this page is causing Internet explorer to run slowly'. IE 6 and 7 gives an error, but only IE 6 still give a message box...

Status: Fixed » Closed (fixed)

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

adrianmak’s picture

adrianmak commented

Does this problem fix by 3.0-rc1 ?

adrianmak’s picture

adrianmak commented

any update on this issue ?

SimonVlc’s picture

SimonVlc commented

Any update on this? I just tried the new 6.x.3.1 and got the same error...