I am working on a language hierarchy module that has changed the schema of the languages table to add an extra field. The full language object is exported by features with my extra field. However, the revert function does not work because the list of fields is hard coded.

See includes/features.locale.inc line 143

// @TODO: get properties from schema.
$properties = array('language', 'name', 'native', 'direction', 'enabled', 'plurals', 'formula', 'domain', 'prefix', 'weight', 'javascript');

I'll be adding a patch to fix this.

CommentFileSizeAuthor
#4 features-2145453-4.patch1.05 KBdanepowell
#1 features-locale-reverting-2145453.patch732 bytescarinadigital

Comments

carinadigital’s picture

carinadigital commented
StatusFileSize
new features-locale-reverting-2145453.patch732 bytes

This patch won't address the initial creating of the languages with any altered field data. However, if you revert after their initial creation by features your custom schema field data will be restored.

henrijs.seso’s picture

henrijs.seso commented
Status: Active » Needs review

Yes, patch works. Needs another review?

rodrigoaguilera’s picture

rodrigoaguilera
Primary language Spanish
Location Madrid
commented
Status: Needs review » Needs work

There's extra spaces in the patch

danepowell’s picture

danepowell commented
Status: Needs work » Needs review
StatusFileSize
new features-2145453-4.patch1.05 KB

Here is #1 with code standards fixed. Works like a charm for me!

rodrigoaguilera’s picture

rodrigoaguilera
Primary language Spanish
Location Madrid
commented
Status: Needs review » Reviewed & tested by the community

I can confirm that this works making the features not overriden.

  • mpotter committed 1491dd5 on 7.x-2.x authored by Dane Powell 
    Issue #2145453 by carinadigital, Dane Powell: Features locale (languages...
mpotter’s picture

mpotter commented
Status: Reviewed & tested by the community » Fixed

Committed to 1491dd5.

Status: Fixed » Closed (fixed)

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