Pb Battlemap

Forum pour parler du site AideDD et de sa communauté (présentation, etc)
User avatar
Puncho
Niv 17 - Pulvérisateur de liches
Niv 17 - Pulvérisateur de liches
Posts: 1558
Joined: Mon Jul 14, 2014 11:31 pm
Location: Québec, Canada

Re: Pb Battlemap

Post by Puncho » Wed Nov 28, 2018 9:21 pm

En supposant que Blueace a effectivement fait cette correction avant ton passage, que doit-on faire pour retrouver la mise en forme des bloc-notes dans les BM ?

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

Re: Pb Battlemap

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

Le comportement actuel me laisse plutôt penser que nous sommes encore en présence du tout premier patch d'urgence de blueace.
(Qui consistait probablement à mettre en commentaire le code défaillant sans se poser plus de question dans un premier temps.)

J'ai bon espoir qu'en faisant la correction proposée tout rentre dans l'ordre aussitôt. Dans le cas contraire, je reste dans le coin. :)

PS pour les geek:
D'ailleurs j'aurais du écrire

Code: Select all

$message_parser = new parse_message($string);
dès le départ, même avec l'ancien code de PHPBB.
Pour des raisons de compatibilité ascendante avec PHP 3 et 4, si PHP ne peut pas trouver une fonction __construct() pour une classe donnée, il cherchera une fonction constructeur représentée, comme dans l'ancien style, par le nom de la classe

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

Re: Pb Battlemap

Post by blueace » Thu Nov 29, 2018 2:26 am

Puncho wrote:
Wed Nov 28, 2018 9:21 pm
En supposant que Blueace a effectivement fait cette correction avant ton passage...
Je n'avais encore rien fait. Par contre maintenant ça y est, c'est fait. Et ça semble marcher.

Mais bon, en cas de pb, Florimel est pas loin, donc profitez-en !

User avatar
Neb
Niv 16 - Tueur de balors
Niv 16 - Tueur de balors
Posts: 1068
Joined: Mon Apr 17, 2017 8:33 am
Location: Normandie, France

Re: Pb Battlemap

Post by Neb » Thu Nov 29, 2018 11:09 am

Merci à vous tous qui avez contribué à solutionner ce problème fort gênant! C'est agréable de disposer d'une communauté serviable et réactive! :ok:

User avatar
dvins
Niv 18 - Démembreur de krakens
Niv 18 - Démembreur de krakens
Posts: 2317
Joined: Thu Feb 02, 2017 4:29 pm
Location: San Francisco, USA

Re: Pb Battlemap

Post by dvins » Thu Nov 29, 2018 3:58 pm

Florimel wrote:
Wed Nov 28, 2018 9:32 pm
blueace wrote:
Thu Nov 29, 2018 2:26 am
Merci à tous les deux pour votre aide et votre réactivité ! :kiss:

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

Re: Pb Battlemap

Post by Johan » Wed May 15, 2019 11:37 am

Florimel wrote:
Wed Nov 28, 2018 9:32 pm
blueace wrote:
Thu Nov 29, 2018 2:26 am


Florimel, j'espère que tu reçois les notifs du forum. Sinon, Blueace, peux-tu le contacter s'il te plait.

Une des battlemaps de la partie 9 ne fonctionne plus et présente le même que celui présenté en novembre dernier :
Battlemap 193 : Voilà la map incriminée
:mad:
Mode édition RAS
Mode consultation affiche une map vide et la console renvoi une erreur 500.

Cela ressemble fort au problème de parse_message de la dernière fois et se produit au moment où Blueace a mis à jour le php du forum.
La chose étrange est que la dernière fois, le problème était généralisée, tandis que seule cette map (à ma connaissance) est concernée cette fois ci :bof: :prof:

Pouvez-vous m'aider ? :god:

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

Re: Pb Battlemap

Post by Florimel » Thu Jun 20, 2019 11:27 pm

