This is similar to #931472: Persistent:"Date Popup module now requires the jQuery UI module as a source for the datepicker. Please install it immediately." .

The check for jquery_ui is modified by the check for uid 1, so the return doesn't fire for other users.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

Status: Fixed » Closed (fixed)

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

pfrenssen’s picture

Status: Closed (fixed) » Needs review
FileSize
1.28 KB

This needs a followup. The function jquery_ui_get_path() has been introduced in jQuery UI 6.x-1.5 so people that have older versions of jQuery UI installed do not see the warning and still get the fatal error.

unc0nnected’s picture

Problem still persists.. I grabbed the latest dev version, I took date_popup.module from the git repo and implemented that and tried the suggestions mentioned here: http://drupal.org/node/931472

Problem still persists, prevents all pages from loaded except for update.php and replaces them with a nice 500 error message message. The domains error log shows:

[Tue Jul 31 07:29:09 2012] [error] [client 127.0.0.1] PHP Fatal error: Call to undefined function jquery_ui_get_path() in /path/to/drupal/modules/date/date_popup/date_popup.module on line 102

Removed the date_popup directory all together stops the issue so hopefully you don't need it.

pfrenssen’s picture

unc0nnected, have you tested the patch I posted in #3? It should solve the problem.

unc0nnected’s picture

Sorry, I just saw that while the jquery update was done the module itself wasn't activated.. once activated teh problem stopped

pfrenssen’s picture

Well, yes, jQuery UI is now required, so installing and activating it makes the fatal error go away. This issue tries to improve this by giving the users a friendlier error message ;)

naero’s picture

I stumbled upon this error, too, when updating Date from 6.x-2.8 to 6.x-2.9. I can confirm that this patch fixed it and I am currently at 6.x-2.9+6-dev.

Reuben Unruh’s picture

Should the date popup .info file include the jQuery UI 6.x-1.5 dependency?

csc4’s picture

Posted by Reuben Unruh on November 9, 2012 at 4:08pm

Should the date popup .info file include the jQuery UI 6.x-1.5 dependency?

Think that would be a great idea - would certainly have saved me some stress this afternoon!

Beanjammin’s picture

Which leads us to this issue on exactly that http://drupal.org/node/973924

gpk’s picture

behindthepage’s picture

Version: 6.x-2.x-dev » 6.x-2.9

I have just updated to Date 6.x-2.9 and this is still a problem. It crashed my site and I had to do some investigating. I am suprised that this is still an issue when there seems to have been a solution on this thread a year ago.

Regards
Geoff

Beanjammin’s picture

Status: Needs review » Reviewed & tested by the community

The patch from #3 applies cleanly to date 6.x-2.9 and I was able to successfully update the module despite having jquery_ui-6.x-1.4 installed.

bala_28’s picture

This error could be due to the low version of jquery.ui. Check for jquery_ui_get_path() function within jquery.ui module. If its missing, copy this code found in this link and paste it in your module or simply update the jquery.ui module to new version.

Hope this solution will work out.. :)

cafuego’s picture

Version: 6.x-2.9 » 6.x-2.x-dev
Status: Reviewed & tested by the community » Fixed

Patch committed to 6.x-2.x-dev.

pfrenssen’s picture

Thanks for finally committing this! If you don't mind, can you please attribute commits in the future? See Adding a commit author. Thanks!

Status: Fixed » Closed (fixed)

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