Add field_ui.js in drupal library (so it can be overwrited) [#1494714]

Now field_ui.admin.inc attach CSS and JS directly with #attached.

615: $form['#attached']['css'][] = drupal_get_path('module', 'field_ui') . '/field_ui.css';
616: $form['#attached']['js'][] = drupal_get_path('module', 'field_ui') . '/field_ui.js';

This is a problem for jquery_update #1448494: jQuery 1.7: field_ui.js use attr for property we can not replace this file with hook_library_alter().

Should we make a form_alter or it will be more appropriate to add CSS and JS using hook_library and add_library?

Comments

Comment #1

gagarine commented 22 March 2012 at 20:00

Category:

feature

» support

I finnaly end up using hook_js_alter https://drupal.org/node/1448494#comment-5767046 . But I will like if someone can tell me if it's ok.

/**
 * Implementation of hook_library_alter().
 */
function jquery_update_js_alter(&$javascript) {
  // Swap out jQuery to use an updated version of the library.
  $path = drupal_get_path('module', 'jquery_update');
  if (isset($javascript['modules/field_ui/field_ui.js'])) {
    $javascript['modules/field_ui/field_ui.js']['data'] = $path . '/replace/modules/field_ui/field_ui.js';
  }
}