[RISOLTO]Validazione AccessKey

[PiccolaStellainCielo]

Prima di tutto complimenti per questo progetto. L'ho trovato molto interessante.

L'ho installato su un sito e fino a quando avevo voci di menu non annidate la validazione era tutta ok.
Da quando ho aggiunto sottovoci alle voci di menu, mi è spuntato un nuovo errore di validazione in quanto attribuisce l'accesskey, oltre che al link, anche all'ul:

Codice: [Seleziona]

<a href="#" accesskey="G"><span>Menu 1</span></a>
<ul accesskey="G">
<li class="item13"><a href="#" accesskey="S"><span>Sottomenu 1</span></a></li>
....

Qualcuno ha notato lo stesso problema? Come posso risolverlo?
Grazie anticipatamente.

[elpaso66]

Prima di tutto complimenti per questo progetto. L'ho trovato molto interessante.

L'ho installato su un sito e fino a quando avevo voci di menu non annidate la validazione era tutta ok.
Da quando ho aggiunto sottovoci alle voci di menu, mi è spuntato un nuovo errore di validazione in quanto attribuisce l'accesskey, oltre che al link, anche all'ul:

Codice: [Seleziona]

<a href="#" accesskey="G"><span>Menu 1</span></a>
<ul accesskey="G">
<li class="item13"><a href="#" accesskey="S"><span>Sottomenu 1</span></a></li>
....

Qualcuno ha notato lo stesso problema? Come posso risolverlo?
Grazie anticipatamente.

È evidentemente un baco.
Se sai programmare lo puoi risolvere correggendo il baco nel componente delle access key. Non ti dimenticare di postare la versione corretta nel tracker del progetto.

Altrimenti non ti rimane che segnalare il baco sul tracker del progetto e aspettare che qualcuno lo corregga.

La soluzione temporanea è di non associare access key ai sottomenu.

[PiccolaStellainCielo]

Purtroppo non è temporanea, perchè pur non associando access key ai sottomenu, nel codice mi viene scritto il tag vuoto, che comunque mi genera errore...

Provo a segnalarlo, anche se non so se sono in grado 

Grazie comunque

[PiccolaStellainCielo]

Alla fine, vista l'urgenza, ci ho studiato sopra un bel po e ho risolto, ma non so come fare per condividerlo nel traker (scusate ma di ste cose ci capisco poco...)

[elpaso66]

Alla fine, vista l'urgenza, ci ho studiato sopra un bel po e ho risolto, ma non so come fare per condividerlo nel traker (scusate ma di ste cose ci capisco poco...)

Ottimo!
Incolla o allega il codice modificato alla segnalazione

http://joomlacode.org/gf/project/joomlafap1_5/tracker/?action=TrackerItemEdit&tracker_id=6709&tracker_item_id=20542

[PiccolaStellainCielo]

Ok... ho modificato la segnalazione.

Lo metto anche qui, dovesse servire:

Ho modificato il file accesskey_helper.php (che è dentro a html/mod_mainmenu del template accessibile) alla riga 96 dove dice:

if(isset($accesskey){

ho esteso il controllo in questo modo:

if((isset($accesskey)) && ($c->name()!="ul")){

Così non viene scritto negli ul e il codice è validato.

Spero di essere stata d'aiuto

[elpaso66]

Ok... ho modificato la segnalazione.

Lo metto anche qui, dovesse servire:

Ho modificato il file accesskey_helper.php (che è dentro a html/mod_mainmenu del template accessibile) alla riga 96 dove dice:

if(isset($accesskey){

ho esteso il controllo in questo modo:

if((isset($accesskey)) && ($c->name()!="ul")){

Così non viene scritto negli ul e il codice è validato.

Spero di essere stata d'aiuto

Si, ho visto, ma il problema che hai segnalato sembra essere già stato risolto da parecchio tempo, stai usando una versione vecchia del template "accessible".

La prossima volta, prima di segnalare un problema, ti prego di accertarti che la versione che usi sia l'ultima disponibile in SVN.

[PiccolaStellainCielo]

Ho l'ultima versione e il problema è proprio lì!

[elpaso66]

Ho l'ultima versione e il problema è proprio lì!

alla riga 96 della versione corrente c'è :

Codice: [Seleziona]

       
        //ABP: accesskey
        if ($node->name() == 'li'){
            foreach ($node->children() as   $c){
                // ABP: FAP access key
                if(isset($accesskey)){
                    $c->addAttribute('accesskey', $accesskey);
                }
            }
        }
 

[PiccolaStellainCielo]

Appunto, la riga va bene, solo va sostituita con

        //ABP: accesskey
        if ($node->name() == 'li'){
            foreach ($node->children() as   $c){
                // ABP: FAP access key
                if((isset($accesskey)) && ($c->name()!="ul")){
                    $c->addAttribute('accesskey', $accesskey);
                }
            }
        }

e non da più l'errore di validazione indacato in precedenza

[elpaso66]

Ok, applicato e aggiornato SVN

Ciao

$ svn ci -m"Fix #20542, grazie a Nicoletta Celotti"
Trasmetto      mod_mainmenu/accesskey_helper.php
Trasmissione dati .
Commit della Revisione 112 eseguito.

[PiccolaStellainCielo]

Contenta di essere stata di aiuto 
Ciao