I was able to install Date Popup without first installing the jQuery UI (yes, schoolboy error) - if the jQuery UI module was listed as a dependency for Date Popup, this wouldn't have been possible.

Comments

horuskol’s picture

Status: Active » Needs review
StatusFileSize
new459 bytes

Created a patch for this issue

arlinsandbulte’s picture

The problem with this is that since jQuery_UI was not originally a dependency, things can go very badly when upgrading date if jQuery_UI is NOT already installed.

See #22 and #23 here for more:
http://drupal.org/node/436490#comment-3325146

I think these are the core issues Karen is talking about there:
#228860: Upgrading a module with a newly added dependencies breaks things badly
#799314: Checking/resolving new dependencies for updated modules

I am not sure where the status of those two issues leaves us.... (the first is 'fixed').
So, I will leave this open for now until I find out or if Karen knows more and can chip in.

horuskol’s picture

Thanks for the links - I didn't realise that there was such a problem in the core.

karens’s picture

Status: Needs review » Closed (won't fix)

This is not ideal, but I don't have any better ideas than what is already done.