i try this module and it worked great until a colleague of mine told me that all of ubercart product dimensions (length, width, height) did not imported properly, they were all zeros
so when i check the data from export file, the dimensions fields have complete values inside them
but when i import it to other drupal site(s), their values is reverted to zero (default value)
when i check the source code, all dimensions fields have 'dim-' as prefix while on export data i did not see any kind of prefix
have you guys also found the same problem as i do?
any solutions? or do i really have to hack the source code to solve this one?
thank you

Comments

Enzomaticus’s picture

One thought - did you create the uc product type via classes or through content type? I noticed that when importing we had the same problem. I then re-created the UC product type using product classes (the same way we did it on the export site) and it imported fine.

Hope this helps!

danielb’s picture

Category: bug » support

I don't know the answer to this one. All I know is if modules that attach data to nodes do it properly using hooks on node_load, node_save, etc... then the data should all be in the node. If there are external dependencies (data in other tables) that aren't put into the node, then things might not work.

danielb’s picture

Status: Active » Fixed

I'm not sure there is anything that can be done about this. If you find out that Node Export is doing something incorrectly, feel free to reopen.

Status: Fixed » Closed (fixed)

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

Dimm’s picture

Component: Node Export » Node export

Node export 6.x-2.24 - ength, width, height did not imported properly

Dimm’s picture

Patch:
file node_export.module,
function node_export_save(&$node) {
.....
after:
$op = 'update';
}

insert:
$node->dim_length = $node->length;
$node->dim_width = $node->width;
$node->dim_height = $node->height;

Dimm’s picture

Category: support » bug
Status: Closed (fixed) » Patch (to be ported)
danielb’s picture

Any explanation as to why the variables have to be renamed for the save to work correctly? Isn't that a little strange??

danielb’s picture

Status: Patch (to be ported) » Closed (works as designed)

It looks like ubercart_product already puts these values into both places when a node gets saved?
http://drupal.org/node/685698
http://drupal.org/files/issues/685698_update_dims.patch

I don't understand why they've done it like that. I think this is a design issue with ubercart. We really shouldn't have to handle special case properties for other contrib modules like this.

Dimm’s picture

Dimm’s picture

drupalinthailand’s picture

Issue summary: View changes

Hello,

Do you know how I could import my ubercart products in drupal 8 ?

Thanks for your help.