Closed (fixed)
Project:
Physical Fields
Version:
7.x-1.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
10 Jun 2012 at 17:12 UTC
Updated:
29 Nov 2016 at 13:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
johnvDo you use the Migrate module? Feeds-module integration works fine with this patch: #1372070: Integrate with Feeds (Patch Ready)
Comment #2
gilles_c commentedYes, I'm working with migrate.
Comment #3
delta commentedFor the physical_weight field use this class as fieldHandler arguments :
use with:
Comment #4
delta commentedfor the Physical dimensions field use this fieldHandler
use with :
in prepareRow migrate method i prepare the dimensions row value:
and don't forget to add 'dimensions' to source_fields argument of MigrateSource
Comment #5
gilles_c commentedThanks a lot ! I'll try this week.
Comment #6
mikeryanSee also #1671108: Support for physical_weight field - one of these should be marked a dupe of the other. The other issue has the advantage of having a patch...
Comment #7
cameron prince commentedI tried this and weight worked perfectly, but dimensions wasn't working. I updated it based on code from addressfield and it seems to be working now.
Comment #8
fearlsgroove commentedHere's an updated patch with subfield support. Marking the other as a dupe
Comment #9
dwkitchen commentedTested and works
Comment #10
bleedev commentedUpdated patch using subfields for dimensions. Length is the dimension key and weight is the weight key.
Comment #11
johnvbetter title for my dashboard.
Comment #12
jcisio commentedNew patch, based on #10, but:
Tested with all the 3 dimensions and the weight field. But I'm not going to RTBC my patch.
Comment #13
ayalon commentedThe patch in #12 is not working, if you have a single value field and no values on the entity.
I made a new patch.
Comment #14
daletrexelI was going to ask a question about the behavior of the patch I was seeing, but in the process of writing my question, I discovered the source of my problem. (Applying the patch to Commerce Kickstart's profile structure did not go as error-free as I'd thought.) I've now got it working, and it appears to be working smoothly for me.
So, instead I'd just like to thank you all for the work you put into this patch! It's quite useful and I hope it gets approved and implemented.
Comment #15
ayalon commentedIf you want this patch to be commited, set the status to reviewed.
Comment #16
daletrexelHow much authority does it take to set the status to reviewed? I can say that the patch is working for me, but I'm not exactly the right person to tell whether it meets Drupal's coding standards.
Comment #17
mikl#13 works great, thanks.
Comment #18
karlshea#13 also worked for me.
Comment #20
bojanz commentedCommitted, thanks.