File target: add support for local file path as source

Rebased again to bring up upstream changes, so the patch can apply.

The patch is available at https://git.drupalcode.org/project/feeds/-/merge_requests/28.patch, but I'm saving a static copy here as well.

Still seems to be an issue with Drupal 10. How do I use the feeds module to import feeds with images from a ../private/ directory? I have tried private://image.jpg and I get:

cURL error 1: Protocol "private" not supported or disabled in libcurl (see https://curl.haxx.se/libcurl/c/libcurl-errors.html) for private://image.jpg

Still an issue with D10 and commerce imports
I'm working locally and views data export only gives local file paths, so I downloaded all the images and placed them in the folders that matched the csv structure. The import fails with cURL error 3 but succeeds if image field is not required and I leave it out of the import.

I guess I'll be editing my csv files or is there another way to add the local file path?

fixing some codeing standard

Thank you for a great module, which has been my go-to for years now. I am currently converting a site from D7 to D10 and really need to be able to import local files using the public:// path. Neither the patch in #45 nor the merge request will apply in Drupal 10.3.1. Any chance of an update?

@tsurvuli
Hm, perhaps if you need this quickly, could you try the patch/code applied on Feeds 8.x-3.0-beta4 instead? Biggest things that you'll miss is the possibility to skip entity validation and importing users with roles (a change in Drupal 10.2 broke role import for Feeds 8.x-3.0-beta4).

All my focus is on getting Feeds (and some modules that extend Feeds) ready for Drupal 11.

@tsurvuli please try the patch in #41. It does apply to Drupal 10.3.1

I have no idea why the patch in #45 was placed in this issue.
@rohit.rawat619 could you open up another issue and place your patch there. It is not germain to this issue.

@SocialNicheGuru
Looking at the patch in #45 now, I see it is indeed only coding standard fixes. There's no need to open a new issue for that. Except a few that are explicitly ignored, all coding standard issues have been fixed.

@SocialNicheGuru, for what it's worth I was able to successfully apply the patch in #41, but only after I removed the patch from issue 2928904 ✨ Add a mapping target to media field Needs review , which I had already added. For some reason, if either patch is present in the composer file, the other won't apply. Right now I have 2928904's patch in my composer file. (And I have the beta4 version of Feeds.)

Since I am trying to create both a series of media importers that will import files as media, and also a node importer that will then import those local media files into node content, it would be extremely helpful if both patches were available to use at the same time. Thanks for the quick responses from you and @MegaCriz.

that's what happens when waiting for 7 years before merging patches

The status is incorrect; this is not ready for review. It still needs tests (#31) and there are the issues I pointed out in #33 and #34.