Ajax Star Rating - IP Sperre raus nehmen

Beitrag von **pwehrmann** » 29.09.2011, 11:13

Moin, hab mir gestern auf meiner Seite das Ajax Star Rating installiert.
Jetzt ist aber das Problem, das jede IP nur einmal bewerten darf. Was natürlich blöd ist, da ich das Script als Foto-Bewertung nehme und möchte, das jedes Bild einmal bewertet werden darf.

Rausgefunden hab ich bisher, das die Sperre in der rpc.php definiert ist. Den Codeblock komplett rausnehmen bringt nix.

Hat da jemand Erfahrung mit?
Soll ich hier mal den kompletten Code der rpc.php posten zum drüber gucken?

Gruss

von **Anzeige von ABAKUS** »

Beitrag von **Synonym** » 29.09.2011, 11:57

Soll ich hier mal den kompletten Code der rpc.php posten zum drüber gucken?

Was sicherlich nicht verkehrt wäre... ich hab hier eine rpc.php, die hat mit Sternen und Bewertungen aber nichts zu tun

Beitrag von **pwehrmann** » 29.09.2011, 12:14

Okidoki

Hab bisher ein wenig rumprobiert, wie z.B. das statt der IP eine Zufallszahl eingetragen wird, aber selbst dann kann man nur einmal voten. Also nur ein einziges Bild von meinen ca 1200 Bildern.

Code: Alles auswählen

<?php
/*
Page&#58;           rpc.php
Created&#58;        Aug 2006
Last Mod&#58;       Mar 18 2007
This page handles the 'AJAX' type response if the user
has Javascript enabled.
--------------------------------------------------------- 
ryan masuga, masugadesign.com
ryan@masugadesign.com 
Licensed under a Creative Commons Attribution 3.0 License.
http&#58;//creativecommons.org/licenses/by/3.0/
See readme.txt for full credit details.
--------------------------------------------------------- */
header&#40;"Cache-Control&#58; no-cache"&#41;;
header&#40;"Pragma&#58; nocache"&#41;;

require&#40;'_config-rating.php'&#41;; // get the db connection info

//getting the values
$vote_sent = preg_replace&#40;"/&#91;^0-9&#93;/","",$_REQUEST&#91;'j'&#93;&#41;;
$id_sent = preg_replace&#40;"/&#91;^0-9a-zA-Z&#93;/","",$_REQUEST&#91;'q'&#93;&#41;;
$ip_num = preg_replace&#40;"/&#91;^0-9\.&#93;/","",$_REQUEST&#91;'t'&#93;&#41;;
$units = preg_replace&#40;"/&#91;^0-9&#93;/","",$_REQUEST&#91;'c'&#93;&#41;;
$ip = $_SERVER&#91;'REMOTE_ADDR'&#93;;

if &#40;$vote_sent > $units&#41; die&#40;"Sorry, vote appears to be invalid."&#41;; // kill the script because normal users will never see this.


//connecting to the database to get some information
$query = mysql_query&#40;"SELECT total_votes, total_value, used_ips FROM $rating_dbname.$rating_tableName WHERE id='$id_sent' "&#41;or die&#40;" Error&#58; ".mysql_error&#40;&#41;&#41;;
$numbers = mysql_fetch_assoc&#40;$query&#41;;
$checkIP = unserialize&#40;$numbers&#91;'used_ips'&#93;&#41;;
$count = $numbers&#91;'total_votes'&#93;; //how many votes total
$current_rating = $numbers&#91;'total_value'&#93;; //total number of rating added together and stored
$sum = $vote_sent+$current_rating; // add together the current vote value and the total vote value
$tense = &#40;$count==1&#41; ? "vote" &#58; "votes"; //plural form votes/vote

// checking to see if the first vote has been tallied
// or increment the current number of votes
&#40;$sum==0 ? $added=0 &#58; $added=$count+1&#41;;

// if it is an array i.e. already has entries the push in another value
&#40;&#40;is_array&#40;$checkIP&#41;&#41; ? array_push&#40;$checkIP,$ip_num&#41; &#58; $checkIP=array&#40;$ip_num&#41;&#41;;
$insertip=serialize&#40;$checkIP&#41;;

//IP check when voting
$voted=mysql_num_rows&#40;mysql_query&#40;"SELECT used_ips FROM $rating_dbname.$rating_tableName WHERE used_ips LIKE '%".$ip."%' AND id='".$id_sent."' "&#41;&#41;;
if&#40;!$voted&#41; &#123;     //if the user hasn't yet voted, then vote normally...

	if &#40;&#40;$vote_sent >= 1 && $vote_sent <= $units&#41; && &#40;$ip == $ip_num&#41;&#41; &#123; // keep votes within range, make sure IP matches - no monkey business!
		$update = "UPDATE $rating_dbname.$rating_tableName SET total_votes='".$added."', total_value='".$sum."', used_ips='".$insertip."' WHERE id='$id_sent'";
		$result = mysql_query&#40;$update&#41;;		
	&#125; 
