GDCD 5.1 Iframe Messaggi e problema dado

Pagine → 1 2

30/12/2011 13:21:09

Seguendo il flusso del lancio di dado in ref_header.inc.php per vedere quale case attiva e vedere dov'e' che somma il dato non necessario per eliminarlo.

Homo Faber Fortunae Suae

quota

zenida

SCRIVI

30/12/2011 21:26:26 e modificato da zenida il 31/12/2011 00:43:31

mr_faber ha scritto: Seguendo il flusso del lancio di dado in ref_header.inc.php per vedere quale case attiva e vedere dov'e' che somma il dato non necessario per eliminarlo.

Ho tentato di capire cosa lanciasse il dado quando il sistema skill è OFF ma dado è ON.

Ma purtroppo uno dei grandi problemi di questo codice è anche la scarsa formattazione del testo, mancano le opportune tabulazioni e rientri di testo, quindi non è facile seguire il codice specie se non commentato.

Senza considerare che è già difficile di per se comprendere un codice scritto da un altro.

Qualcuno sa gentilmente dirmi come risolvere questo bug in GDRCD 5.1?

Per la cronaca ecco un esempio di cosa compare:

21:19Mr.X usa : 10, dado 4, totale 14

Il dieci che vedete in grassetto sottolineato è appunto l'anomalia. Tra l'altro viene indicato un valore senza specificare cosa sia (la caratteristica appunto)

il problema è che poi questo valore viene anche aggiunto al totale del dado dando un risultato falsato.

EDIT: Se viene messo su ON il sistema delle skill il lancio del dado avviene in maniera perfetta.
Cioè io non seleziono nessuna abilità ne altro, ma lancio solo il dado, e voilà il risultato appare ovviamente corretto.

--------------------------------------------------------------------------------
EDIT 2: RISOLTO😆
alla riga 70 del ref_header.inc.php

} else if (gdrcd_filter('get', $_POST['id_stats']) != 'no_stats' && gdrcd_filter('get',$_POST['dice']) != 'no_dice')

come potete notare (leggendo il resto del codice) anche se il sistema abilità è off e solo i dadi sono ON, non si capisce per quale strano motivo secondo lui tramite il metodo POST c'è anche un parametro per id_stats e quindi entra in questa sezione di codice dedicata solo ai tiri su caratteristica.
è bastato aggiungere un controllo poco prima

if ($PARAMETERS['mode']['skillsystem']=='ON')

con un else subito dopo

else
{
mt_srand((double)microtime()*1000000);
$die=mt_rand(1,gdrcd_filter('num',$_POST['dice']));

gdrcd_query("INSERT INTO chat ( stanza, imgs, mittente, destinatario, ora, tipo, testo ) VALUES (".$_SESSION['luogo'].", '".$_SESSION['sesso'].";".$_SESSION['img_razza']."', '".$_SESSION['login']."', '', NOW(), 'D', '".$_SESSION['login'].' '.gdrcd_filter('in',$MESSAGE['chat']['commands']['die']['cast']).gdrcd_filter('num',$_POST['dice']).': '.gdrcd_filter('in',$MESSAGE['chat']['commands']['die']['sum']).' '.gdrcd_filter('num',$die)."')");
}

in maniera tale che vi entri solo se il sistema delle skill è on altrimenti entra nel successivo else che usa solo il tiro di dado, funzionando alla perfezione.

Questo diciamo che però è un bypass del problema, xkè bisogna capire come mai viene passato cmq un paramentro via POST di stats quando questo non esiste.