Spoiler post upgrade

[BomAle 420]

Ho eseguito con la collaborazione di @Daniel un upgrade di un database per conto di un privato, abbiamo effettuato alcuni controlli sui bbcode impostazioni e configurazioni necessarie per il corretto funzionamento del software.

Purtroppo ci siamo imbattuti in un bug che riguarda il bbcode Spoiler, molto utilizzato nelle precedenti release, infatti se provo a incollare il codice in formato bbcode:

[spoiler]liv1
----[spoiler]liv1.1[/spoiler]
----[spoiler]liv1.2[/spoiler]
----[spoiler]liv1.3[/spoiler]
----[spoiler]liv1.4[/spoiler]
[/spoiler]

e riportarlo qui sotto avremo un risultato inaspettato:

liv1

----
----
----
----

liv1.1

liv1.2

liv1.3

liv1.4

Gli spoiler finiscono in fondo senza rispettare l'ordine di scrittura.

Purtroppo non vedo come si evolverà la situazione, spero correggano almeno il parser dei bbcode e di avere un immediato feedback

vi ringrazio per l'attenzione, chiunque sta considerando di fare un upgrade consiglio di aspettare fino a quando la situazione sarà effettivamente stabile

[Amy Pond 40]

...

...

...

*esplode* 

[Daniel 61]

 il merito è di @BomAle ma scrivo io. IPS ha finalmente riconosciuto questo bug ed hanno annunciato che lo fixeranno nella 4.1.2

Ale ha comunque preparato un fix che ho testato e funziona, ma comunque consiglio di aspettare in attesa di un fix ufficiale

https://invisionpower.com/4bugtrack/active-reports/419-spoiler-freezingcrashing-browser-in-some-situations-r11019/

di seguito riporto il messaggio.

--------------------------------------------------------------------

same issues, i tested made with my old forum db's in localhost. BEFORE upgrade modify this line

\IPS\Text\LegacyParser, line 342.

$value = str_replace( "[spoiler]", '</p><blockquote class="ipsStyle_spoiler" data-ipsSpoiler><p>', $value );
$value = str_replace( "[/spoiler]", "</p></blockquote><p>", $value );

in

$value = str_replace( "[spoiler]", '</p><div class="ipsSpoiler" data-ipsSpoiler><div class="ipsSpoiler_header ipsSpoiler_closed"><a href="#" data-action="toggleSpoiler">&nbsp;</a><span></span></div><div class="ipsSpoiler_contents"><p>', $value );
$value = str_replace( "[/spoiler]", "</p></div></div><p>", $value );

in my original forum (updated when 4.1.9 was out) nested spoiler show like this

with fix on my test in localhost nested spoielr show corretcly

 

code diff of post before and after fix

 

speriamo che ci sia un fix anche per chi ha già aggiornato e non solo per chi aggiorna DOPO il fix..