Update required for new fields [#1964822]

You have implemented export feature for style, and also added admin_title for each style set.
But you didn't implemented update function to add this column and remove 'description' in already working installations.

Comment	File	Size	Author
#2	semantic_panels-update_hook-1.patch	930 bytes	berdyshev
#1	update.patch	724 bytes	berdyshev

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

berdyshev CreditAttribution: berdyshev commented 8 April 2013 at 19:30

Status:

Active

» Needs review

File	Size
update.patch	724 bytes

Here is the patch

Comment #2

berdyshev CreditAttribution: berdyshev commented 11 April 2013 at 17:14

Title:

Update required for admin_title field

» Update required for new fields

File	Size
semantic_panels-update_hook-1.patch	930 bytes

I see there is one more field appeared after improving exportable styles. Here is updated patch

Comment #3

emattias CreditAttribution: emattias commented 19 April 2013 at 12:17

Status:

Needs review

» Closed (fixed)

I've realized that pushing the experimental style export code that could (and did) change fundamentally wasn't "ready" for the dev branch so it's now moved to a wip branch. Everything that has to do with the export has been removed from the dev branch.

If somebody needs to go from first implementation of style exports to second implementation, this might be useful though.

Comment #4

berdyshev CreditAttribution: berdyshev commented 20 April 2013 at 07:58

Ok, I have created sandbox, where all fixes are added. The export style functionality works good here.
If you are ok, we can merge it as new version of Semantic Panels module, like 7.x-2.x.
Here is the link to my sandbox: http://drupal.org/sandbox/BerdArt/1975342
I'm also planning to simplify a bit UI for the style configuration.

Thanks,
Artem