Closed (fixed)
Project:
Automatic Updates
Version:
3.0.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
28 Feb 2024 at 09:29 UTC
Updated:
20 Mar 2024 at 04:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
abhishek_gupta1 commentedComment #3
abhishek_gupta1 commented@rehm, fixed the issue created Patch . please review it.
Comment #4
tedbow@rehm did you install the module via Composer? Can provide your composer lock file? I wondering if there is some problem with your version of php-tuf/composer-stager for some reason.
@abhishek_gupta1 thanks for the patch but first we need to figure why there is boolean instead of string provide instead of just throwing an exception.
Comment #5
tedbowI wondering if the problem comes from
\Drupal\package_manager\PathExcluder\SiteFilesExcluder::excludeSiteFilesrealpath()could return a falseComment #6
tedbow@rehm this could be caused by your private or public file settings point to path that does not exist. These are usually set in setting.php . $settings['file_public_path'] or $settings['file_private_path'] you can check the current values at admin/config/media/file-system
I set
and it caused the same error you experienced.
@rehm can you check these paths and see if they point to real paths?
Comment #8
tedbowComment #9
tedbowComment #11
tedbowComment #12
phenaproximaOne request (we should use the file_system service's realpath() function), otherwise RTBC from me. We might want to also add an @see to the test, or maybe a more expansive comment, to explain what's going on.
Comment #13
tedbow@phenaproxima back to you
Comment #14
phenaproximaComment #15
phenaproximaLooks pretty good to me!
Comment #17
tedbow🎉