Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error media.php line 197 #373

Closed
mediaiberia opened this issue Sep 12, 2012 · 2 comments
Closed

Error media.php line 197 #373

mediaiberia opened this issue Sep 12, 2012 · 2 comments

Comments

@mediaiberia
Copy link

Error en algunas pantallas

PHP Notice: Array to string conversion in ../libraries/cms/form/field/media.php on line 197

Esto se debe a que se está reutilizando el nombre de la variable $attr como array cuando ya estaba definida como string en la línea 167 de media.php.

        $attr = array(
            'id' => $this->id . '_preview',
            'class' => 'media-preview',
            'style' => $style,
        );
        $img = JHtml::image($src, JText::_('JLIB_FORM_MEDIA_PREVIEW_ALT'), $attr);

Solución: Cambiar esta variable en este punto por otro nombre distinto, por ejemplo $aAttr
$aAttr = array(
'id' => $this->id . 'preview',
'class' => 'media-preview',
'style' => $style,
);
$img = JHtml::image($src, JText::
('JLIB_FORM_MEDIA_PREVIEW_ALT'), $aAttr);

@realityking
Copy link
Contributor

Thanks for the report. This has been fixed in master.

I could deduce the meaning of your report because I had already fixed the bug but in the future please keep reports in english - that way more developers will be able to read the report and you have a higher chance of the bug being fixed.

@javigomez
Copy link
Contributor

Thanks @realityking for you effort in understanding spanish ^_^

Gracias @angelbupr por reportar el error. Cómo dice Rouven, en github habría que hacer los reportes en inglés para que otros programadores de todo el mundo puedan entender los reportes. En cualquier caso gracias de nuevo por tu excelente aportación, y si tienes otras aportaciones contacta conmigo ( en twitter @javi_gomez ) y te ayudo a traducir el texto.

dneukirchen added a commit to dneukirchen/joomla-cms that referenced this issue Sep 28, 2017
* Make select all reactive

* Refactoring
laoneo pushed a commit that referenced this issue Apr 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants