When going from D6 to D7, updates should be able to be run before the module is updated AFAIK. In this case, 7001 creates skinr, but it doesn't create the other skinr tables (which are later manipulated in 7002):

Failed: DatabaseSchemaObjectDoesNotExistException: Cannot add field skinr_rules.rule_type: table doesn't exist. in DatabaseSchema_mysql->addField() (line 321 of /Users/jacob/www/d7up.ds.dev/includes/database/mysql/schema.inc).

Is the error.

CommentFileSizeAuthor
#6 patch_commit_78aa31722341.patch13.04 KBmoonray

Comments

moonray’s picture

moonray commented
Title: During update.php when migrating to D7, 7002 fails because the table skinr_rules doesn't exist » Create migration path for Skinr from D6 to D7 version

Drupal 7 is still in active development. The update path doesn't work. This is known.
That being said, thanks for testing anyway. :)

moonray’s picture

moonray commented
Status: Active » Postponed

Need to wait until we've got the module all working before starting work on this.

drupalusering’s picture

drupalusering commented

Note, do not use if upgrading until this issue resolved. Breaks site. If you do consider enabling the module even after the update throws an error certainly backup prior. If you have not backed up delete skinr folder - should fix the error

moonray’s picture

moonray commented

Once #1007482: Make use of Context API instead of custom rules is committed we can start working on this upgrade path.

moonray’s picture

moonray commented
Status: Postponed » Active
moonray’s picture

moonray commented
Category: bug » task
Status: Active » Fixed
StatusFileSize
new patch_commit_78aa31722341.patch13.04 KB

Committed the attached patch.
The patch adds an upgrade path from any previous version of skinr (6.x-1.x, 6.x-2.x, and older versions of 7.x-2.x).

Status: Fixed » Closed (fixed)
Issue tags: -7x upgrade

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

Issue tags: +7x upgrade

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.