[RISOLTO] GDRCD 5.5.1 Tre "piccoli" problemi

alejandra

SCRIVI

[RISOLTO] GDRCD 5.5.1 Tre "piccoli" problemi postato il 26/06/2021 10:48:46 nel forum programmazione, gdrcd, open source, hosting e modificato da alejandra il 01/07/2021 09:14:33

Salve, premetto che non sono una programmatrice e avrei bisogno di direttive chiare come se fossi tonta.

Problema 1 ) Errore al Cambio Nome:
Parse error: syntax error, unexpected '}' in /membri/ostrov/pages/user_cambio_nome.inc.php on line 71

Sono andata a vedere la riga che mi dice ma non so cosa non vada ^^"

71 <?php
72 if($_SESSION['permessi'] == SUPERUSER) {



Problema 2) da gestore, provo a cancellare un account ma non mi considera minimamente nonostante il sito mi dica "modifica eseguita" il pg rimane tranquillamente anche in anagrafe.

Problema 3) Una volta iscritto un personaggio nuovo questo non riesce ad effettuare il login ne con la password mandata ne con il recupero password dando "personaggio o password non riconusciuti"

Grazie a tuttǝ

Ale 🎲

quota

Pagine → 1 

kasa

SCRIVI

MODERATORE

26/06/2021 11:34:25

alejandra ha scritto: ...


> Problema cambio nome
Prova a correggere la linea di codice prima di quella evidenziata, che dovrebbe essere cosi:

