I am using the then Fusion 7.x-1.x-dev (2011-Jun-01) (starter sub theme), seven as admin theme, Drupal 7.4
and skinr 7.x-2.x-dev (2011-Jul-15).
When I try to activate block position - only for fusion starter- I get following errors:

Notice: Trying to get property of non-object in skinr_ui_admin_skin_infos_submit() (Zeile 704 von /home/www/cleantech/sites/all/modules/skinr/skinr_ui.admin.inc).
....
-many times the same error
.... 
-and at last
PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'xb1278_db4.skinr_skinsets' doesn't exist: UPDATE {skinr_skinsets} SET status = 0; Array ( ) in skinr_ui_admin_skin_infos_submit() (Zeile 708 von /home/www/cleantech/sites/all/modules/skinr/skinr_ui.admin.inc).

Comments

moonray’s picture

moonray commented

You are missing tables that need to be installed.

You're going to have to disable, then uninstall Skinr. As previously stated many times, this is not a production ready module, and thus the upgrade path from previous versions is lacking.

Note that this will cause you to loose any Skinr related settings.

Morn’s picture

Morn commented

I deactivated skinr, uninstalled it, deleted the skinr files in .../sites/all/modules, downloaded a fresh version,
deleted caches, activated skinr, executed update.php

Then I simply saved the Skinr configuration (without any modification) and got the same error!

Grezvany13’s picture

Grezvany13 commented

I've got the same problem on a clean setup.

I found that skinr.install has the following code (line 190 - 199):

/**
 * Install the new {skinr_skinsets} and {skinr_skins} tables.
 *
 * @todo Should we remove this? It was ported from skinr_update_6002() and
 *   skinr_update_6003(), but should not have been.
 *
 * Contents removed because the tables added here are no longer required.
 */
function skinr_update_7003() {
}

And therefor doesn't create the skinr_skinsets table, which is used by the function "skinr_ui_admin_skin_infos_submit" (file: skinr_ui.admin.inc - line: 700)

The only 2 tables which are created are "skinr_rules" and "skinr_skins"

Satri’s picture

Satri commented

Subscribing: same problem with a fresh Skinr install on D7.

moonray’s picture

moonray commented

This is old code. The install file hasn't been updated due to too many and often radical changes. The updates will be taken care of before the first beta release.

moonray’s picture

moonray commented

We now have a proper migration path. See #1082686: Create migration path for Skinr from D6 to D7 version.
Does this fix the problem?

moonray’s picture

moonray commented
Status: Active » Postponed (maintainer needs more info)
moonray’s picture

moonray commented
Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

Closing. No new info, and most likely fixed.