• User

    Ciao Pikadilly, avevo dato un'occhiata al tuo sito pensando di rispondere con calma poi mi sono dimenticato. Anche se hai tolto il componente diversi errori di codifica di mxcomment me li ricordo.

    1. Javascript inseriti nella pagina in modo un po' troppo disinvolto 🙂
      Se trovi nel codice il punto dove viene fatto l'output dei javascript modifica così

    <script language="JavaScript" type="text/JavaScript">
    <![CDATA[

    lasci in mezzo tutto com'è poi prima del tag di chiusura metti

    ]]>
    </script>

    1. In molte url usa come separatore dei parametri & invece di &
      Anche queste vanno cercate nel codice e modificate

    2. ci sono alcuni attributi html con lettere maiuscole, mi pare di ricordare onMouseOver onMouseClick. Vanno tutti trasformati in minuscolo -> onmouseover onmouseclick

    Non conosco il componente per cui non so dirti dove cercare esattamente. In genere l'output del codice viene fatto in file con doppia estensione .html.php almeno se hanno fatto le cose standard.

    Per lo spam forse c'è l'opzione per approvare i commenti prima della pubblicazione.

    Se hai un'installazione locale e vuoi provare a fare intanto queste correzioni il numero degli errori di validazione dovrebbe ridursi parecchio. Se posti quelli che restano si possono provare a correggere.

    Posta se hai problemi.


  • Super User

    Intanto ti ringrazio per le dritte.
    Sì, è vero, c'è il controllo per lo spam però alla fine ho sommato tutti i problemi e ho deciso che ne avevo abbastanza.
    Io avevo corretto il problema delle & proprio perchè non me le accettava così e mi consigliava di usare & però usando questa cosa non ti dico che cose oscene venivano fuori. 😄
    Comunque ho un altro sito prova proprio on line...quindi proverò lì tutto quello che mi hai detto, se funziona io ti faccio una statua.
    Grazie mille, davvero 😄
    Ti faccio sapere 🙂


  • Super User

    Vedi che le persone competenti si trovano sempre ! 😄
    Grazie gmassi 😉


  • Super User

    Eh già, vedo vedo e menomale che ci sono...che ci siete 😄


  • Super User

    Ok, responso.
    Ho modificato tutto quello che sono riuscita a trovare testando di volta in volta il funzionamento, ho lasciato le impostazioni di default.
    Adesso mi segnala 23 errori che non riesco a modificare, tra cui uno incomprensibile che riguarderebbe il mio foglio di stile che nell'altro sito non da problemi.
    Gmassi aveva detto di sostituire le & con &, ma ho dovuto fare il contrario e infatti gli la maggior parte degli errori è sparita (ho seguito proprio il consiglio del validatore che mi ha rimandata a questa pagina:
    http://www.htmlhelp.com/tools/validator/problems.html#amp ).
    Testato e approvato il funzionamento mi sono fermata perche non riesco a capire gli altri errori dove sono nascosti. Di sicuro sono nel componente poichè disistallandolo la validazione passa tranquillamente.
    Questa è la pagina testata:

    http://www.pikadilly.it/jomprova/ultime/benvenuto-in-joomla.html

    Ho tolto tutti i riferimenti a ditte e altro perchè non vuole essere autopromozione 😄

    Questo è il risultato del validatore:

    http://validator.w3.org/check?uri=http%3A%2F%2Fwww.pikadilly.it%2Fjomprova%2Fultime%2Fbenvenuto-in-joomla-6.html&charset=(detect+automatically)&doctype=Inline&group=0&verbose=1

    E questo è il componente zippato compreso di mambot che ho sistemato fino ad ora:
    http://www.pikadilly.it/provafilmatoforum/mxcomment_pul.zip

    Mancano solo 23 errori...solo 23 e poi...woooow...ma è meglio stare zitti 😄


  • Super User

    Hai interpretato male l'intervento di gmassi:

    1. In molte (http://www.giorgiotave.it/forum/../wikigt/os/URL) usa come separatore dei parametri & invece di &
      Anche queste vanno cercate nel codice e modificate

    in molte url usa [ (il soggetto è sottinteso) mxcomment ], quindi, di default...
    Quindi, rivolgendosi a te, ti diceva di fare l'esatto contrario di quello che hai interpretato 😉


  • Super User

    Ah è vero, scusa Gmassi e grazie Globo...allora aveva ragione lui,ecco 😄
    Alle volte per leggere veloce mi mangio le parole...sorry...


  • User

    Ho guardato il file zip con le tue modifiche, queste sono altre cose da modificare

    file maxcomment.class.php

    Riga 103 ho messo uno spazio prima di style

    DEFINE("_OPACITY"," style="filter:alpha(opacity=" . $mxc_opacityeffectpercent . "); -moz-opacity:". $mxc_opacityeffectpercent2 .";" onmouseover="javascript:mxclightup(this,100);" onmouseout="javascript:mxclightup(this," . $mxc_opacityeffectpercent . ");"");

    Riga 439. Va aggiunto type="text/javascript"

    $_MXC->SHOWFORM .= "<script type="text/javascript">\n


    templates/expand/full.php

    Righe 61 e 62. Va tolto align="top" prima di valign

    <td width="12" valign="top">   </td>
    <td width="82" valign="top">


    templates/expand/usercomment.php

    Riga 50
    Ogni commento è racchiuso da <div id="usercomment">. Il problema è che non si può avere più di un elemento con lo stesso id per cui hai un errore tutte le volte che una pagina contiene due o più commenti. Non saprei che correzione fare. Puoi provare a mettere <div class="usercomment"> però poi devi anche aggiustare il css.

    Per ora sono arrivato fino a qui. Prova queste modifiche anche se di sicuro qualche altro errore ci sarà ancora.


  • Super User

    Ok, stasera lo modifico e posto il responso.
    Sì infatti bisogna cambiare in class, solo che poi succede che spaiscono le righe intorno alla form sia chiusa che aperta e si spostano delle cose tipo il titolo, bisognerebbe riscrivere un pezzo di css, provo a farlo 😄
    Grazie ancora 😄 Io alcuni errori nel .class non li avevo visti proprio 😞
    😄


  • Super User

    Ed eccomi.
    Ho pulito e sistemato quello che mi è stato detto da gmassi.
    In più ho apportato le modifiche necessarie al foglio di stile per far sì che non uscisse l'errore del <div id="usercomment"> cambiandolo in class e modificando i bordi direttamente nella pagina usercomment.php, assieme all'img per la valutazione utenti che si spostava in malo modo insieme al titolo.
    Ho spostato tutto a destra.
    Adesso il mozpoacity non funza ancora, ma dovremmo riuscire a sistemarlo o usando i css e non javascript o non so..:D
    Adesso gli errori sono quattro di cui uno super peloso perchè riguarda la toolbar della formattazione del testo nei commenti, più precisamente il problema è questo style presente mxcomment.class.php, lo stesso style che gmassi mi ha detto di modificare aggiungendo type=text/css, quell'errore è sparito ma adesso giustamente mi dice che style non può stare all'interno di body, ergo si dovrebbe usare uno stile non interno, ma il vecchio modo con le tabelle e la formattazione senza stile.:(
    Poi è rimasto l'errore al link che non ho capito, un table non chiuso e questo:
    <input type="hidden" name="rating" value="0" />
    che non so modificare, dice che c'è il tag imput non può stare lì :?.
    Comunque adesso va molto meglio di prima, erano 125 ora solo 4, ma per ora solo in quella pagina e solo la versione expand. Nella versione default segnala solo due errori inerenti ai link dei css...:D
    Vediamo un pò, potete trovare le ultime modifiche nel file zip dell'altro post.
    🙂


  • User

    Sono rimasti pochi errori, ma tosti 🙂
    Il blocco style giustamente devi toglierlo da body, forse potresti rimuoverlo dal sorgente del componente, copiare il contenuto in un file .css a parte e richiamare questo css nella parte <head> del template. Così ti finirà su tutte le pagine del sito, ma a questo si può trovare una soluzione.

    L'errore su </table> è perché la tabella è aperta e chiusa senza nessun <tr> e relativi <td> dentro. Però il punto dove viene creato questo html non lo trovo, va visto con calma.

    Anche per l'errore sul tag <input> è difficile trovare dove questo html è generato nel codice. Ci guardo meglio più tardi.


  • Super User

    Ok, intanto provo a fare quello che hai detto 🙂
    Grazie mille 😄


  • User

    Dunque queste dovrebbero essere le ultime modifiche

    File maxcomment.html.php la riga 420 diventa

    echo "<tr><td><input type="hidden" name="rating" value="0" /></td></tr>";

    Poi ti allego il file /templates/expand/intro.php perché le modifiche erano troppe. Spero funzioni perché non ho potuto provarlo. Considera che se cambi template di mxcomment gli errori te li ritrovi tutti 🙂
    Però le modifiche dovrebbero poter essere riportate anche sui files di bubbles acid e default.

    Resterebbe quel tag style da togliere da body.

    [ATTACH]1005[/ATTACH]


  • Super User

    Scusa il ritardo della risposta.
    Fatto quello che dicevi, ma niente quei quattro errori persistono...:x

    🙂


  • User

    Proviamo intanto a togliere l'errore del campo input, forse ti ho fatto modificare la riga sbagliata

    Nel file maxcomment.class.php

    la riga 412

    $_MXC->SHOWFORM .= "<input type="hidden" name="rating" value="0" />\n";

    va modificata in

    $_MXC->SHOWFORM .= "<tr><td><input type="hidden" name="rating" value="0" /></td></tr>\n";


  • User

    Forse ho trovato anche l'altro in templates/expand/full.php da riga 108 a 114

    [php]<table width="100%" border="0" cellpadding="0" cellspacing="0">
    <?php if ( $_MXC->KEYWORDS ) { ?>
    <tr>
    <td><p class="keywordstitle"><?php echo _MXC_TPL_KEYWORDS ?> : <span class="keywords"><?php echo $_MXC->KEYWORDS ?></span></p></td>
    </tr>
    <?php } ?>
    </table>[/php]Vanno modificate in
    [php]<?php if ( $_MXC->KEYWORDS ) { ?>
    <table width="100%" border="0" cellpadding="0" cellspacing="0">
    <tr>
    <td><p class="keywordstitle"><?php echo _MXC_TPL_KEYWORDS ?> : <span class="keywords"><?php echo $_MXC->KEYWORDS ?></span></p></td>
    </tr>
    </table>
    <?php } ?>[/php]Spero sia questo perché se no non saprei dove cercare.


  • Super User

    Siamo scesi a 3 :D:D
    La fine di table non si trova...bua.
    Sto componente vuole fare il pidocchioso, Sgrunt.


  • User

    Mi sono installato il componente perché senza provarlo direttamente non c'era modo di ripulirlo.

    Sono rimasti due errori per il file expand_css.css e un tag <style> tutti e due messi in body dove non possono stare. Questi non ho trovato il modo di correggerli. Ti conviene tagliarli via dal sorgente e incollare il contenuto in un file .css separato o appenderlo alla fine di template_css.css del tuo template.

    Ho visto all'ultimo anche un errore di id duplicato sulla home page (fullarticle), credo tu possa correggerlo facilmente come quello sulla pagina dell'articolo usando class al posto di id e cambiando il css

    Ho corretto solo il template expand gli altri di sicuro hanno errori.

    Qui non mi fa allegare il pacchetto completo (troppo grande) lo trovi a
    http://www.mediafire.com/?3u4okeywnu4

    Se non vuoi reinstallare il tutto puoi sovrascrivere tutti i file, le modifiche non te le elenco una per una perché sono troppe e sparse.


  • Super User

    Ohi ciao, grazie, ci stai mettendo tanto impegno...farò come dici e ti faccio sapere, grazie mille dell'aiuto 😉


  • Super User

    Ti faccio sapere che ancora non ho trovato soluzione per quei due errori di style...o meglio sto seguendo i tuoi consigli e e pian piano sicuro ci riesco...speriamo di essere più efficiente domani...:)