&#125; //end for the "if&#40;!$voted&#41;"
// these are new queries to get the new values!
$newtotals = mysql_query&#40;"SELECT total_votes, total_value, used_ips FROM $rating_dbname.$rating_tableName WHERE id='$id_sent' "&#41;or die&#40;" Error&#58; ".mysql_error&#40;&#41;&#41;;
$numbers = mysql_fetch_assoc&#40;$newtotals&#41;;
$count = $numbers&#91;'total_votes'&#93;;//how many votes total
$current_rating = $numbers&#91;'total_value'&#93;;//total number of rating added together and stored
$tense = &#40;$count==1&#41; ? "vote" &#58; "votes"; //plural form votes/vote

// $new_back is what gets 'drawn' on your page after a successful 'AJAX/Javascript' vote

$new_back = array&#40;&#41;;

$new_back&#91;&#93; .= '<ul class="unit-rating" style="width&#58;'.$units*$rating_unitwidth.'px;">';
$new_back&#91;&#93; .= '<li class="current-rating" style="width&#58;'.@number_format&#40;$current_rating/$count,2&#41;*$rating_unitwidth.'px;">Current rating.</li>';
$new_back&#91;&#93; .= '<li class="r1-unit">1</li>';
$new_back&#91;&#93; .= '<li class="r2-unit">2</li>';
$new_back&#91;&#93; .= '<li class="r3-unit">3</li>';
$new_back&#91;&#93; .= '<li class="r4-unit">4</li>';
$new_back&#91;&#93; .= '<li class="r5-unit">5</li>';
$new_back&#91;&#93; .= '<li class="r6-unit">6</li>';
$new_back&#91;&#93; .= '<li class="r7-unit">7</li>';
$new_back&#91;&#93; .= '<li class="r8-unit">8</li>';
$new_back&#91;&#93; .= '<li class="r9-unit">9</li>';
$new_back&#91;&#93; .= '<li class="r10-unit">10</li>';
$new_back&#91;&#93; .= '</ul>';
$new_back&#91;&#93; .= '<p class="voted">'.$id_sent.'. Rating&#58; <strong>'.@number_format&#40;$sum/$added,1&#41;.'</strong>/'.$units.' &#40;'.$count.' '.$tense.' cast&#41; ';
$new_back&#91;&#93; .= '<span class="thanks">Thanks for voting!</span></p>';

$allnewback = join&#40;"\n", $new_back&#41;;

// ========================

//name of the div id to be updated | the html that needs to be changed
$output = "unit_long$id_sent|$allnewback";
echo $output;
?>

Beitrag von **Synonym** » 29.09.2011, 13:34

ok, da komme ich so auch nicht weiter. Ich würde sagen, dass man da durchaus mehrfach vote kann, zumindest erklärt sich mir das "WHERE used_ips LIKE '%".$ip."%' AND id='".$id_sent."' "" so. Die IP ist da erst mal egal, wenn id_sent ein anderer Wert ist.

Doch der kommt aus $_REQUEST['q'] und davon ist im Script nichts zu sehen. Wäre also die Frage, wie das andere Script aussieht, das die Daten sendet. Das hier empfängt ja nur.

Beitrag von **pwehrmann** » 29.09.2011, 14:09

Hm, grad mall alle Dateien durchgeschaut, da ist nix wirklich drin.

Als ID wird immer der Name des Bildes genommen. So sieht die Datenbank aus für die Bewertung:

Bild

Beitrag von **pwehrmann** » 29.09.2011, 14:20

Edit: Falsch, hier kommen die Daten her

Code: Alles auswählen

<?php
/*
Page&#58;           _drawrating.php
Created&#58;        Aug 2006
Last Mod&#58;       Mar 18 2007
The function that draws the rating bar.
--------------------------------------------------------- 
ryan masuga, masugadesign.com
ryan@masugadesign.com 
Licensed under a Creative Commons Attribution 3.0 License.
http&#58;//creativecommons.org/licenses/by/3.0/
See readme.txt for full credit details.
--------------------------------------------------------- */
function rating_bar&#40;$id,$units='',$static=''&#41; &#123; 

require&#40;'_config-rating.php'&#41;; // get the db connection info
	
//set some variables
$ip = $_SERVER&#91;'REMOTE_ADDR'&#93;;
if &#40;!$units&#41; &#123;$units = 10;&#125;
if &#40;!$static&#41; &#123;$static = FALSE;&#125;

