Jump to content

Spoiler post upgrade


Recommended Posts

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

  • Like 2
Link to post
Share on other sites
  • 1 month later...

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

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

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

2016-05-22_200127.png

2016-05-22_200155.png

with fix on my test in localhost nested spoielr show corretcly

2016-05-22_200517.png

2016-05-22_200551.png

2016-05-22_200540.png

 

code diff of post before and after fix

2016-05-22_200838.png2016-05-22_200914.png2016-05-22_200929.png2016-05-22_200949.png

 

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

 

  • Like 1
Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. to insert a cookie message.