ich habe mir schon den Hintern wundgesucht und nichts gefunden,aber wie ich jemanden kenne gibt es das wohl hier doch schon :biggrin1::biggrin1:
Gibt es einen Schnippi für das sortieren von Textlinks ?
Druckbare Version
ich habe mir schon den Hintern wundgesucht und nichts gefunden,aber wie ich jemanden kenne gibt es das wohl hier doch schon :biggrin1::biggrin1:
Gibt es einen Schnippi für das sortieren von Textlinks ?
kannst du eigentlich auch das hier einbauen
http://www.designerscripte.net/downl...do=file&id=199
das Teil hab ich ja drin für die Banner,aber auf Textlinks hat das Teil keine Lust :biggrin1:
Hab alles in die paidlinks.php eingefügt so wie es in der Beschreibung steht für FC,aber dann zeigt er die Links nicht mehr sondern nur die URL´s der jeweiligen Kampagnen und nach einem Klick kommt Werbetan nicht bekannt.
Der Code der eingefügt werden muss:
Eventuell muss ja nur was geändert werden :rolleyes:PHP-Code:Öffne jetzt die klick4.php
Suche folgendes :
$usr = mysql_fetch_assoc (db_query ('SELECT max_forced FROM '.$db_prefix.'_userdaten WHERE uid = '.$_SESSION['uid'].' LIMIT 1'));
Ersetze dies mit folgendem :
$usr = mysql_fetch_assoc (db_query ('SELECT max_forced, var, vonu FROM '.$db_prefix.'_userdaten WHERE uid = '.$_SESSION['uid'].' LIMIT 1'));
Suche :
ON (t1.tan = t2.tan AND (t2.uid = '.$_SESSION['uid'].' OR t2.ip = "'.$ip.'") AND t2.bis >= '.time().')
WHERE t2.tan IS NULL AND t1.werbeart = "forcedbanner" AND t1.menge > 0 AND t1.status = 1 AND t1.verdienst > 0 AND t1.sponsor != '.$_SESSION['uid'].' ORDER BY t1.verdienst DESC LIMIT '.$usr['max_forced'].'');
Ersetze dies mit folgendem :
ON (t1.tan = t2.tan AND (t2.uid = '.$_SESSION['uid'].' OR t2.ip = "'.$ip.'") AND t2.bis >= '.time().')
WHERE t2.tan IS NULL AND t1.werbeart = "forcedbanner" AND t1.menge > 0 AND t1.status = 1 AND t1.verdienst > 0 AND t1.sponsor != '.$_SESSION['uid'].' ORDER BY t1.'.$usr['var'].' '.$usr['vonu'].' LIMIT '.$usr['max_forced'].'');
ich hatte einmal das auch für Textlinks gehabt aber da es damals keine wollte habe ich alle rechte daran verkauft wenn ich den jenigen finde dem ich es damals gab , sag ich dir bescheid :thumb:
das wäre nett :thumb:
Wenn ich mich jetzt nicht ganz schwer täusche musst du es so ersetzen:Zitat:
Zitat von Plati
Ansonsten wird nicht die Werbeart Paidlinks sondern die Werbeart Forcedbanner aus der Tabelle ausgelesen.PHP-Code:ON (t1.tan = t2.tan AND (t2.uid = '.$_SESSION['uid'].' OR t2.ip = "'.$ip.'") AND t2.bis >= '.time().')
WHERE t2.tan IS NULL AND t1.werbeart = "paidlinks" AND t1.menge > 0 AND t1.status = 1 AND t1.verdienst > 0 AND t1.sponsor != '.$_SESSION['uid'].' ORDER BY t1.'.$usr['var'].' '.$usr['vonu'].' LIMIT '.$usr['max_forced'].'');
Wieso wird hier eigentlich Gnadenlos alle unbekannten Variablen und selbst IP-Adressen ohne Escapen in der DB abgefragt?
Wieso werden solche Klamotten nicht vorher Escapt?
Und was ist, wenn ein User die Daten zum Sortieren durch ein SQL-Injection ersetzt?
Meinst duZitat:
Zitat von breaker
$usr['var'].' '.$usr['vonu']
Die kommen ja aus der db, also sollten sie da überprüft werden, wo sie eingetragen werden, und nicht jedesmal, wenn sie verwendet werden.
Und wenn es hier noch um das Addon von adi geht, da wird afaik geprüft, dass dort auch nur "gültige" Werte eingetragen werden können.
Un zwecks unbekannten Variablen und IP Adresse:
Wo ist da was unbekannt in dem Query?
Die Session Variable sollte Scriptseitig nur mit einem Integer belegt werden können, die IP Adresse, da hat der Server bzw. dessen Konfiguration für zu sorgen, dass es auch eine IP Adresse ist.
zb. das:Zitat:
Zitat von jpwfour
Punkt 1 und 2 sieht mir nicht nach behandelten Vars aus...Punkt 3 ist wohl schneller, wenn es vorher einer Var zugewiesen wurde (besonders bei vielen Querys)HTML-Code:(t2.uid = '.$_SESSION['uid'].' OR t2.ip = "'.$ip.'") AND t2.bis >= '.time().')
Nur um das richtig zu verstehen:Zitat:
Zitat von breaker
Punkt1: $_SESSION['uid']
Wie sollte man hier was "reinschummeln" können?
Dieser Session Variablen wird die UserID aus der Datenbank zugewiesen, und sofern der Server nicht komplett schwachsinnig konfiguriert ist, kann man da keine anderen Werte "selber" reinsetzen, somit kann das nur eine Zahl von 0-XXXX sein oder ein leerer String (falls ausgeloggt)
Punkt 2: $ip
= $_SERVER['REMOTE_ADDR']
Auch hier gilt, sofern der Server/PHP nicht komplett falsch/veraltet sind, kommt hier eine gültige IP Adresse rein
Bei beiden ist es somit a) nicht Aufgabe des Scriptes an der Stelle, diese Variablen zu prüfen, b) wenn man das tatsächlich müsste, dann müsste man ja überall solche Variablen prüfen, was in ziemlichem Overkill enden würde (der eben sinnlos ist)
Punkt 3: time()
Also ob es schneller ist, diesen Wert eine Variablen zuzuweisen, bezweifle ich, auch wenn ich das nicht mit Sicherheit sagen kann.
Was aber dagegen spricht, ist bei "vielen Querys", dass ich vielleicht immer je Query die aktuelle Zeit haben will und nicht die, als time() einer Variablen zugewiesen wurde?
Auf jeden Fall spart man Speicherplatz, weil die Systemzeit eh immer vorhanden ist, also warum vorher speichern?
Selbst bei Aufrufen von date() hat vernünftige Software Caching Mechanismen, die zuvor Abgefragte Werte speichern.