GDRCD 5.4 - Errore quando invio messaggio privato multiplo

GDRCD 5.4 - Errore quando invio messaggio privato multiplo postato il 01/07/2017 12:32:30 nel forum programmazione, gdrcd, open source, hosting

Buongiorno,

Ho riscontrato questo errore quando su gdrcd nella messaggistica privata seleziono "destinatario multiplo"

GDRCD MySQLi Error [File: messages_center.inc.php; Line: 59]
Error Code: 1064
Error String: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

Error Detail: INSERT INTO messaggi (moderazione, mittente, destinatario, spedito, testo, ambito) VALUE

Questa parte del codice invero non l'avevo mai toccata e quindi ho fatto questa scopera solamente ora.

La riga 59 nel codice è la seguente:

gdrcd_query($query);

Ci sta solo questo scritto nella riga 59.

Ora vi elenco di seguito il continuo dopo la riga 59 perché ci sta questo commento:

/** * Bugfix: commentato la stampa della variabile $query. In caso di messaggio multiplo stampava
* l'ultima query eseguita.
* @author Rhllor
*/

Non so a che serve questa scelta destinatari multipli, ma spero che siano le ML di gilda (spero) perché per inviare messaggi multipli dice che basta mettere delle virgole nel destinatario singolo.

Crossout ↗
Tuffati nel mondo di Crossout, il gioco MMO d'azione post apocalittico! Costruisci le tue speciali macchine da combattimento!

quota

Pagine → 1 2 3

seralia

SCRIVI

01/07/2017 19:08:12

gdrcd_query($query);

Come vedi la riga 59 processa una Query che è contenuta in una variabile, la variabile $query.
Puoi cercare, nelle righe immediatamente sopra, come è valorizzata la variabile $query ?
Prché è nella variabile che è dettagliata la query che ti da errore.

Storie di Agarthi ↗
Un Varco si apre davanti a te, un mondo tra i mondi è a portata di mano. Lasciati alle spalle le certezze, inizia l'avventura!

quota

fargus

SCRIVI

01/07/2017 19:54:51

seralia ha scritto:

gdrcd_query($query);

Allora le righe che ci sono sopra eseguono l'inserimento dei messaggi nel database. Ci sono tutte le tipologie, quelle per il destinatario singolo, quelle per tutti gli utenti presenti e quelle per tutti gli utenti registrati.

Di seguito tutta la parte di codice sopra la riga 59.

