When importing a file URI that is inside the root public directory, ex: "public://some-image.jpg", the logic that determines if the trailing slash should be used is handled incorrectly. Instead of hardcoding the forward slash, it should be dynamically determined based on the destination value.

CommentFileSizeAuthor
#2 feeds-remove_extra_trailing_slash-2904220-2.patch3.31 KBcoderbrandon

Comments

CoderBrandon created an issue. See original summary.

coderbrandon’s picture

coderbrandon commented
StatusFileSize
new feeds-remove_extra_trailing_slash-2904220-2.patch3.31 KB
megachriz’s picture

megachriz
Primary language Dutch
commented
Status: Active » Needs review

Can you give me the easiest way to reproduce the issue? This way the bug can be backed up with an automated test.

bluegeek9’s picture

bluegeek9 commented
Status: Needs review » Closed (outdated)

Drupal 7 reached end of life and the D7 version of Feeds is no longer being developed. To keep the issue queue focused on supported versions, we’re closing older D7 issues.

If you still have questions about using Feeds on Drupal 7, feel free to ask. While we won’t fix D7 bugs anymore, we’re happy to offer guidance to help you move forward. You can do so by opening (or reopening) a D7 issue, or by reaching out in the #feeds channel on Drupal Slack.

If this issue is still relevant for Drupal 10+, please open a follow-up issue or merge request with proposed changes. Contributions are always welcome!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.