// get votes, values, ips for the current rating bar
$query=mysql_query&#40;"SELECT total_votes, total_value, used_ips FROM $rating_dbname.$rating_tableName WHERE id='$id' "&#41;or die&#40;" Error&#58; ".mysql_error&#40;&#41;&#41;;


// insert the id in the DB if it doesn't exist already
// see&#58; http&#58;//www.masugadesign.com/the-lab/scripts/unobtrusive-ajax-star-rating-bar/#comment-121
if &#40;mysql_num_rows&#40;$query&#41; == 0&#41; &#123;
$sql = "INSERT INTO $rating_dbname.$rating_tableName &#40;`id`,`total_votes`, `total_value`, `used_ips`&#41; VALUES &#40;'$id', '0', '0', ''&#41;";
$result = mysql_query&#40;$sql&#41;;
&#125;

$numbers=mysql_fetch_assoc&#40;$query&#41;;


if &#40;$numbers&#91;'total_votes'&#93; < 1&#41; &#123;
	$count = 0;
&#125; else &#123;
	$count=$numbers&#91;'total_votes'&#93;; //how many votes total
&#125;
$current_rating=$numbers&#91;'total_value'&#93;; //total number of rating added together and stored
$tense=&#40;$count==1&#41; ? "Vote" &#58; "Votes"; //plural form votes/vote

// determine whether the user has voted, so we know how to draw the ul/li
$voted=mysql_num_rows&#40;mysql_query&#40;"SELECT used_ips FROM $rating_dbname.$rating_tableName WHERE used_ips LIKE '%".$ip."%' AND id='".$id."' "&#41;&#41;; 

// now draw the rating bar
$rating_width = @number_format&#40;$current_rating/$count,2&#41;*$rating_unitwidth;
$rating1 = @number_format&#40;$current_rating/$count,1&#41;;
$rating2 = @number_format&#40;$current_rating/$count,2&#41;;


if &#40;$static == 'static'&#41; &#123;

		$static_rater = array&#40;&#41;;
		$static_rater&#91;&#93; .= "\n".'<div class="ratingblock">';
		$static_rater&#91;&#93; .= '<div id="unit_long'.$id.'">';
		$static_rater&#91;&#93; .= '<ul id="unit_ul'.$id.'" class="unit-rating" style="width&#58;'.$rating_unitwidth*$units.'px;">';
		$static_rater&#91;&#93; .= '<li class="current-rating" style="width&#58;'.$rating_width.'px;">Currently '.$rating2.'/'.$units.'</li>';
		$static_rater&#91;&#93; .= '</ul>';
		$static_rater&#91;&#93; .= '<p class="static">  &#40;'.$count.' '.$tense.'&#41; <em>This is \'static\'.</em></p>';
		$static_rater&#91;&#93; .= '</div>';
		$static_rater&#91;&#93; .= '</div>'."\n\n";

		return join&#40;"\n", $static_rater&#41;;


&#125; else &#123;

      $rater ='';
      $rater.='<div class="ratingblock">';

      $rater.='<div id="unit_long'.$id.'">';
      $rater.='  <ul id="unit_ul'.$id.'" class="unit-rating" style="width&#58;'.$rating_unitwidth*$units.'px;">';
      $rater.='     <li class="current-rating" style="width&#58;'.$rating_width.'px;">Currently '.$rating2.'/'.$units.'</li>';

      for &#40;$ncount = 1; $ncount <= $units; $ncount++&#41; &#123; // loop from 1 to the number of units
           if&#40;!$voted&#41; &#123; // if the user hasn't yet voted, draw the voting stars
              $rater.='<li><a href="db.php?j='.$ncount.'&q='.$id.'&t='.$ip.'&c='.$units.'" title="'.$ncount.' out of '.$units.'" class="r'.$ncount.'-unit rater" rel="nofollow">'.$ncount.'</a></li>';
           &#125;
      &#125;
      $ncount=0; // resets the count

      $rater.='  </ul>';
      $rater.='  <p';
      if&#40;$voted&#41;&#123; $rater.=' class="voted"'; &#125;
      $rater.='>  &#40;'.$count.' '.$tense.'&#41;';
      $rater.='  </p>';
      $rater.='</div>';
      $rater.='</div>';
      return $rater;
 &#125;
&#125;
?>

Beitrag von **Synonym** » 30.09.2011, 07:34

Als ID wird immer der Name des Bildes genommen.

So hatte ich das auch verstanden. Also müsste man verschiedene Bilder mit der gleichen IP bewerten können, denn die ID wird bei der SQL-Abfrage berücksichtigt.

