Threaded Mode | Mod liniar

09-27-2011 03:10 PM

Azi am descoperit o nouă problemă legată de una dintre modificările de top pentru MyBB. Ea apare la toate versiunile de My Advertisements care au fost publicate până pe saitul oficial mybb.com. În ce constă problema...
Păi în momentul în care adăugați o zonă publicitară pe forumul dvs. cu această modificare și dați click pe acel spațiu, în baza de date se contabilizează numărul de click-uri date pe zona respectivă. Problema e că se pot efectua și click-uri false (fake), adică se pot adăuga click-uri fără să se dea click pe reclamă. De unde apare problema? În corpul funcției myadvertisements_click este buba.

Cod PHP:

function myadvertisements_click()
{
	global $mybb, $db;
	
	if ($mybb->settings['myadvertisements_disabled'])
		return;
	
	// in case we switch page (we clicked a lin, right?) the script is not stopped
	ignore_user_abort(true);
	
	// this query could be avoided if we logged all advertisement ID's in cache or something
	// TODO: store active advertisements ID's and expiration time in cache and check if the aid exists in the line below instead of running the query
	$query = $db->simple_select('myadvertisements_advertisements', '*', 'aid='.intval($mybb->input['aid']));
	$ad = $db->fetch_array($query);
	if (!$ad)
		return; // do not log clicks as the ad doesn't exist
			
	// increase clicks
	$db->update_query('myadvertisements_advertisements', array('clicks' => 'clicks+1'), 'aid='.$ad['aid'], 1, true);
}

Autorul modificării a uitat să adauge un test de verificare a autenticității cererii "post" efectuată de scriptul myadvertisements.js. După corectarea acestei probleme, funcția ar trebui să arate așa :

Cod PHP:

function myadvertisements_click()
{
	global $mybb, $db;
	
	if ($mybb->settings['myadvertisements_disabled'])
		return;
	
	// in case we switch page (we clicked a lin, right?) the script is not stopped
	ignore_user_abort(true);
	
    if (isset($mybb->input['action']) && $mybb->input['action'] == "do_click" && isset($mybb->input['my_post_key']) && verify_post_key($mybb->input['my_post_key'], true)) 
    {
        // this query could be avoided if we logged all advertisement ID's in cache or something
	    // TODO: store active advertisements ID's and expiration time in cache and check if the aid exists in the line below instead of running the query
	    $query = $db->simple_select('myadvertisements_advertisements', '*', 'aid='.intval($mybb->input['aid']));
	    $ad = $db->fetch_array($query);
	    if (!$ad)
            return; // do not log clicks as the ad doesn't exist
			
	    // increase clicks
	    $db->update_query('myadvertisements_advertisements', array('clicks' => 'clicks+1'), 'aid='.$ad['aid'], 1, true);
    }
}

Acum ar trebui ca la accesarea unui URL de genul următor DOMENIU/xmlhttp.php?aid=ZONA_PUBLICITARA să nu mai crească numărul de click-uri!
Problema o voi raporta și autorului acestei modificări... Să sperăm că face un update cât mai repede posibil!

10-28-2011 12:59 PM

Bun !

Citat:Problema o voi raporta și autorului acestei modificări... Să sperăm că face un update cât mai repede posibil!

Ti-a raspuns ? In legatura cu update-ul ?

10-29-2011 10:03 PM

Dap a si facut update! Versiunea de fata nu mai are aceasta problema!

Threaded Mode \| Mod liniar Fix My Advertisements
Autor	Mesaj