/*Inserimento nuovo messaggio nel db*/
if (gdrcd_filter('get',$_POST['op'])=="send_message")
{
if (gdrcd_filter('get',$_POST['multipli'])=='singolo')
{
$check_dest=explode(',',gdrcd_filter('get',$_POST['destinatario']));
$destinat=trim($check_dest[0]);
if (gdrcd_filter('get',$_POST['url'])!="")
{
$_POST['testo'] = $_SESSION['login'].' ti ha segnalato questo [url='.$_POST['url'].']link[/url].';
}//if

$result=gdrcd_query("SELECT nome FROM personaggio WHERE nome = '".$destinat."'", 'result');
if ((gdrcd_query($result, 'num_rows')>0)&&(empty($destinat)===FALSE))
{
gdrcd_query("INSERT INTO messaggi (moderazione, mittente, destinatario, spedito, testo, ambito) VALUES ('".gdrcd_filter('in',$_POST['mod'])."', '".$_SESSION['login']."', '".gdrcd_capital_letter(gdrcd_filter('in',$destinat))."', NOW(), '".gdrcd_filter('in',$_POST['testo'])."', '".gdrcd_filter('in',$_POST['ambito'])."')");

gdrcd_query("INSERT INTO backmessaggi (moderazione, mittente, destinatario, spedito, testo, ambito) VALUES ('".gdrcd_filter('in',$_POST['mod'])."', '".$_SESSION['login']."', '".gdrcd_capital_letter(gdrcd_filter('in',$destinat))."', NOW(), '".gdrcd_filter('in',$_POST['testo'])."', '".gdrcd_filter('in',$_POST['ambito'])."')");
}//if
}
else if ($_POST['multipli']=='presenti')
{
$query = "SELECT personaggio.nome, personaggio.cognome, personaggio.permessi, personaggio.sesso, personaggio.id_razza, razza.sing_m, razza.sing_f, razza.icon, personaggio.disponibile, personaggio.online_status, personaggio.is_invisible, personaggio.ultima_mappa, personaggio.ultimo_luogo, personaggio.posizione, personaggio.ora_entrata, personaggio.ora_uscita, personaggio.ultimo_refresh, mappa.stanza_apparente, mappa.nome as luogo, mappa_click.nome as mappa FROM personaggio LEFT JOIN mappa ON personaggio.ultimo_luogo = mappa.id LEFT JOIN mappa_click ON personaggio.ultima_mappa = mappa_click.id_click LEFT JOIN razza ON personaggio.id_razza = razza.id_razza WHERE personaggio.ora_entrata > personaggio.ora_uscita AND DATE_ADD(personaggio.ultimo_refresh, INTERVAL 4 MINUTE) > NOW() ORDER BY personaggio.is_invisible, personaggio.ultima_mappa, personaggio.ultimo_luogo, personaggio.nome";
$result = gdrcd_query($query, 'result');

while ($record = gdrcd_query($result, 'fetch'))
{
gdrcd_query("INSERT INTO messaggi (moderazione, mittente, destinatario, spedito, testo, ambito) VALUES ('".gdrcd_filter('in',$_POST['mod'])."', '".$_SESSION['login']."', '".$record['nome']."', NOW(), '".gdrcd_filter('in',$_POST['testo'])."', '".gdrcd_filter('in',$_POST['ambito'])."')");
}
}
else if ($_POST['multipli']=='multiplo')
{
$check_dest=explode(',',$_POST['destinatario']);
$query="INSERT INTO messaggi (moderazione, mittente, destinatario, spedito, testo, ambito) VALUES";
foreach ($check_dest as $destinat)
{
$destinat=trim($destinat);

$result=gdrcd_query("SELECT nome FROM personaggio WHERE nome = '".gdrcd_filter('in',$destinat)."'", 'result');
if (gdrcd_query($result, 'num_rows')>0)
{
$query.=" ('".$_SESSION['login']."', '".gdrcd_capital_letter(gdrcd_filter('in',$destinat))."', NOW(), '".gdrcd_filter('in',$_POST['testo'])."'),";
}
}
$query=substr($query,0,-1);
gdrcd_query($query);

Ho lasciato visibile la riga 59 come ultima righe del contenuto della finestra di cui sopra.

Che devo fare ora?

Sponsorizzati con noi! ↗
Sponsorizza il tuo gioco su GDR-online! Trova nuovi giocatori fra i migliaia di appassionati che ogni giorno visitano il portale!

quota

rematore

SCRIVI

02/07/2017 09:34:27 e modificato da rematore il 02/07/2017 09:36:56

Li inserisci i nomi dei personaggi separati dalla virgola?

L'opzione serve ad inviare lo stesso messaggio a più utenti

https://pixpet.net/u/doItAgain ↗

quota

seralia

SCRIVI

02/07/2017 10:40:59

rematore ha scritto: Li inserisci i nomi dei personaggi separati dalla virgola?

Credo parli del messaggio multiplo attivo dai capi gilda in su... Appena ho un attimo guardo come e' la pagina di GDRcd...
Tu hai già controllato fargus? Quella parte di codice e' identica alla versione e base di GDRcd?

The Coven ↗
Negli scorci di luce delle paludi della Louisiana fioriscono nuove ombre. Estend e la Congrega ti aspettano per vivere emozionanti avventure!

quota

fargus

SCRIVI

02/07/2017 11:06:50

seralia ha scritto:

rematore ha scritto: Li inserisci i nomi dei personaggi separati dalla virgola?

si è identica, ho solo aggiunto nel database due colonne ma quelle funzionano e la funzione di "destinatario multiplo" non l'ho mai toccata. ho scoperto l'errore solo perché volevo capire a cosa servisse.

Crossout ↗
Tuffati nel mondo di Crossout, il gioco MMO d'azione post apocalittico! Costruisci le tue speciali macchine da combattimento!

quota

seralia

SCRIVI

04/07/2017 19:40:07

else if ($_POST['multipli']=='multiplo')
{
$check_dest=explode(',',$_POST['destinatario']);
$query="INSERT INTO messaggi (moderazione, mittente, destinatario, spedito, testo, ambito) VALUES";
foreach ($check_dest as $destinat)
{
$destinat=trim($destinat);

$result=gdrcd_query("SELECT nome FROM personaggio WHERE nome = '".gdrcd_filter('in',$destinat)."'", 'result');
if (gdrcd_query($result, 'num_rows')>0)
{
$query.=" ('".$_SESSION['login']."', '".gdrcd_capital_letter(gdrcd_filter('in',$destinat))."', NOW(), '".gdrcd_filter('in',$_POST['testo'])."'),";
}
}

Hai aggiunto il campo moderazione nella query, ma non il relativo valore...
La variabile $query si compone di due parti: nella prima hai aggiunto

INSERT INTO messaggi (moderazione, mittente, destinatario, spedito, testo, ambito)

ma nella seconda hai lasciato come primo VALUE il mittente

$query.=" ('".$_SESSION['login']."',

senza aggiungere il valore per moderazione...

Entropia Universe ↗
Lascia che il tuo avatar esplori nuovi mondi e viaggi tra i pianeti in questo stupendo MmoRpg Sci-Fi Free to Play!

quota

fargus

SCRIVI

05/07/2017 09:51:35 e modificato da fargus il 05/07/2017 10:02:45

seralia ha scritto:

Hai aggiunto il campo moderazione nella query, ma non il relativo valore...
La variabile $query si compone di due parti: nella prima hai aggiunto

INSERT INTO messaggi (moderazione, mittente, destinatario, spedito, testo, ambito)

ma nella seconda hai lasciato come primo VALUE il mittente

$query.=" ('".$_SESSION['login']."',

senza aggiungere il valore per moderazione...

Ok hai ragione, il fatto è che non sapevo andasse messo li visto che la query è diversa dalle altre.

quindi l'ho aggiunto ed ora mi da un problema di virgolette.

GDRCD MySQLi Error [File: messages_center.inc.php; Line: 59]
Error Code: 1064
Error String: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near " at line 1

Error Detail: INSERT INTO messaggi (moderazione, mittente, destinatario, spedito, testo, ambito) VALUE.

Ho provato a cercare se per errore avessi messo delle virgolette di troppo o se invece ne mancavano, ma sembrano tutte aprirsi e chiudersi... Ho provato a rimuovere dalla query moderazione e ambito, lasciando quindi il codice originale di gdrcd e l'errore me lo da ugualmente. Presumo quindi che questo errore di virgolette " sia nativo.

else if ($_POST['multipli']=='multiplo')
{
$check_dest=explode(',',$_POST['destinatario']);
$query="INSERT INTO messaggi (moderazione, mittente, destinatario, spedito, testo, ambito) VALUES";
foreach ($check_dest as $destinat)
{
$destinat=trim($destinat);

$result=gdrcd_query("SELECT nome FROM personaggio WHERE nome = '".gdrcd_filter('in',$destinat)."'", 'result');
if (gdrcd_query($result, 'num_rows')>0)
{
$query.=" ('".gdrcd_filter('in',$_POST['mod'])."', '".$_SESSION['login']."', '".gdrcd_capital_letter(gdrcd_filter('in',$destinat))."', NOW(), '".gdrcd_filter('in',$_POST['testo'])."', '".gdrcd_filter('in',$_POST['ambito'])."', ),";
}
}
$query=substr($query,0,-1);
gdrcd_query($query);

Che il problema sia qui?

'".gdrcd_capital_letter(gdrcd_filter('in',$destinat))."'

Non mi sembra anche se credo che sia qui perché in tutte le altre query la frase gdrcd_capital_letter non ci sta, quindi essendo nativo il problema mi chiedo se magari manca qualcosa.

Sea of Conquest ↗
Vivi un'avventura tra i mari! Dal cuore del Mare del Diavolo, un paradiso piratesco brulicante di magia, tesori e avventure, salperai verso l'ignoto!

quota

seralia

SCRIVI

05/07/2017 20:13:36

Il problema non credo sia nativo, visto che non mi risultavano problemi in altri GDRCD installati...
L'errore 1064 indica un problema di sintassi della query sql.
La variabile $_POST['mod'] è valorizzata?

War Thunder ↗
Aviazione, veicoli corazzati e flotte militari della seconda guerra mondiale. Parteciperai a tutte le principali battaglie!

quota

fargus

SCRIVI

05/07/2017 20:39:07

seralia ha scritto: Il problema non credo sia nativo, visto che non mi risultavano problemi in altri GDRCD installati...
L'errore 1064 indica un problema di sintassi della query sql.
La variabile $_POST['mod'] è valorizzata?

Si, per tutti gli altri funziona.

destinatario singolo,

tutti i presenti
tutti gli utenti il messaggio viene inviato correttamente, l'errore è presente solo nel "destinatario multiplo".

e come ti ho detto, se provo a rimuovere tutte le mie modifiche, mi da il problema delle virgolette.

Poco sopra credo di averti messo tutto il codice della pagina mi sa...

World of Warship ↗
MMO gratuito con frenetiche battaglie navali ambientate nel ventesimo secolo. Salpa con la tua nave ed affronta i nemici!

quota

fargus

SCRIVI

07/07/2017 16:42:21

Seralia, altrimenti se a te funziona quella parte, perché non mi mostri la porzione di quel codice affinché possa vedere come è diversa dal mio ed apportare la correzione?