For Joomla 3.x don’t use document.id(‘xxx’)

Dimitris

Navegador Iniciado

Posts: 2

10 años 1 mes antes #444 por Dimitris

For Joomla 3.x don’t use document.id(‘xxx’) Publicado por Dimitris

Joomla 3.x is supporting IE8 and newer. Using this legacy mootools code is useless. For example the following code:

<script type="text/javascript">
    js = jQuery.noConflict();
    js(document).ready(function() {
        
    });
    Joomla.submitbutton = function(task)
    {
        if (task == 'basic.cancel') {
            Joomla.submitform(task, document.getElementById('basic-form'));
        }
        else {
            
            if (task != 'basic.cancel' && document.formvalidator.isValid(document.id('basic-form'))) {
                
                Joomla.submitform(task, document.getElementById('basic-form'));
            }
            else {
                alert('<?php echo $this->escape(JText::_('JGLOBAL_VALIDATION_FORM_FAILED')); ?>');
            }
        }
    }
</script>

Can be:

if (task != 'basic.cancel' && document.formvalidator.isValid(document.getElementById('basic-form'))) {

Without any compromise.

Recently I propose removing these legacy mootools methods in Joomla PR

Thanks Dimitris

Por favor, Identificarse o Crear cuenta para unirse a la conversación.

Andres Maeso

Navegador Platino

Posts: 338

10 años 1 mes antes #446 por Andres Maeso

Respuesta de Andres Maeso sobre el tema For Joomla 3.x don’t use document.id(‘xxx’)

Hey there!

Thanks for your feedback, actually it's a good idea, but as you probably know we stick to Joomla Standards, so we can't remove them until they're removed from Joomla Core. But please, do not hesitate to come back to us when they accept the PR

Regards,

Andrés.

Andrés Maeso
Customer relations manager at Joomla Component Creator.

Esta dirección de correo electrónico está siendo protegida contra los robots de spam. Necesita tener JavaScript habilitado para poder verlo.

Por favor, Identificarse o Crear cuenta para unirse a la conversación.

Dimitris

Navegador Iniciado

Posts: 2

9 años 10 meses antes #582 por Dimitris

Respuesta de Dimitris sobre el tema For Joomla 3.x don’t use document.id(‘xxx’)

FYI Joomla 3.4 is scheduled for mid February and has those changes. Also There is a JTHML::(behavior.formvalidator); instead of the JTHML::(behavior.formvalidation); which uses only jQuery.

Por favor, Identificarse o Crear cuenta para unirse a la conversación.

Andres Maeso

Navegador Platino

Posts: 338

9 años 10 meses antes #583 por Andres Maeso

Respuesta de Andres Maeso sobre el tema For Joomla 3.x don’t use document.id(‘xxx’)

Thank you Dimitris, duly noted. I'll pass it to our developers.

Por favor, Identificarse o Crear cuenta para unirse a la conversación.

Tiempo de carga de la página: 0.058 segundos

Gracias a Foro Kunena