Add emptyAsRoot option to PropertyDirectoryNamer about vichuploaderbundle HOT 5 CLOSED

Could you provide an example?

from vichuploaderbundle.

sure:
Let's say the value of the property is either null or ''. At the moment the namer will throw an exception since it checks for empty($name).
That new configuration option would make the namer not throw an exception in cases like that but behave like if there was no directory namer (returning '').

Use case would be (at least for me) a fallback for old entities that were around before directory naming was enabled.

from vichuploaderbundle.

That seems a poor solution to me: how can you distinguish between the empty value returned for that reason (no naming enabled yet) and an empty value returned for some other reason?
By the way, I guess you can solve your use case by listening to the exception.

from vichuploaderbundle.

I mean, there's no empty value being returned by that ever at the moment. Because if it's empty it throws the exception.

from vichuploaderbundle.

The listener solution seems still suitable.
Another possible solution is a script that fills that empty values with something meaningful.

All in all, it looks like this new option would solve only an edge case.

from vichuploaderbundle.