The export for commerce_customer is improperly namespaced as commerce_customer_type. Patch coming as soon as I have an issue id.
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | commerce_features-n2292895-11.patch | 1.49 KB | ssvargass |
The export for commerce_customer is improperly namespaced as commerce_customer_type. Patch coming as soon as I have an issue id.
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | commerce_features-n2292895-11.patch | 1.49 KB | ssvargass |
Comments
Comment #1
lee20 commentedComment #2
aleksijohansson commentedPatch seems to be working fine!
Comment #3
dmsmidt+1 for getting this in, works like a charm.
Comment #4
br0kenIntegration does not work now! When I'm creating a feature, then I have
features[commerce_customer_type][] = prescriberin info file, but in code we can see empty function that export nothing:commerce_customer_features_export_render().Comment #5
mglamanComment #6
jorgegc commentedHi @BR0kEN, what integration are you talking about? As far as I am concerned all this module does at the moment is add an entry to your module's .info file like this "features[commerce_customer][] = billing" and tell Features what fields to add as a dependency.
The patch in #1 actually works. I just don't understand why somebody would want to have features export nothing. What am I missing here?
Comment #7
damienmckennaThe patch doesn't resolve the problem for me - I add customer types to the export but they aren't added to the features.inc file. This patch updates commerce_customer_features_export_render() to add the export type.
Comment #8
damienmckennaThis corrects some mistaken "coupon" strings with "customer".
Comment #9
damienmckennaSorry, coding while tired.
Comment #10
jorgegc commentedHi @DamienMcKenna, although your patch fixes the export part, the hook_commerce_customer_default_types doesn't actually work. I have tried changing the details in the hook but features doesn't pick them up.
I am uploading a new patch that changes the default hook.
Comment #11
sumthief commentedHi! Patch from #10 works for me (now I can to export customer profile types between different instances via features).
I think if there someone else will confirm that this patch works, we can set RTBC for this issue.
Comment #12
br0kenComment #13
mglamanNo. We should not have order types loaded from the customer profile type.
Comment #14
ssvargass commentedHi @jorgegc, Your patch is working for me, but is not necessary modify the .module file as @mglaman indicated.
This is the new patch and it's working for me.
Comment #15
mglamanComment #16
jorgegc commentedHi @ssvargass, thanks for making the changes that @mglaman suggested. I have reviewed the patch in #14 and can confirm that it works as expected. Shall we RTBC?
Cheers!
Comment #17
br0kenComment #19
mglamanCommitted, thanks everyone.