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
<victor.berc...@gmail.com>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 <o...@theprone.com> 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 symfony-devs@googlegroups.com
> To unsubscribe from this group, send email to
> symfony-devs+unsubscr...@googlegroups.com
> 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 symfony-devs@googlegroups.com
To unsubscribe from this group, send email to
symfony-devs+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/symfony-devs?hl=en

Reply via email to