Ban list non registra lo storico ban

[frank]

Sicuramente l'estensione va resa compatibile con la 3.1.8

[frank]

La parte da modificare dovrebbe essere questa in controller/main.php

Codice: Seleziona tutto

...
// query totale ban
$sql_ary = "SELECT COUNT(*) AS bans
FROM ".BANLIST_TABLE.",".USERS_TABLE."
WHERE ban_userid > 0
AND ban_userid = user_id
$banend";
//la eseguo
$result = $this->db->sql_query($sql_ary);
//risultati
$total_results = $this->db->sql_fetchfield('bans');
$this->db->sql_freeresult($result);
//mando al template
if($total_results == 1)
{
$this->template->assign_vars(array('TOTAL_BAN'       => $this->user->lang['TOTAL_BAN'])); 
}else{
$this->template->assign_vars(array(
   'TOTAL_BAN'       => sprintf($this->user->lang['TOTAL_BANS'], $total_results),
)); 
}
		//$l_message = !$this->config['acme_demo_goodbye'] ? 'DEMO_HELLO' : 'DEMO_GOODBYE';
		//$this->template->assign_var('DEMO_MESSAGE', $this->user->lang($l_message, $name));


		return $this->helper->render('banlist_body.html', $this->user->lang['BAN_LIST']);
	}
}

[brunino]

Frank, non ho purtroppo modo di aggiornare il mio forum a 3.1.8 in questo periodo, e di verificare quale sia stato l'aggiornamento che hanno fatto e perché non sia più compatibile.

[frank]

Qualcosa ho individuato: nello storico ban rimangono solo i ban permanenti

[brunino]

Si, ho capito, il fatto è che vengono eliminati tutti i Ban conclusi. Se vengono proprio eliminati dal database, si può fare poco.

[frank]

Poi ho notato un'altra cosa: se banno un utente per IP, non appare nella lista ban. Di solito quando si banna un utente, lo si fa per Ip, non per username perché è più sicuro.

[brunino]

Bannare per IP non sere pressoché a nulla... Ban per utente permette di bloccare quell'utente e costringerlo quantomeno a registrarsi di nuovo.
Inoltre non è che puoi fare una lista pubblica di IP bannati...

[frank]

Non è necessario rendere visibile gli ip degli utenti bannati.

Poi c'è da dire che non tutti hanno ip dinamici, l'ideale sarebbe bannare in un colpo solo, ip, username ed email.

[alex75]

bannare l'ip è controproducente. ormai quasi tutti usano ip dinamici e molto raramente chi ha un indirizzo ip statico si mette a rompere le scatole.

[frank]

Appunto, allora per bannare al meglio bisognerebbe creare un'estensione che banna ip, user ed email in un colpo.