Bannersortierung

[Xenon]

genau das ist die, hab ich da was falsch gemacht??

ne sry war Grad mein Fehler, habe habe da oben das Copyright gesehn vom vms-shop.de aber das ist das Textlink addon und das habe ich verwechselt und dachte es geht um ne andere bannersortierung, sry

[SilentKiller]

ich hab da bei der Bannersortierung ein kleines Fehlerchen beim Einbau in 1.2.4.

Hab alles entsprechend der Anleitung eingebaut, wobei mir aber beim Einbau schon erste Abweichungen in der Vorgabe aufgefallen ist-
Da hab ich improvisiert aber dazu gleich mehr-

Der Fehler der beim Aufruf des Klickbereichs erscheint:

Parse error: syntax error, unexpected T_STRING in klick4.php on line 48

Das ist bei mir die Zeile mit der WHERE t2.tan IS NULL AND t1... Abfrage

Laut Anleitung dieses Addons soll man ja

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'].''); 

ersetzen durch:

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'].''); 

ok, da es bei mir im script aber anders aussieht, nämlich so:

ON (t1.tan = t2.tan AND (t2.uid = '.$_SESSION['uid'].' OR t2.ip = "'.$ip.'"))
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']); 

habe ich versucht die Veränderung auf meins zu übetragen und zwar so:

ON (t1.tan = t2.tan AND (t2.uid = '.$_SESSION['uid'].' OR t2.ip = "'.$ip.'"))
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']); 

Naja das funktioniert halt nicht und führt zu dem besagten Fehler- auch wenn ich die Zeile aus der Anleitung übernehme gibts den gleichen Fehler wie bei meiner gebastelten.

Was noch anders war bei der Installation war z.b. das man die Zeile

$usr = mysql_fetch_assoc (db_query ('SELECT max_forced FROM '.$db_prefix.'_userdaten WHERE uid = '.$_SESSION['uid'].' LIMIT 1')); 

ersetzen sollte durch:

$usr = mysql_fetch_assoc (db_query ('SELECT max_forced, var, vonu FROM '.$db_prefix.'_userdaten WHERE uid = '.$_SESSION['uid'].' LIMIT 1')); 

da meine Zeile dort aber lautete:

$usr = mysql_fetch_assoc (db_query ('SELECT * FROM '.$db_prefix.'_userdaten WHERE uid = '.$_SESSION['uid'].' LIMIT 1')); 

habe ich weiderum versucht das zu übertragen indem ich es durch folgendes ersetzte:

$usr = mysql_fetch_assoc (db_query ('SELECT *, var, vonu FROM '.$db_prefix.'_userdaten WHERE uid = '.$_SESSION['uid'].' LIMIT 1')); 

in welchem der Teile ich nun den Fehler verursacht habe, schnall ich noch nicht so ganz, wäre für jede Hilfe dankbar.

Ich hielt meine Änderungen zumindest im Ansatz für vernünftig, da ja die Gegebenheiten andere sind- aber egal ob ich nun mit meinen Änderungen benutze, oder stur bei den Vorgaben des Addons bleibe, es bleibt fehlerhaft. Weis auch nicht ob es überhaupt was mit 1.2.4 zu tun hat, oder ob ich grad einfach nur Begriffstutzig bin... Daher verzeiht mir falls das mal wieder ne dumme Frage ist

[didith1207]

1. fehlt bei dir der reload in den ersten beiden Zeilen ist das teil vorhanden in deiner nicht!

 AND t2.bis >= '.time().') 

2. wenn du

SELECT * FROM 

abfragst fragst du ja alles ab was da drinnen steht ....
das was du da gebastelt hast hab ich so noch nie gesehen ^^

SELECT *, var, vonu FROM 

<<< das ist definitiv nix!

also dein

SELECT * FROM

reicht da völlig aus
aber das mit dem reload solltest du dir ansehen ansonnsten funktioniert dein teil nicht richtig!

[SilentKiller]

bei mir fehlt der reload, ja- Da er wie ich es gesagt habe in dem script von 1.2.4 dort nicht vorhanden ist- woher nehmen wenn nicht stehlen? Klar in den Beispielen von der Installationsanleitung ist der Reload drin, aber ich fands irgendie nicht sinnvoll den zu übernehmen wenn er vorher garnicht da war ^^

Und selbst wenn ich das jetzt einfach mal ohne über den Sinn nachzudenken dazunehme, und wie von Dir gesagt das

SELECT *, var, vonu FROM 

weglasse und wieder zum

SELECT * FROM 

übergehe, so belibt der Fehler dennoch bestehen, es muss noch an was anderem liegen...

[Dyman]

In den Beispielen sind es die Dateien

topframe_forced.php

topframe_text.php

ich weis nicht wie sich das mit der klick4.php verhält.

[SilentKiller]

hä? welche Beispiele meinst Du den jetzt?

Meine Beispiele jedenfalls sind nur aus der Installationsanleitung und meinem Script-

Mit topframe_forced.php und topframe_text.php, hat das doch in dem Fall garnix zu tun, da gehts doch nur um die Änderung in der klick4.php laut
Installationsanleitung

[didith1207]

poste mal deine klick4

[SilentKiller]

meine klick4.php VOR dem Einbau der Bannersortierung, also
ohne die Änderungen die gemacht werden sollten:

<?userstatus ();?>
<?
$wartung = "0";

