The error message:

Undefined index: system in jquery_ui_library_alter() line 79

The attached patch fix the bug.

CommentFileSizeAuthor
#1 jquery_ui-HEAD.hook-library.patch914 bytessun
jquery_ui.module.patch742 bytesquiptime

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Fixed
StatusFileSize
new jquery_ui-HEAD.hook-library.patch914 bytes

Committed attached patch. However, this concept/flow no longer works, because 'add callbacks' has been removed from the core hook_library() patch. How to solve this, is to be discussed in a separate issue.

quiptime’s picture

quiptime commented

Drupal 6: Implementation of hook_foobar().

Drupal 7: Implement hook_foobar().

Other thing is:

function jquery_ui_library_alter(&$libraries, $module)

Implementation of hook_library() is not right. Right is:

Implement hook_library_alter().

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

I've fixed the wrong hook name, thanks.

But I won't adhere to the new "Implement ..." thing, because that change to coding standards was short-sighted, silly, and plain wrong. Additionally, it is intended to be a "temporary standard" only, or whatever. Sorry. -- Please note that I don't mean you with anything in this rant! ;) But thanks for noting it.

quiptime’s picture

quiptime commented

I think this is not a temporary standard: Converting 6.x modules to 7.x

Status: Fixed » Closed (fixed)

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