Pb Battlemap

Forum pour parler du site AideDD et de sa communauté (présentation, etc)
User avatar
Johan
Niv 20 - Démolisseur de tarrasques
Niv 20 - Démolisseur de tarrasques
Posts: 3402
Joined: Mon Dec 29, 2014 4:12 pm

Pb Battlemap

Post by Johan » Mon Nov 26, 2018 7:38 am

Bonjour BlueAce,

Il y a un bug dans les parties qui est apparu ce week-end, probablement suite à la mise à jour.

Les battlemaps ne sont pas visibles en consultation. Si on va sur une battlemap en suivant le lien classique
Battlemap 193 : Exemple : Battlemap de la partie 09
Ca donne uniquement une grille vide.
Mais si le DM créateur de la map passe en mode édition, il peut voir la map normalement avec tout son contenu.

Arnaud a aussi constaté la même chose sur la battlemap de l'arène qu'il dirige en ce moment (invisible en mode consultation, visible en mode édition).

User avatar
Arnaud
Niv 20 - Démolisseur de tarrasques
Niv 20 - Démolisseur de tarrasques
Posts: 5178
Joined: Sun Jul 13, 2014 10:09 am
Location: Val-de-Marne, France

Re: Maintenance du forum

Post by Arnaud » Mon Nov 26, 2018 12:41 pm

