Ilia,

While your opinion that we shouldn't count on the "filename"
attribute in multiparted queries is quite to the point,
your patch totally disregards the old behaviour.

We must neither use php_basename() for pathes of a spec
different from that defined by the platform which is determined
in compile time. That most likely leads to inconsistent behaviour
between platforms.

In addition, you had removed the portion Rui added in the past
that allowed multibyte-encoded filenames to get parsed properly
without any notice. Actually the function in question doesn't
work at all with such filenames at this momemnt.

Due to those issues, I'm going to revert your patch and recommit
less radical one.

Regards,
Moriyoshi

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php



Reply via email to