- Issue created by @Greg Boggs
- 🇮🇳India sidharth_soman Bangalore
I've introduced a try-catch block to make sure execution doesn't stop. I'm not sure how viable it would be to check if a file is valid or exists before wrapping it... since you need the URI in the first place to check if the file exists. Correct me if I'm wrong about this.
- 🇺🇸United States Greg Boggs Portland Oregon
You still have an item object, so item is not empty.
Looks like this is a core bug:
https://www.drupal.org/project/drupal/issues/3269813 💬 Drupal\Core\File\Exception\InvalidStreamWrapperException: in Drupal\Core\File\FileUrlGenerator->doGenerateString() (line 106 of /docroot/core/lib/Drupal/Core/File/FileUrlGenerator.php) Closed: outdated
- First commit to issue fork.
- last update
12 months ago 717 pass - last update
12 months ago 709 pass, 2 fail - Status changed to Needs review
12 months ago 2:22pm 22 March 2024 - last update
12 months ago Patch Failed to Apply - 🇳🇱Netherlands megachriz
I can reproduce the issue after I disable the private file system on a site (by commenting out
$settings['file_private_path']
in settings.php). So therefore I think it is worth to fix this in Feeds.The tests only MR should fail tests.
The fix is similar to that of the patch in #2, the difference is that the url will be rendered as plain text if generating it as link failed.
- last update
11 months ago 717 pass -
MegaChriz →
committed 34268245 on 8.x-3.x
Issue #3412146 by MegaChriz, sidharth_soman, Greg Boggs: Fixed missing...
-
MegaChriz →
committed 34268245 on 8.x-3.x
- Status changed to Fixed
11 months ago 7:55am 1 May 2024 Automatically closed - issue fixed for 2 weeks with no activity.