yep, je confirme (et j'en profite pour suivre ce sujet) : en édition, je n'ai rien perdu (ouf !) mais en mode normal (lien accessible aux joueurs) rien ne s'affiche.
Battlemap 398 : exemple avec la BM de l'arène

User avatar
blueace
Admin
Admin
Posts: 1786
Joined: Fri Feb 21, 2014 1:41 am
Location: Mexico DF

Re: Maintenance du forum

Post by blueace » Tue Nov 27, 2018 2:12 am

Alors, pour les battlemaps, comme cela été dit pas le passé, le script n'est pas de moi.
J'ai envoyé un mail à @Florimel, l'auteur des scripts. Je vous tiens au courant.

User avatar
dvins
Niv 16 - Tueur de balors
Niv 16 - Tueur de balors
Posts: 1443
Joined: Thu Feb 02, 2017 4:29 pm
Location: San Francisco, USA

Re: Maintenance du forum

Post by dvins » Tue Nov 27, 2018 4:15 am

blueace wrote:
Tue Nov 27, 2018 2:12 am
Alors, pour les battlemaps, comme cela été dit pas le passé, le script n'est pas de moi.
J'ai envoyé un mail à @Florimel, l'auteur des scripts. Je vous tiens au courant.
Merci !

Il semble que le script n'ai pas changé mais qu'il y ait un petit souci côté serveur.

En mode édition (qui s'affiche correctement), on récupère les infos en faisant la requête suivante:
battlemap/get.php?id=383&nocache=1543291823331

En mode lecture (qui s'affiche "vide"), on récupère les infos en faisant la requête suivante:
battlemap/get.php?id=383&nocache=1543291823331&decode=1

Cette dernière renvoie une erreur 500 (erreur serveur) et le script n'affiche donc pas la battlemap.

Y a-t-il eu des changements côté serveur récemment sur cet URL ? Peut-être que la MAJ du forum a changé quelque chose ?

User avatar
blueace
Admin
Admin
Posts: 1786
Joined: Fri Feb 21, 2014 1:41 am
Location: Mexico DF

Re: Maintenance du forum

Post by blueace » Tue Nov 27, 2018 1:42 pm

Est-ce que qq1 peut essayer de nouveau ?

Je préviens, il va surement y avoir des problèmes d'accent et de BBcode, mais c'est que du texte. L'appli devrait marcher.

C'est pas définitif, juste en attendant que le maitre Florimel nous règle cela correctement.

User avatar
Puncho
Niv 16 - Tueur de balors
Niv 16 - Tueur de balors
Posts: 1465
Joined: Mon Jul 14, 2014 11:31 pm
Location: Québec, Canada

Re: Maintenance du forum

Post by Puncho » Tue Nov 27, 2018 1:47 pm

À première vue, c'est rentré dans l'ordre. Les éditeurs de BM nous diront si leurs nouvelles éditions se sont passés sans anicroche.

User avatar
blueace
Admin
Admin
Posts: 1786
Joined: Fri Feb 21, 2014 1:41 am
Location: Mexico DF

Re: Maintenance du forum

Post by blueace » Tue Nov 27, 2018 1:51 pm

Cool. Alors si qq1 (Dvins ???) peut m'aider à solutionner ça à la racine, le pb vient du fait que le code de la battlemap fait un appel comme ceci

Code: Select all

$message_parser = new parse_message();
$message_parser->parse_message($string);
Or le code PhpBB vient de passer de

Code: Select all

class parse_message extends bbcode_firstpass
{
[...]
       function parse_message($message = '')
       {	 	    
               // Init BBCode UID	 	           
               $this->bbcode_uid = substr(base_convert(unique_id(), 16, 36), 0, BBCODE_UID_LEN);
               $this->message = $message;	 	      
       }
[...]
à

Code: Select all

class parse_message extends bbcode_firstpass
{
[...]
       function __construct($message = '')
       {	 	    
               // Init BBCode UID	 	           
               $this->bbcode_uid = substr(base_convert(unique_id(), 16, 36), 0, BBCODE_UID_LEN);
               $this->message = $message;	 	      
       }
[...]
Bref, il n'y a plus de fonction parse_message, c'est remplacé par le construct.

Faut-il maintenant faire

Code: Select all

 $message_parser = new parse_message($string);
???

User avatar
blueace
Admin
Admin
Posts: 1786
Joined: Fri Feb 21, 2014 1:41 am
Location: Mexico DF

Re: Maintenance du forum

Post by blueace » Tue Nov 27, 2018 2:02 pm

blueace wrote:
Tue Nov 27, 2018 1:51 pm
Bref, il n'y a plus de fonction parse_message, c'est remplacé par le construct
Je sens que cette phrase va donner l'envie à certains de mettre des créatures artificielles dans leur prochain scénar... :evil:

User avatar
Johan
Niv 20 - Démolisseur de tarrasques
Niv 20 - Démolisseur de tarrasques
Posts: 3402
Joined: Mon Dec 29, 2014 4:12 pm

Re: Pb Battlemap

Post by Johan » Tue Nov 27, 2018 2:44 pm

J'arrive à visionner les battlemaps et j'ai pu tester l'édition d'une des miennes sans problème particulier.

Je n'ai pas repéré de bugs d'accents, par contre, le BBcode ne marche pas (j'avais une balise italique sur une description et le code s'affiche au lieu de s'exécuter).
Mais bon, si c'est juste le BBcode, ça va :)


Merci pour le patch rapide en attendant une résolution finale de Florimel :god:



Pour la fonction parse_message :
Je ne suis pas expert de php, mais je dirais que ta solution est la bonne.

Je suis rouillé en php (en tout cas, je ne l'utilisais pas en orienté objet), donc je ne me rappelle plus trop des normes, mais je suis surpris qu'il n'y ait pas de setter, que ce soit dans la classe message_parser ou dans sa classe mère bbcode_firstpass, de ce que j'ai pu voir en téléchargeant la nouvelle version du de PhpBB.
Après, j'imagine qu'il n'y a pas besoin d'injecter une valeur à $message en dehors du moment de création de l'objet, ce qui rend le setter inutile.

User avatar
Johan
Niv 20 - Démolisseur de tarrasques
Niv 20 - Démolisseur de tarrasques
Posts: 3402
Joined: Mon Dec 29, 2014 4:12 pm

Re: Pb Battlemap

Post by Johan » Tue Nov 27, 2018 2:50 pm

Désolé du double-post :
J'ajoute que la mise en page des descriptions (retour à la ligne) saute aussi. Ca fait parti des éléments qui doivent passer dans le parser de la même manière que les balises BBcode si je ne me trompe pas, donc c'est logique que ça ne marche pas alors.

User avatar
dvins
Niv 16 - Tueur de balors
Niv 16 - Tueur de balors
Posts: 1443
Joined: Thu Feb 02, 2017 4:29 pm
Location: San Francisco, USA

Re: Maintenance du forum

Post by dvins » Wed Nov 28, 2018 3:52 pm

blueace wrote:
Tue Nov 27, 2018 1:51 pm
Faut-il maintenant faire

Code: Select all

 $message_parser = new parse_message($string);
???
Oui ça me paraît être la bonne solution !

User avatar
Florimel
Niv 2 - Renifleur de gnolls
Niv 2 - Renifleur de gnolls
Posts: 5
Joined: Thu Mar 27, 2014 9:30 pm

Re: Maintenance du forum

Post by Florimel » Wed Nov 28, 2018 9:19 pm

Salut à tous !

Merci à blueace et Puncho de m'avoir prévenu. Ceci dit, j'ai l'impression d'arriver un peu après la bataille ... Vous avez fait tout le boulot d'analyse et même identifé la correction.

De ce que je comprends, je dirais effectivement qu'il suffit de remplacer :

Code: Select all

$message_parser = new parse_message();
$message_parser->parse_message($string);
par

Code: Select all

 $message_parser = new parse_message($string);
pour corriger le problème.
Last edited by Florimel on Wed Nov 28, 2018 9:34 pm, edited 1 time in total.