<?php} else {


Mettila così:

<?php } else {


La mancanza dello spazio, non codifica bene la chiusura della parentesi e quindi manda in errore la pagina.

Sarebbe da aggiornare su GDRCD, come BUG.

> Problema cambio nome

Per quanto riguarda la pagina user_cancella_pg.inc.php, purtroppo ho individuato diverse problematiche di base. I MODERATOR è giusto che disabilitino gli account, senza cancellarli, mentre i SUPERUSER dovrebbero proprio cancellarli. Ho applicato una modifica che allinei con questa logica.

Devi sostituire questo:

*Cancella altri*/
if((gdrcd_filter('get', $_POST['op']) == 'force') && ($_SESSION['permessi'] == MODERATOR)) {
gdrcd_query("UPDATE personaggio SET permessi = -1 WHERE nome = '".gdrcd_filter('in', $_POST['account'])."' AND permessi < ".SUPERUSER."");
gdrcd_query("INSERT INTO log (nome_interessato, autore, data_evento, codice_evento, descrizione_evento) VALUES ('".gdrcd_filter('in', $_POST['account'])."','".$_SESSION['login']."', NOW(), ".DELETEPG." ,'".gdrcd_filter('in', $MESSAGE['interface']['user']['delete']['deleted'])."')");
?>
<div class="warning">
<?php echo gdrcd_filter('out', $MESSAGE['warning']['modified']); ?>
</div>
<div class="link_back">
<a href="main.php?page=user_cancella_pg">
<?php echo gdrcd_filter('out', $MESSAGE['interface']['user']['link']['back']); ?>
</a>
</div>
<?php }
/*Cambio pass utenti*/
if((gdrcd_filter('get', $_POST['op']) == 'force') && ($_SESSION['permessi'] == SUPERUSER)) {
gdrcd_query("UPDATE personaggio SET permessi = -1 WHERE nome = '".gdrcd_filter('in', $_POST['account'])."'");
gdrcd_query("INSERT INTO log (nome_interessato, autore, data_evento, codice_evento, descrizione_evento) VALUES ('".gdrcd_filter('in', $_POST['account'])."','".$_SESSION['login']."', NOW(), ".DELETEPG." ,'->')");
?>
<div class="warning">
<?php echo gdrcd_filter('out', $MESSAGE['warning']['modified']); ?>
</div>
<div class="link_back">
<a href="main.php?page=user_cancella_pg">
<?php echo gdrcd_filter('out', $MESSAGE['interface']['user']['link']['back']); ?>
</a>
</div>
<?php }
/*Cambio pass utenti*/
if((gdrcd_filter('get', $_POST['op']) == 'get_back') && ($_SESSION['permessi'] >= MODERATOR)) {
gdrcd_query("UPDATE personaggio SET permessi = 0 WHERE nome = '".gdrcd_filter('in', $_POST['account'])."'");
gdrcd_query("INSERT INTO log (nome_interessato, autore, data_evento, codice_evento, descrizione_evento) VALUES ('".gdrcd_filter('in', $_POST['account'])."','".$_SESSION['login']."', NOW(), ".DELETEPG." ,'<-')");
?>
<div class="warning">
<?php echo gdrcd_filter('out', $MESSAGE['warning']['modified']); ?>
</div>
<div class="link_back">
<a href="main.php?page=user_cancella_pg">
<?php echo gdrcd_filter('out', $MESSAGE['interface']['user']['link']['back']); ?>
</a>
</div>
<?php }


Con questo:


/*Cancella altri - MODERATORE (Disabilita account)*/
if((gdrcd_filter('get', $_POST['op']) == 'force') && ($_SESSION['permessi'] == MODERATOR)) {
gdrcd_query("UPDATE personaggio SET permessi = -1 WHERE nome = '".gdrcd_filter('in', $_POST['account'])."' AND permessi < ".SUPERUSER."");
gdrcd_query("INSERT INTO log (nome_interessato, autore, data_evento, codice_evento, descrizione_evento) VALUES ('".gdrcd_filter('in', $_POST['account'])."','".$_SESSION['login']."', NOW(), ".DELETEPG." ,'".gdrcd_filter('in', $MESSAGE['interface']['user']['delete']['deleted'])."')");
?>
<div class="warning">
<?php echo gdrcd_filter('out', $MESSAGE['warning']['modified']); ?>
</div>
<div class="link_back">
<a href="main.php?page=user_cancella_pg">
<?php echo gdrcd_filter('out', $MESSAGE['interface']['user']['link']['back']); ?>
</a>
</div>
<?php }
/*Cancella altri - SUPERUSER (Cancella account) */
if((gdrcd_filter('get', $_POST['op']) == 'force') && ($_SESSION['permessi'] == SUPERUSER)) {
gdrcd_query("DELETE FROM personaggio WHERE nome = '".gdrcd_filter('in', $_POST['account'])."'");
gdrcd_query("INSERT INTO log (nome_interessato, autore, data_evento, codice_evento, descrizione_evento) VALUES ('".gdrcd_filter('in', $_POST['account'])."','".$_SESSION['login']."', NOW(), ".DELETEPG." ,'->')");
?>
<div class="warning">
<?php echo gdrcd_filter('out', $MESSAGE['warning']['modified']); ?>
</div>
<div class="link_back">
<a href="main.php?page=user_cancella_pg">
<?php echo gdrcd_filter('out', $MESSAGE['interface']['user']['link']['back']); ?>
</a>
</div>
<?php }
/*Ripristina account*/
if((gdrcd_filter('get', $_POST['op']) == 'get_back') && ($_SESSION['permessi'] >= MODERATOR)) {
gdrcd_query("UPDATE personaggio SET permessi = 0 WHERE nome = '".gdrcd_filter('in', $_POST['account'])."'");
gdrcd_query("INSERT INTO log (nome_interessato, autore, data_evento, codice_evento, descrizione_evento) VALUES ('".gdrcd_filter('in', $_POST['account'])."','".$_SESSION['login']."', NOW(), ".DELETEPG." ,'<-')");
?>
<div class="warning">
<?php echo gdrcd_filter('out', $MESSAGE['warning']['modified']); ?>
</div>
<div class="link_back">
<a href="main.php?page=user_cancella_pg">
<?php echo gdrcd_filter('out', $MESSAGE['interface']['user']['link']['back']); ?>
</a>
</div>
<?php }


Kasa.



Ho provato a correggerle, aggiornando anche i commenti.

img

GitHub: https://github.com/GDRCD/GDRCD ↗
Discord: https://discord.gg/zh69CDUf3V ↗

quota

alejandra

SCRIVI

26/06/2021 11:52:35

kasa ha scritto:
alejandra ha scritto: ...



Intanto GRAZIE mi hai risolto non pochi dilemmi.
Il cambio nome funziona, il cancella account funziona alla grande.

Unico problema è che, se faccio il cambio pass a un account, quando faccio il login non lo fa accedere e mi dice fallimento
Ale 🎲

quota

26/06/2021 12:07:51

alejandra ha scritto:
Unico problema è che, se faccio il cambio pass a un account, quando faccio il login non lo fa accedere e mi dice fallimento


Strano.
Ho effettuato diverse verifiche con la versione vanilla di GDRCD#5.5.1 e non ho riscontrato problemi. Hai la certezza di aver comunicato la giusta password all'utente? Compresa anche di lettere in maiuscolo e via dicendo? La nuova password conteneva caratteri speciali?

Kasa.

quota

26/06/2021 13:38:05

Ho notato che in alcuni casi il sistema inserisce uno spazio dopo la casella email e/o dopo la password ed è quello che da problemi. Almeno, tutte le volte che un utente da noi ha cercato di recuperare la pass, senza riuscire, era perché poi controllando la sua email, c'era uno spazio alla fine che dava fastidio.

Non so se questo ti può essere d'aiuto.

img
Leggendra.eu - Fantasy steampunk itinerante
https://www.leggendra.eu ↗

quota

26/06/2021 15:58:36

kasa ha scritto:
Strano.
Ho effettuato diverse verifiche con la versione vanilla di GDRCD#5.5.1 e non ho riscontrato problemi. Hai la certezza di aver comunicato la giusta password all'utente? Compresa anche di lettere in maiuscolo e via dicendo? La nuova password conteneva caratteri speciali?

Kasa.


Risolto questo punto però adesso il cambio password eseguito dall'utente non funziona.
Quando inserisce la mail di registrazione e poi la nuova pass, da "impossibile eseguire"


Io continuo a ringraziarvi per la pazienza e i consigli <3
Ale 🎲

quota

27/06/2021 21:12:48

alejandra ha scritto: ...


L'errore deriva da un BUG presente su GDRCD#5.5.1.
Con le ultime modifiche applicate al metodo di iscrizione, oltre alla password, viene criptata anche l'email dell'utente, probabilmente per tenere l'engine in linea con le ultime disposizioni GDPR. Tuttavia, il modulo pages/user_cambio_pass.inc non è stato allineato con questo aggiornamento ed è rimasto indietro sulla verifica dell'email.

Per correggere, sostituire questa linea:

if(($email == gdrcd_filter_email($_POST['email'])) && (gdrcd_check_pass($_POST['new_pass']) === true)) {


Con questa:

if((gdrcd_password_check(gdrcd_filter_email($_POST['email']),$email)) && (gdrcd_check_pass($_POST['new_pass']) === true)) {


Dovrebbe tornare a funzionare.

Kasa.

quota

27/06/2021 22:17:18

kasa ha scritto:

Dovrebbe tornare a funzionare.

Kasa.



Io ti devo di sicuro almeno una bevuta. Se passi dalle parti di Livorno avvisami *_* che un grazie come minimo te lo devo.
Ale 🎲

quota

27/06/2021 22:45:43

alejandra ha scritto:
Io ti devo di sicuro almeno una bevuta. Se passi dalle parti di Livorno avvisami *_* che un grazie come minimo te lo devo.


Figurati!
Non serve alcuna bevuta, si tratta alla fine di meri BUG.

Ricordati di segnalre il thread come risolto, in modo che se un giorno ci saranno altre persone che riscontrano lo stesso problema, glielo si può inoltrare.

Per il resto,
Buon Gioco!

Kasa.

quota

01/07/2021 11:35:54

WARNING!

Effettuando qualche verifica sul funzionamento delle operazioni di manutenzione assieme a @staff shadowland, qui riassunte https://www.gdr-online.com/readforum.asp?id=241471 ↗, ho notato che la disattivazione degli account prevista in Menu Utente > Cancella account è corretta, anche in caso di SUPERUSER.

Di fatto, gli account non vengono cancellati, come spiegavo, ma disattivati. Una volta disattivati, per completare la cancellazione occorre andare in Gestione > Manutenzione e quindi effettuare l'operazione Elimina i personaggi provvisoriamente cancellati.

In questo modo, oltre che a confermare la cancellazione degli account (che non appariranno nemmeno più in Anagrafe, si andranno ad eliminare i messaggi e i log, evitando anomalie.

Se si effettua la cancellazione diretta, come avevo suggerito erroneamente, non sarà più possibile cancellare messaggi e log, salvo ovviamente aggiungere ulteriori query nel processo in Manutenzione.

Pertanto, chiunque ha applicato la mia modifica, lo invito a sostituire in user_cancella_pg.inc.php la riga:

/*Cancella altri - SUPERUSER (Cancella account) */
if((gdrcd_filter('get', $_POST['op']) == 'force') && ($_SESSION['permessi'] == SUPERUSER)) {
gdrcd_query("DELETE FROM personaggio WHERE nome = '".gdrcd_filter('in', $_POST['account'])."'");


Con:

/*Cancella altri - SUPERUSER (Disattiva account) */
if((gdrcd_filter('get', $_POST['op']) == 'force') && ($_SESSION['permessi'] == SUPERUSER)) {
gdrcd_query("UPDATE personaggio SET permessi = -1 WHERE nome = '".gdrcd_filter('in', $_POST['account'])."'");


In questo modo, il processo tornerà ad essere corretto e si potrà effettuare una cancellazione più accurata del Personaggio tramite Gestione > Manutenzione.

Chiedo venia per la svista.

Kasa.

quota

30/07/2021 18:37:48

kasa ha scritto: WARNING!

Effettuando qualche verifica sul funzionamento delle operazioni di manutenzione assieme a @staff shadowland, qui riassunte https://www.gdr-online.com/readforum.asp?id=241471 ↗, ho notato che la disattivazione degli account prevista in Menu Utente > Cancella account è corretta, anche in caso di SUPERUSER.

Di fatto, gli account non vengono cancellati, come spiegavo, ma disattivati. Una volta disattivati, per completare la cancellazione occorre andare in Gestione > Manutenzione e quindi effettuare l'operazione Elimina i personaggi provvisoriamente cancellati.

In questo modo, oltre che a confermare la cancellazione degli account (che non appariranno nemmeno più in Anagrafe, si andranno ad eliminare i messaggi e i log, evitando anomalie.

Se si effettua la cancellazione diretta, come avevo suggerito erroneamente, non sarà più possibile cancellare messaggi e log, salvo ovviamente aggiungere ulteriori query nel processo in Manutenzione.

Pertanto, chiunque ha applicato la mia modifica, lo invito a sostituire in user_cancella_pg.inc.php la riga:

/*Cancella altri - SUPERUSER (Cancella account) */
if((gdrcd_filter('get', $_POST['op']) == 'force') && ($_SESSION['permessi'] == SUPERUSER)) {
gdrcd_query("DELETE FROM personaggio WHERE nome = '".gdrcd_filter('in', $_POST['account'])."'");


Con:

/*Cancella altri - SUPERUSER (Disattiva account) */
if((gdrcd_filter('get', $_POST['op']) == 'force') && ($_SESSION['permessi'] == SUPERUSER)) {
gdrcd_query("UPDATE personaggio SET permessi = -1 WHERE nome = '".gdrcd_filter('in', $_POST['account'])."'");


In questo modo, il processo tornerà ad essere corretto e si potrà effettuare una cancellazione più accurata del Personaggio tramite Gestione > Manutenzione.

Chiedo venia per la svista.

Kasa.



Mi scuso per il ritardo nel rispondere ma ero nel periodo esame. Comunque funziona tutto a dovere con la nuova correzione. Grazie ancora
Ale 🎲

quota

Discussione seguita da › alejandra

Pagine → 1 

Rispondi alla Discussione Segui Discussione Inoltra Discussione Forum Programmazione, GDRCD, Open Source, Hosting Elenco Forum

Articoli, Interviste e altre Risorse!

I Vampiri
I Vampiri
Tesi di laurea dal titolo: Il vampiro della tomba accanto
VPS
VPS
Cos'è il VPS e quale il suo utilizzo nel Gaming?
Il Gioco di Ruolo
Il Gioco di Ruolo
Corposa tesi (oltre 200 pagine!) di Gilberto Fulvi sul Gioco di Ruolo
Gioco e Pregiudizio
Gioco e Pregiudizio
Tesi dal titolo "Giocare contro il pregiudizio. Serious games e bullismo omofobico."

Pubblicità

Ultime dal Portale

news RAID Shadow Legends: Summon Boost e Champion Chase

news Storie di Esperia: Uno strappo nella notte

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

news Crystal Tokyo: Ingresso nella Via Magica

forum alexandros si è accreditato come gestore di Shinobi no Saisei - Naruto x Boruto GDR

risposta Il gestore di Naruto x Boruto GDR - New Rebirth ha risposto alla recensione di jimcho

risposta Il gestore di Naruto x Boruto GDR - New Rebirth ha risposto alla recensione di theduelist92

World of Tanks World of Tanks → Lanciati in epiche battaglie spalla a spalla con altri giocatori. Conquista la supremazia nel mondo dei Carri Armati!

news War Thunder: Chieftain 900: Capo della Royal Ordnance

gioco Il portale raggiunge i 17.200 giochi catalogati! Grazie a tutti!

news Hero Wars: Anomalia dei Pets! 🐾

news New Orleans: [On Game] Casting per i Cantanti!

Lineage II Lineage II → Entra in uno sconfinato mondo fantasy dominato da razze in contrasto tra loro. Scatena i tuoi poteri in uno dei Mmo più famosi al mondo!

gdrrank I dati del generatore di rank sono stati aggiornati!

news Le Cronache di Raja Dunia: Ampliamento Enciclopedia Creature

news Enlisted: Combattimento alla pari: Regno Unito contro Italia

news World of Tanks: Manovre militari torna a novembre!

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

Articoli

Hogwarts Express Hogwarts Express - Intervista allo staff del play by forum potteriano Hogwarts Express

Doppio clic per il reame Doppio clic per il reame - Presentazione del gdr eXtremelot della rivista Step1

Jooke Jooke - Presentazione del portale Jooke dedicato alle cronache Play by Email

Crea il tuo GDR Crea il tuo GDR - Scopri come creare da zero un gdr in PHP! Partiamo dalla registrazione..

Marco Cuccu Marco Cuccu - Intervista a Marco Cuccu, amministratore del gruppo Facebook Giochi di Ruolo - Il GdR in Italia

To Infinity and Beyond To Infinity and Beyond - Intervista ai gestori del play by forum fantasy To Infinity and Beyond GdR

Skype Chiude! Skype Chiude! - L'Addio a Skype: un'era volge al termine, ma il futuro è in Teams!