PDOException: %message in skinr_update_7002() [#945086]

I get this PDOException when running update.php to update Skinr 6.x-2.x-dev (2010-Oct-08), latest dev releases of both D7 and Skinr (Oct-18)

I have snapshots of code and db saved so I can easily go back and re-run for testing

Screen dumps of relevant area attached as well as skinr snippet from the sql dump I import before I run update.php, there are no data in the tables

If you need more info let me know.

Comment	File	Size	Author
#2	945086.patch	601 bytes	ezra-g
	skinr_sql_dump.txt	2.37 KB	vikingew
	skinr_update_7002.gif	6.13 KB	vikingew
	skinr_pending_updates.gif	8.23 KB	vikingew

Comments

Comment #1

vikingew commented 18 October 2010 at 20:47

Status:

Active

» Needs review

Well made the effort to read the code a bit and a quite simple fix really ;-)

Index: sites/all/modules/skinr/skinr.install
===================================================================
--- sites/all/modules/skinr/skinr.install       (revision 920)
+++ sites/all/modules/skinr/skinr.install       (working copy)
@@ -289,7 +289,7 @@
       'default' => '',
     ));
     db_update('skinr_rules')
-      ->fields(array('type' => 'page'))
+      ->fields(array('rule_type' => 'page'))
       ->execute();
     db_update('skinr')
       ->fields(array('module' => 'rules'))

I don't attach it as a patch because I maintain my sites in a svn repos using the technique of vendor drops for core and modules, so the patch would probably fail anyway although it's correct.