Zend\Filter\File\RenameUpload behaves differently when folder exists and not about laminas-filter HOT 5 OPEN

@svycka I've checked it and I believe it is desired behaviour. As target we can provide the directory or the file name and in script we can't recognise what you have on mind. So:

• if provided string is a directory (so must exist) then the file will be moved into that directory,
• otherwise, path does not exist, so script assumes that it should be a file

Originally posted by @michalbundyra at zendframework/zend-filter#4 (comment)

from laminas-filter.

@webimpress I do not agree with you

if provided string is a directory (so must exist) then the file will be moved into that directory,

sounds ok.

otherwise, path does not exist, so script assumes that it should be a file

this is not documented so I am not sure how you decided this is the desired behavior. Don't understand how having two hidden behaviors from one option is good. maybe new file_prefix or something would be better.

Originally posted by @svycka at zendframework/zend-filter#4 (comment)

from laminas-filter.

@svycka Ok, reopening. Feel free to submit PR - either: documentation update of propose a fix.

Originally posted by @michalbundyra at zendframework/zend-filter#4 (comment)

from laminas-filter.

fixing would mean BC break? if this is the desired behavior

Originally posted by @svycka at zendframework/zend-filter#4 (comment)

from laminas-filter.

@svycka If you propose fix which will be BC Break we will plan it for v3.

I've described what is the current behaviour, and, yes, I can see it can be a bit tricky, but I think it is logical. There is not anything like 'file prefix' right now, the provided target is either directory "where file should be moved" or the new file name. As you are using randomize=true there is randomized string added to the name.

Originally posted by @michalbundyra at zendframework/zend-filter#4 (comment)

from laminas-filter.