As a co-maintainer of Nice messages, which uses also the jGrowl jQuery Plugin for displaying drupal system messages I would like to ask the maintainers of pmgrowl to change the path of jGrowl from inside the module folder to the libraries folder under sites/all/libraries and to point users to download and install jGrowl to the new proper location. So the jGrowl plugin will be loaded only once inside of one drupal installation and multiple moduls using jGrowl.

I put it already on the to-do list of the Nice messages module project.

Comments

Macronomicus’s picture

Macronomicus CreditAttribution: Macronomicus commented

This seems like a logical change... could you upload a patch against 7.x-1.x-dev ?

polskikrol’s picture

polskikrol CreditAttribution: polskikrol commented

Presume a patch was never submitted for this.

emcniece’s picture

emcniece CreditAttribution: emcniece commented

PLEASE! Multiple inclusion is unnecessary. Can anyone confirm what happens during the Drupal module upgrade process - are module folders removed and re-downloaded, or are existing files just overwritten? It would be unfortunate to have to re-install jgrowl within a module on every upgrade.

polskikrol’s picture

polskikrol CreditAttribution: polskikrol commented

Ran into this issue recently. Upgraded the module but the library was removed during the upgrade. The drupal upgrade refused to progress, resulting in an outage, until either (a) the library was downloaded or (b) module manually disabled via database modification.