Aber so kommen wir nicht weiter, zumindest ich nicht. Aus dem ersten File sind die ganzen $_REQUEST unbekannt. Irgendwo müssen die gesendet werden.

Das zweite Script für auch zu keiner Sperre, wenn IP oder ID verschieden sind. Aber auch hier sieht man nicht, wo die ID her kommt und wie die dann aussieht.

Irgendwo muss die Funktion "function rating_bar($id,$units='',$static='')" aufgerufen werden und da kommt auch die ID dann her.

Beitrag von **Synonym** » 30.09.2011, 07:41

Nachtrag:
Da kommen die $_REQUESTS her:

Code: Alles auswählen

$rater.='<li><a href="db.php?j='.$ncount.'&q='.$id.'&t='.$ip.'&c='.$units.'" title="'.$ncount.' out of '.$units.'" class="r'.$ncount.'-unit rater" rel="nofollow">'.$ncount.'</a></li>';

IP wird ermittelt
ncoutn wird berechnet
id wird von Dir an die Funktion übergeben
units ist Systemvorgabe oder auch der Funktion übergeben

1. Also sehe ich da nur zwei Punkte. Das Script hat Fehler.
oder
2. Du übergibst falsche Werte an die Funktion

Beitrag von **Synonym** » 30.09.2011, 07:45

Klickt man auf einen Stern, dann geht der Request an die db.php. Dort werden die Daten ausgewertet und dann mit der DB verglichen:

$voted=mysql_num_rows(mysql_query("SELECT used_ips FROM $rating_dbname.$rating_tableName WHERE used_ips LIKE '%".$ip."%' AND id='".$id_sent."' "));

Und hier müssen IP und ID wieder beide stimmen, sonst zählt der Voter als neuer und wird gewertet. Also kann ich mir da nur vorstellen, dass Du eine falsche ID sendest.

Beitrag von **pwehrmann** » 30.09.2011, 10:47

Also irgendwie komm ich da gar nicht weiter, mittlerweile können nicht mal andere User voten, ohne das ich was geändert habe. Die ID kann man doch irgendwie gar nicht falsch übergeben, sie wird in der Datenbank ja auch richtig zugeordnet. Das komische, ich hatte vor längerer Zeit dieses Script schon mal problemlos auf einer anderen Seite genutzt, da ging es problemlos.

https://masugadesign.com/software/unobt ... ating-bar/

Beitrag von **Synonym** » 30.09.2011, 11:59

Ich habe mir das Script schon angesehen. Aber ohne die Informationen (die immer noch fehlen) kann ich keine Antwort geben.

"Die ID kann man doch irgendwie gar nicht falsch übergeben"
Das ist keine Antwort. Denn ich vermute genau hier den Fehler. Das Update des Datensatzes braucht nur die ID und die IP. Also muss davon was falsch sein oder die DB Verbindung geht nicht - dann könnte man aber gar nicht voten - auch nicht einmal.

"Also irgendwie komm ich da gar nicht weiter, mittlerweile können nicht mal andere User voten"
Das spricht noch mehr für einen Benutzerfehler.

Scripte funktionieren oder funktionieren nicht. Die suchen sich das aber nicht nach belieben aus - zumindest nicht so einfache.

Nenn doch einfach schon mal die Seite, auf der das Script eingebunden ist - dann kann man sich das schon mal ansehen.

Dann einen DB-Export mit mehr als einem Datensatz. 5-10 wären da schon sinnvoll.

Auch möglich: Du hast keine IP aus dem IPv4-Bereich, sondern aus v6, dann könnte das Script durchaus Probleme machen. Es erwartet bei der IP nämlich eine Ziffernfolge aus 0 bis 9.

Beitrag von **pwehrmann** » 30.09.2011, 18:53

Danke für deine Geduld erstmal

Ich setz mich Montag wieder mit dem Script auseinander. Jetzt übers Wochenende ist schlecht, Besuche und so und mal den Kopf frei kriegen:

Habs auch erstmal ausgebaut, aber ich schick dir gleich per PN ein paar Infos.

Beitrag von **Synonym** » 01.10.2011, 07:55

Habs auch erstmal ausgebaut, aber ich schick dir gleich per PN ein paar Infos.

Ah, danke... Auf den beiden Seiten war ich schon gestern mittag. Auf der einen ist ein Script, aber ein anderes, auf der anderen Seite hatte ich gar keine Bewertungen gefunden. Nun auch nicht.

Aber ist ja ausgebaut. Das ist natürlich gut zu wissen und die beste Voraussetzung, sich das live anzusehen.

Ach ja, in der PM stand davon aber nichts. Hätte ich mir die letzten 30 Minuten ja auch sparen können. Dachte ich wäre blind und sehe es nur nicht.