head ('Paidbanner');
if($wartung == '1') {
echo "Klick4 Bereich kurzzeitig in Wartung.";
}
else {
if (!isset($_GET['blacklist'])) $_GET['blacklist'] = '';

if ($_GET['blacklist']=='true' && $_GET['tan']) {
db_query("INSERT INTO ".$db_prefix."_blackliste (uid,tan,time) VALUES ('".$_SESSION['uid']."','".$_GET['tan']."','".time()."')");
$sperrzeit_tage = 14; // Hier bitte angeben wieviel Tage der Banner in Reload sein soll
$sperrzeit_tag  = 86400; // Diesen Wert bitte nicht �ndern
$sperrzeit      = $sperrzeit_tage * $sperrzeit_tag;
$new_reload = time() + $sperrzeit;
db_query("INSERT INTO ".$db_prefix."_reloads (ip,uid,tan,bis) VALUES ('".$ip."','".$_SESSION['uid']."','".$_GET['tan']."','".$new_reload."')");
}

$usr = mysql_fetch_assoc (db_query ('SELECT * FROM '.$db_prefix.'_userdaten WHERE uid = '.$_SESSION['uid'].' LIMIT 1'));


$fbanner = db_query ('SELECT t1.*
          FROM vms_gebuchte_werbung t1
          LEFT JOIN vms_reloads t2
          ON (t1.tan = t2.tan AND (t2.uid = '.$_SESSION['uid'].' OR t2.ip = "'.$ip.'"))
          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']);


while ($f_banner = mysql_fetch_assoc ($fbanner)){
    echo '<div align="center" id="banner_'.$f_banner['tan'].'"><a href="fc.php?tan='.$f_banner['tan'].'" target="_blank"><img src="'.$f_banner['banner'].'" border="0" height="60" width="468" alt="'.$f_banner['tan'].'" onClick="document.getElementById(\'banner_'.$f_banner['tan'].'\').style.display=\'none\';"></a>'
    . '<br>Reload:'. $f_banner['reload']/3600 .' Std. | Verdienst: '.number_format($f_banner['verdienst'],2,',','.').' | Aufenthalt: '.$f_banner['aufendhalt'].' Sekunden | <a href="?content=/verdienen/klick4&blacklist=true&tan='.$f_banner['tan'].'&user='.$_SESSION['uid'].'" target="_self"><font color="red"><b>Banner Melden</b></font> </a><br><br></div>';
}  

if (mysql_num_rows ($fbanner) > 0) echo '<div align="center"><input type="button" name="mehr_banner" value="Weitere Banner !" onclick="javascript:location.reload();" /></div>';  
else echo '<div style="text-align: center; font-weight: bold; color: #ff0000;">Alle Banner im Reload !</div>';
}
foot ();
?>

Das habe ich dann versucht abzuändern, und wohl falsch gemacht so:

<?userstatus ();?>
<?
$wartung = "0";

head ('Paidbanner');
if($wartung == '1') {
echo "Klick4 Bereich kurzzeitig in Wartung.";
}
else {
if (!isset($_GET['blacklist'])) $_GET['blacklist'] = '';

if ($_GET['blacklist']=='true' && $_GET['tan']) {
db_query("INSERT INTO ".$db_prefix."_blackliste (uid,tan,time) VALUES ('".$_SESSION['uid']."','".$_GET['tan']."','".time()."')");
$sperrzeit_tage = 14; // Hier bitte angeben wieviel Tage der Banner in Reload sein soll
$sperrzeit_tag  = 86400; // Diesen Wert bitte nicht �ndern
$sperrzeit      = $sperrzeit_tage * $sperrzeit_tag;
$new_reload = time() + $sperrzeit;
db_query("INSERT INTO ".$db_prefix."_reloads (ip,uid,tan,bis) VALUES ('".$ip."','".$_SESSION['uid']."','".$_GET['tan']."','".$new_reload."')");
}

$usr = mysql_fetch_assoc (db_query ('SELECT *, var, vonu FROM '.$db_prefix.'_userdaten WHERE uid = '.$_SESSION['uid'].' LIMIT 1'));

$fbanner = db_query ('SELECT t1.*
          FROM vms_gebuchte_werbung t1
          LEFT JOIN vms_reloads t2
          ON (t1.tan = t2.tan AND (t2.uid = '.$_SESSION['uid'].' OR t2.ip = "'.$ip.'"))
          
          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']);

while ($f_banner = mysql_fetch_assoc ($fbanner)){
    echo '<div align="center" id="banner_'.$f_banner['tan'].'"><a href="fc.php?tan='.$f_banner['tan'].'" target="_blank"><img src="'.$f_banner['banner'].'" border="0" height="60" width="468" alt="'.$f_banner['tan'].'" onClick="document.getElementById(\'banner_'.$f_banner['tan'].'\').style.display=\'none\';"></a>'
    . '<br>Reload:'. $f_banner['reload']/3600 .' Std. | Verdienst: '.number_format($f_banner['verdienst'],2,',','.').' | Aufenthalt: '.$f_banner['aufendhalt'].' Sekunden | <a href="?content=/verdienen/klick4&blacklist=true&tan='.$f_banner['tan'].'&user='.$_SESSION['uid'].'" target="_self"><font color="red"><b>Banner Melden</b></font> </a><br><br></div>';
}  

if (mysql_num_rows ($fbanner) > 0) echo '<div align="center"><input type="button" name="mehr_banner" value="Weitere Banner !" onclick="javascript:location.reload();" /></div>';  
else echo '<div style="text-align: center; font-weight: bold; color: #ff0000;">Alle Banner im Reload !</div>';
}

<!-- Diese Addon basiert auf vms1.1 wurde jedoch auf vms1.2 getestet ,das entfernen dieses Copyrighthinweises ohne Erlaubnis zieht
rechtliche Schritte mit sich
-->
<div align="center"><font size="1">Bannersortierung &copy; by <a href="http://www.vms-tutorial.de" target="_blank">adi</a></font></div>

foot ();
?>