[Bug] BBCode center introduce <br> indesiderati

[hashcat]

In questo caso la seguente demo vale più di mille parole: LINK.
Notare cosa accade alla firma nei messaggi della discussione successivi al primo...



P.S.: Di seguito riporto la mia attuale firma (come impostata tramite PCU):

Codice: Seleziona tutto

[center][i][b]<<The quieter you become, the more you can hear>>[/b][/i]

[i][b]<<Mangiate merda, milioni di mosche non possono sbagliare>>[/b][/i][/center]

[The Doctor]

Scende sempre di una riga

[hashcat]

Scende sempre di una riga

Esatto, ad ogni messaggio viene aggiunto un <br> (line break) sopra e sotto.

[Zane]

Uhmm, hai modo di capire se è un problema che si verifica solo su TLI oppure è di phpBB?

Nel primo caso bisogna indagare sulle cause (in questo momento l'unica idea che ho riguarda in tag "center" che abbiamo creato noi, dato che con la mia firma sembra non farlo), mentre nel secondo bisogna segnalare a phpBB.

[hashcat]

Uhmm, hai modo di capire se è un problema che si verifica solo su TLI oppure è di phpBB?

Effettivamente il problema si verifica con una firma del genere:

Codice: Seleziona tutto

[center][i][b]<<The quieter you become, the more you can hear>>[/b][/i]

[i][b]<<Mangiate merda, milioni di mosche non possono sbagliare>>[/b][/i][/center]

Ma non con questa (l'ho cambiata):

Codice: Seleziona tutto

[center][i][b]<<The quieter you become, the more you can hear>>

<<Mangiate merda, milioni di mosche non possono sbagliare>>[/b][/i][/center]

L'unica differenza è che nel primo caso i tag "i" e "b" vengono aperti e chiusi per entrambe le citazioni:

Codice: Seleziona tutto

[center]
    [i]
        [b]
            "Citazione 1"
        [/b]
    [/i]
    "Riga vuota"
    [i]
        [b]
            "Citazione 2"
        [/b]
    [/i]
[/center]

Nella seconda firma (divenuta ora la mia attuale) i tag "i" e "b" vengono aperti e chiusi solo una volta (ed includono la riga vuota):

Codice: Seleziona tutto

[center]
    [i]
        [b]
            "Citazione 1"
            "Riga vuota"
            "Citazione 2"
        [/b]
    [/i]
[/center]

[hashcat]

Ciò che accade a livello di HTML utilizzando la prima firma è che, per ogni mio messaggio successivo al primo nella stessa discussione, viene aggiunto un <br> tra il <div align="center"> ed il primo <span style="font-style: italic"> ed un altro tra l'ultimo </span> ed il </div>.

Codice: Seleziona tutto

<div align="center">
    <br>
    <span style="font-style: italic"><span style="font-weight: bold"><<The quieter you become, the more you can hear>></span></span>
    <br>
    <br>
    <span style="font-style: italic"><span style="font-weight: bold"><<Mangiate merda, milioni di mosche non possono sbagliare>></span></span>
    <br>
</div>

[The Doctor]

A questo punto la domanda che mi viene è: "Bug or Feature"??

[hashcat]

Uhmm, hai modo di capire se è un problema che si verifica solo su TLI oppure è di phpBB?

A lume di naso direi che si tratta di un bug relativo al tag center. Con una firma del genere phpBB 3 si comporta bene:

Codice: Seleziona tutto

[u][i][b]<<The quieter you become, the more you can hear>>[/b][/i]

[i][b]<<Mangiate merda, milioni di mosche non possono sbagliare>>[/b][/i][/u]

L'unica differenza rispetto a quella problematica è la sostituzione del tag center con un altro tag di formattazione (in questo caso u).

[Zane]

L'attuale codice per il tag center è stato inserito (v. documentazione phpBB) così:

Codice: Seleziona tutto

BBCode usage: [center]{TEXT}[/center]
HTML replacement: <div align="center">{TEXT}</div>

Se avete proposte per il fix sono tutt'orecchi.

[PippoDJ]

Ciao Zane,

Se avete proposte per il fix sono tutt'orecchi.

il problema non è solo nel tag center, ma in tutti i tag BBCode che generano un tag DIV quando vengono convertiti in HTML.

Sembra che l'interpretazione del BBCode venga applicata ricorsivamente ogni volta che la firma (dello stesso utente) appare nella pagina del topic:

prima istanza della firma:

Codice: Seleziona tutto

<div id="sig44365" class="signature">
    <wrapper>
        <wrapper>
            <div class="codebox">
                prova
            </div>
            <br>
            <div class="codebox">
                prova
            </div>
        </wrapper>
    </wrapper>
</div>

seconda istanza della firma:

Codice: Seleziona tutto

<div id="sig44395" class="signature">
    <wrapper>
        <wrapper>
            <wrapper>
                <wrapper>
                    <div class="codebox">
                        <br>
                        <br>
                        prova
                        <br>
                    </div>
                    <br>
                    <br>
                    <div class="codebox">
                        <br>
                        <br>
                        prova
                        <br>
                    </div>
                </wrapper>
            </wrapper>
        </wrapper>
    </wrapper>
</div>

ecc. ecc.

Questo parsing ricorsivo, oltre ad inserire i BR, aggiunge anche un paio di tag WRAPPER ad ogni ricorsione. Indipendentemente dai DIV contenuti nella firma. [a proposito: cosa diavolo è un tag WRAPPER? ]

A occhio (a meno che non ci siano estensioni che intervengono nel rendering della firma) sembrerebbe proprio un errore nel codice di phpBB: probabilmente passato inosservato perché si nota solo in situazioni molto particolari.

[Zane]

Grazie PippoDJ per l'analisi. Non mi risulta che ci sia nulla in termini di estensioni che possa generare il problema. Se hai modo di aprire un bug ai dev magari ce lo sistemano.

Elemento wrapper mai sentito nemmeno io!

[PippoDJ]

Se hai modo di aprire un bug ai dev magari ce lo sistemano.

C'è già: "PHPBB3-14530: Signature parsing inconsistant" e queste sono le fix: viewtopic.php, acp_users.php e ucp_profile.php.
Magari se ci guardavo subito...

Elemento wrapper mai sentito nemmeno io!

Questo rimane un mistero.