Well if you say so.. but imho something smells, it needs to be either
overriden, or a getClientExtension method should be provided and documented
accordingly

On Sat, Jun 18, 2011 at 12:03 PM, victor berchet
<[email protected]>wrote:

> It works as expected.
>
> All the methods from the File class returns information about the
> underlying file (getBasename(), getPath(), ...),
> so should getExtension() do.
>
> As a side note, getExtension() is part of \SplFileInfo as of PHP 5.3.6
>
> On Jun 17, 10:26 am, Omar Ayoub-Salloum <[email protected]> wrote:
> > getExtension in UploadedFile should be overriden to:
> >
> >     public function getExtension()
> >     {
> >         return pathinfo($this->getClientOriginalName(),
> PATHINFO_EXTENSION);
> >     }
> >
> > The current implementation always returns an empty string because it
> refers
> > to the temporary name generated by php.
>
> --
> If you want to report a vulnerability issue on symfony, please send it to
> security at symfony-project.com
>
> You received this message because you are subscribed to the Google
> Groups "symfony developers" group.
> To post to this group, send email to [email protected]
> To unsubscribe from this group, send email to
> [email protected]
> For more options, visit this group at
> http://groups.google.com/group/symfony-devs?hl=en
>

-- 
If you want to report a vulnerability issue on symfony, please send it to 
security at symfony-project.com

You received this message because you are subscribed to the Google
Groups "symfony developers" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/symfony-devs?hl=en

Reply via email to