OK I couldn't get this module to install. I just moved the function around but there may some other preferred solution.

CommentFileSizeAuthor
#1 jquery_ui-paper-bag-263345-1.patch1.75 KBwebchick
jquery_ui.patch2.48 KBmfb

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new jquery_ui-paper-bag-263345-1.patch1.75 KB

Oh DANG IT ALL TO HECK. :P

Committed this, will release 1.1 in a moment. ;)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Ok, this is now fixed. I unpublished the 1.0 release for good measure. THANK YOU.

Btw, the reason I want the get_version function in the .module file is so modules could call if (jquery_ui_get_version < $something) { drupal_set_message('OMG! ERROR'); } and if it's in the .install file, that won't work since .install files are only loaded at certain times.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Priority: Normal » Critical
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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

doublejosh’s picture

doublejosh commented

PS: If the module is installed at modules/contrib/jquery_ui the installation can't find the .module file to load the jquery_ui_get_version function.

So it has to be installed at modules/jquery_ui

Bummer, but not the end of the world.