Add Commerce File field mapping

Yes, I'm pretty sure we'll have to add that as a mapping. Thanks for pointing that out. Patches are welcome.

I assume you're having to make changes in both commerce_file and commerce_feeds? If so, please post to an issue in commerce_file and crossreference it here.

Setting to "Needs review"

I think that both patches attached by rennevat are for commerce_feeds module.

Keeping it as needs review as I'd like to review and test this with more detail.

they are for commerce_feeds. but i think the code at http://drupal.org/node/1162602#comment-4581266 was supposed to import files that are already in the files table by their fid without duplicating them

the patch in #2 uses the files path and creates a fileduplicate

What's the status of this? Importing a file to a commerce file field is an essential feature.

Managed to piece together this snippet in my own custom FeedsProcessor by looking through the code from Commerce File. Seems to be creating Commerce File fields properly. Here, $new_product is a Commerce Product with a Commerce File field named "field_image_file" on it, and $filename is the full path to the file I want to sell access to:

                $file           = new stdClass();
                $file->uid      = 0;
                $file->uri      = $filename;
                $file->filemime = 'application/octet-stream';
                $file->filename = basename($file->uri);
                $file = file_save($file);
                
                // Attach default license data:
                $cf_defaults = field_info_instance_settings(COMMERCE_FILE_FIELD_TYPE);
                
                $new_product->field_image_file[LANGUAGE_NONE][0] = array(
                  'fid' => $file->fid,
                  'data' => $cf_defaults['data']
                );

Not sure if I did this properly... but seems to be working fine.

The attached patch builds on #2. Key changes are (a) always using private:// for the target file destination (this is what Commerce File defaults to), though it would be better if Commerce File exposed an API so that the scheme can be queried, and (b) attaching license data to the file (this is the "'field_commerce_file_data' cannot be null" error @rennevat ran into. I'm sure the code can be improved further, but it is working for me now.

One caveat is that files to be imported need to be reachable by drupal_http_request - this may not always be the case for file:// for example. My work around at present is to upload files (e.g. using sftp) into a path within public://

feeds support has been added natively to Commerce File at #1162602: Feeds support for Commerce File

That's great news, thanks. Shame that there has been duplicated effort. Setting this to won't fix with respect to Commerce Feeds.