'image' field is broken after D5 -> D6 upgrade due to module path change.

raintonr - June 28, 2009 - 04:30
Project:Smileys
Version:6.x-1.0-alpha5
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Description

On upgrading from D5 to D6 the smiley 'image' paths are broken as the smileys module will live under sites/all/modules or sites/<sitename>/modules.

A database upgrade step is required to detect the path smileys is currently in (no doubt using drupal_get_path('module', 'smileys') and change the leading modules/smileys in the smileys.image database field to that path.

I have marked this as critical as it completely breaks smileys functionality after D6 upgrade.

#1

Gurpartap Singh - June 28, 2009 - 05:05
Priority:critical» normal

Not sure if I want to do this. Because changing the directory where modules are placed, isn't enforced in Drupal 6. You can still keep them in modules directory. If the location to the smileys is changed, it's up to the administrator to update their paths. This is so, because some users may not change smiley packs folder while upgrading and we can't force a change in image path. (Not the sole reason though; Not sure either.)

The D5 to D6 update script, as required, has already been included in D6 release of module, and proven to work for everyone so far too!

What do you think?

#2

Gurpartap Singh - June 28, 2009 - 05:07

Also, now you may want to fetch the latest 6.x-1.x-dev release, and move your smileys to sites/default/files/smileys (default or something else) folder. Just to be safe for future upgrades.

 
 

Drupal is a registered trademark of Dries Buytaert.