Fixed in
http://trac.symfony-project.org/changeset/22869

Could you open a ticket for such ting? I just spend 5 minutes
searching in trac, because I thought there was one :)

Fabian

On Tue, Sep 29, 2009 at 2:24 PM, Bert-Jan <[email protected]> wrote:
> Hi Folks,
>
> The password reminder e-mail doesn't arrive for me so I couldn't make a
> ticket..
>
> I've found a small bug in sfWidgetFormInputFileEditable.
> If you pass in is_image=true the img tag doesn't have an alt attribute
> which breaks html compliance. Adding it through the render() method
> didn't work because of this tiny bug. It breaks the array_merge.
>
> Have a nice day,
>
> Bert-Jan
>
> >
>
> Index: 
> dcFrontendPlugin/lib/vendor/symfony/lib/widget/sfWidgetFormInputFileEditable.class.php
> ===================================================================
> --- 
> dcFrontendPlugin/lib/vendor/symfony/lib/widget/sfWidgetFormInputFileEditable.class.php
>       (revision 22596)
> +++ 
> dcFrontendPlugin/lib/vendor/symfony/lib/widget/sfWidgetFormInputFileEditable.class.php
>       (working copy)
> @@ -99,7 +99,7 @@
>   {
>     if ($this->getOption('is_image'))
>     {
> -      return false !== $this->getOption('file_src') ? 
> $this->renderTag('img', array_merge(array('src' => 
> $this->getOption('file_src'))), $attributes) : '';
> +      return false !== $this->getOption('file_src') ? 
> $this->renderTag('img', array_merge(array('src' => 
> $this->getOption('file_src')), $attributes)) : '';
>     }
>     else
>     {
>
>

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"symfony users" 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-users?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to