- last update
about 1 year ago 21 pass - Issue created by @daou
- Status changed to Needs work
7 months ago 9:20pm 13 May 2024 The patch changes other things besides adding a try-catch. It removed the
prepareDirectory
function from the if check, which shouldn't have been changed. It doesn't throw an exception on failure; it returns false.- Merge request !19Fix checks for dir create and file move and add try/catch. → (Open) created by solideogloria
- last update
7 months ago 10 pass, 3 fail - Status changed to Needs review
7 months ago 10:58pm 13 May 2024 - last update
7 months ago 12 pass, 2 fail The test failures appear to be unrelated. I can't figure out what is causing them.
- last update
7 months ago 12 pass, 2 fail I found significant bugs in the original
filefield_paths_filefield_paths_process_file()
code.It gets
$temporary_scheme_name
and$source_scheme_name
incorrectly, causing the file move/rename not to happen at all in some cases.This is fixed by using
StreamWrapperManager::getScheme()
- last update
7 months ago 21 pass - last update
7 months ago 12 pass, 2 fail - last update
7 months ago 12 pass, 2 fail 24:08 20:40 Running- last update
7 months ago 21 pass - last update
7 months ago 12 pass, 2 fail - last update
7 months ago 12 fail - last update
7 months ago 12 fail - last update
7 months ago 12 pass, 2 fail