-
-
Notifications
You must be signed in to change notification settings - Fork 86
Avoid change chmod by using Nette\Utils\FileSystem with pass null on 3rd argument on write #209
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…3rd argument on write
@vasilvestre could you verify this patch? Thank you. |
@@ -4,6 +4,7 @@ | |||
|
|||
namespace Symplify\EasyCodingStandard\Caching\ValueObject\Storage; | |||
|
|||
use Nette\Utils\FileSystem as UtilsFileSystem; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vasilvestre for manual test, with directly change on vendor, change it with prefixed use statement:
use ECSPrefix202405\Nette\Utils\FileSystem as UtilsFileSystem;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will try soon
It's not working for me, the ownership still changes to root:root despite an original user with UID 1000 |
I am not sure why it can change ownership, chaning chmod is what we seen in the past, but ownership is strange as it should not know what new one, it probably related with somewhere on your php setting and executable as user you run. I suggest to not run on docker as it may be the root of the issue. I will may still merge it to sync with rector. |
@TomasVotruba I am merging it so we have sync usage with rector ;) |
Not an option in our workflow. Thank you for the test tho ! |
This patch should equal with what Rector has:
and avoid symfony issue on rename.