Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Was having a play with this module this afternoon and thought i'd have a go at the JSON storage format. Attached is a patch, it's not complete but it's a start.
To Do
- Add tests
- Test on PHP 5.3. We're using JSON_PRETTY_PRINT which wasn't added until PHP 5.4. I'd presume it would just be ignored but can say for sure.
- Refactor the admin configuration form so we don't have a hardcoded array of classes.
If anyone else wants to have a go at this, feel free to takeover.
Comment | File | Size | Author |
---|---|---|---|
#6 | 0001-Added-ability-to-export-configuration.inc-as-json.patch | 7.84 KB | Josh Waihi |
#4 | configuration-add-json-storage-format-2125837-4_0.patch | 5.64 KB | benjy |
configuration-add-json-storage-format.patch | 5.85 KB | benjy | |
Comments
Comment #1
benjy CreditAttribution: benjy commentedComment #3
dagmarSeems this patch was created using another repository. We shouldn't see this
a/sites/all/modules/
In patches.
Comment #4
benjy CreditAttribution: benjy commentedYeah my mistake. I didn't actually have the module in source control alone, I just dropped it into a site for testing.
Comment #5
Josh Waihi CreditAttribution: Josh Waihi at Acquia for Flight Centre Travel Group commentedpatch works well (still) but needs support to also export configuration.inc json.
Comment #6
Josh Waihi CreditAttribution: Josh Waihi as a volunteer and at Acquia commentedHere is a patch that exports configurations to JSON - however it will not likely be able to read it back in again (yet).