?
avatar echterhoff
echterhoff
28 May 2021

Hi there,

every here and then some things go wrong. A message about some undeletable file is great but actually worth nothing, if you don't know where to look for that specific problem.
It would be great if that delete method could provide also the path to look for the specific file.

Since this file gets rewritten during a core file refresh, changing it by myself is no option.

throw new FilesystemException(__METHOD__ . ': Failed deleting inaccessible file ' . $filename);

Thanks
Lars

avatar echterhoff echterhoff - open - 28 May 2021
avatar joomla-cms-bot joomla-cms-bot - change - 28 May 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 28 May 2021
avatar brianteeman
brianteeman - comment - 28 May 2021

So instead of changing it yourself and having it rewritten why dont you submit your changes as pull request here

avatar echterhoff
echterhoff - comment - 28 May 2021

Hi Brian,

simply because I have no clue how to do this in a fast, economic way other than forking, changing and request you guys to pull my changes. (I know that's the reason for Github to exist.)
My thought on this small thing was more like: Well, maybe someone with it's hands already on the code might add it and...

I will add it and send you a pull request later that day. My fault.

Have a nice day
Lars

avatar brianteeman
brianteeman - comment - 28 May 2021

I wouldnt know what the code is that you want to change to. But if its just in a single file then you can find the file here on github, click on the pencil icon in the top right and make your changes following the onscreen instructions.

avatar echterhoff echterhoff - change - 28 May 2021
Status New Closed
Closed_Date 0000-00-00 00:00:00 2021-05-28 12:25:45
Closed_By echterhoff
avatar echterhoff echterhoff - close - 28 May 2021

Add a Comment

Login with GitHub to post a comment