En effet, ça ressemble bien à une exception dans « battlemap/get.php »
Mais sans l'aide de blueace, je vais avoir du mal à deboguer, car je n'ai pas la main sur le forum...
Idéalement, faudrait ajouter des traces , un « try/catch » etc. dans le php pour comprendre ce qu'il se passe.
J'ai essayé d'importer ta BM sur donjonsetdragons.net où j'aurais été autonome pour débugger, mais malheureusement je ne reproduis pas le problème là-bas http://donjonsetdragons.net/battlemap/?id=673
On peut supposer que c'est le décodage des bbcode des champs notes de la BM qui déconne.
Tu peux essayer de virer les balises et toute présence de crochets dans tes champs notes, pour essayer de cerner le problème ?

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

Re: Pb Battlemap

Post by Johan » Fri Jun 21, 2019 8:55 am

Bonjour Florimel

Merci d'être venu pour regarder ça.
J'ai fait le tour complet des notes pour virer les balises bbcode et les crochets : ça n'a pas marché.
J'en ai profiter pour nettoyer les quelques objets qui avaient des liens d'image morts, ce n'était pas ça non plus.

Mais comme tu as suggéré que le contenu des notes pouvait être la source du bug, j'ai creusé la question et j'ai fini par trouvé !
Il y a un bon moment, suite à une erreur de frappe, j'avais renseigné les notes générales (les notes quand aucun objet n'est sélectionné) avec un zéro. A l'époque, cela ne gênait en rien et je n'avais pas réussi à le supprimer, donc j'ai laissé comme ça.
En remplaçant le 0 par autre chose (au hasard, toto :mad: ), ça remarche ! :party: :party:
(J'ai remplacé toto par une phrase bateau au final, l'important est de virer le 0)

Donc je pense que le 0, depuis la dernière MAJ du php du forum, créait une exception qui faisait planter le code de récupération des données.
Ca explique aussi pourquoi mes différents tests de copies de contenu sur d'autres BM et ton importation ne reproduisaient pas le bug : c'était sur la BM en elle-même et non sur son contenu.


Merci beaucoup de ton aide. :)
Et merci à Blueace aussi d'avoir prévenu Florimel.

User avatar
grainedetoile
Niv 15 - Tourmenteur de sphinx
Niv 15 - Tourmenteur de sphinx
Posts: 873
Joined: Sun Apr 29, 2018 3:02 pm

Re: Pb Battlemap

Post by grainedetoile » Fri Jun 21, 2019 9:07 am

Yesssss! Bravo à tous les deux ! On va pouvoir avoir notre Battlemap. :D

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

Re: Pb Battlemap

Post by Florimel » Fri Jun 21, 2019 7:47 pm

Reste à comprendre pourquoi ça plante avec 0 et à corriger le bug...

Alors, blueace, si tu as le temps à l'occasion, je te propose la modif suivante (dans « /battlemap/get.php »).
  • Un petit test en plus sur la présence d'un « [ », histoire de ne pas décoder inutilement (le cas du « 0 » isolé étant un cas particulier)
  • et un « try/catch » qui évitera de planter et aidera au debug si un problème du même ordre se présente à nouveau

Code: Select all

// Decode bbcode
function bb_decode(&$string)
{
  if (!isset($string)) return;
  if (strpos($string, '[') === false) return; 
  try {
    $message_parser = new parse_message($string);
    $message_parser->parse(true, true, true);
    $string = $message_parser->format_display(true, true, true, false);
  } catch (Exception $e) {
    header('Warning: ' . $e->getMessage());
  }
}

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

Re: Pb Battlemap

Post by Florimel » Fri Jun 21, 2019 8:02 pm

Johan wrote:
Fri Jun 21, 2019 8:55 am
je n'avais pas réussi à le supprimer
Et voici la correction du bug qui empêche de vider complètement le champ « notes » de la BM.
Cette fois, ça se passe dans « /battlemap/post.php »

Code: Select all

...
if (!isset($row['notes'])) $row['notes'] = '';   // <<-- ajouter cette ligne
$id = $row['battlemap_id'];
unset($row['battlemap_id']);
...

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

Re: Pb Battlemap

Post by blueace » Sun Jun 23, 2019 5:42 pm

Merci Florimel.
Les 2 modifs ont été faites.