I will post a patch for features integration for the 7.x version of the module in this issue.

CommentFileSizeAuthor
#11 1424534-features_integration-11.patch6.66 KBguillaumev
#9 1424534-features_integration-9.patch7.44 KBguillaumev
#8 0001-Issue-1424534-by-guillaumev-Added-Features-integrati.patch7.96 KBszantog
#4 custom_pub-features_integration-1424534-4.patch9.79 KBguillaumev
#3 custom_pub-features_integration-1424534-3.patch7.59 KBguillaumev
#2 custom_pub-features_integration-1424534-2.patch7.75 KBguillaumev
#1 custom_pub-features_integration-1424534-1.patch7.75 KBguillaumev

Comments

guillaumev’s picture

guillaumev commented
Status: Active » Needs review
StatusFileSize
new custom_pub-features_integration-1424534-1.patch7.75 KB

Here is the patch...

guillaumev’s picture

guillaumev commented
StatusFileSize
new custom_pub-features_integration-1424534-2.patch7.75 KB

Same patch without whitespace issues...

guillaumev’s picture

guillaumev commented
StatusFileSize
new custom_pub-features_integration-1424534-3.patch7.59 KB

New patch with a few bug fixes...

guillaumev’s picture

guillaumev commented
StatusFileSize
new custom_pub-features_integration-1424534-4.patch9.79 KB

New patch combining features integration and rules integration (see http://drupal.org/node/1424290)

kevinquillen’s picture

kevinquillen commented

Will review soon.

chertzog’s picture

chertzog
he/him
Primary language English
Location Harrisburg, PA, USA
commented
Status: Needs review » Reviewed & tested by the community

No problems here.

kevinquillen’s picture

kevinquillen commented

Thanks all. Sorry, been so swamped lately.

szantog’s picture

szantog commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 0001-Issue-1424534-by-guillaumev-Added-Features-integrati.patch7.96 KB

Patch no more applicable, rerolled against latest dev. Only this issue! So needs review again..

guillaumev’s picture

guillaumev commented
StatusFileSize
new 1424534-features_integration-9.patch7.44 KB

szantog > I wasn't able to apply your patch (it gave me errors). Here is a new patch which should apply properly, although I haven't tested it yet.

kevinquillen’s picture

kevinquillen commented

I don't use Features at all really, so I am unsure if the patch above works as intended.

I assume Features integration would allow CPO to appear on Feature builders and allow it to be exported, and imported cleanly into another site?

What would I be looking for here as a working patch? If I know that I can help review it with you.

guillaumev’s picture

guillaumev commented
StatusFileSize
new 1424534-features_integration-11.patch6.66 KB

Here is a new patch which applies on the latest dev version.

kevinquillen > Yes, features integration allows CPO to appear on Feature builders, which allows it to be exported in a feature and imported into another site, exactly as you described. In order to review the patch, you would need to create a CPO, export it into a feature, then install the feature in another site and make sure the CPO is installed properly in this other site...

kevinquillen’s picture

kevinquillen commented

Alright let me see if I can get this going on Features. I'll have to create another sandbox on my machine with Features and CPO. If so, we can knock this out and have a clean issue queue and put out 1.0!

kevinquillen’s picture

kevinquillen commented
Status: Needs review » Reviewed & tested by the community

Alright, I installed a new sandbox with Features and CPO, applied the patch to both sandboxes and exported two CPO options from the main to a feature, enabled it on the other sandbox without error and had the two options available to use.

It would seem like this works.

kevinquillen’s picture

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

Status: Fixed » Closed (fixed)

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