Vai al contenuto

Spoiler post upgrade

Featured Replies

Inviato

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

Inviato

...

...

...

*esplode* 

  • 1 mese dopo...
Inviato

 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
 

 

Per favore accedi per lasciare un commento

Potrai lasciare un commento dopo aver effettuato l'accesso

Accedi Ora

Chi sta navigando 0

  • Nessun utente registrato visualizza questa pagina.

Informazioni importanti

Abbiamo inserito dei cookies nel tuo dispositivo per aiutarti a migliorare la tua esperienza su questo sito. Puoi modificare le impostazioni dei cookie, altrimenti puoi accettarli cliccando su continua. to insert a cookie message.

Configure browser push notifications

Chrome (Android)
  1. Tap the lock icon next to the address bar.
  2. Tap Permissions → Notifications.
  3. Adjust your preference.
Chrome (Desktop)
  1. Click the padlock icon in the address bar.
  2. Select Site settings.
  3. Find Notifications and adjust your preference.