Adminforce: Interfacebanner

[Gremlin]

Eben diese PN erhalten:


Habe da ein Fehler im Adminbereich entdeck und behoben
[...]
Nun erstmal zum Fehler *GG*:

Geht um Interfacebanner -> Webmasterlose
Also um die Datei adminforce/content/interfacebanner/webmasterlose.php

In zeile 7 ist da die if anfrage ob löschen auf true usw :

if ($_GET['loeschen']=='true' && $_GET['kid']) {
db_query("DELETE FROM ".$db_prefix."_gebuchte_werbung WHERE kid='".$_GET['kid']."' and sponsor='webmasterlose' LIMIT 1");
}
...........

Aber in Zeile 27 steht dies :

<td align="center" width="50%"><a href="?content=/interfacebanner/webmasterlose&amp;loeschen=true&amp;tan='.$anzeige n['tan'].'" target="_self">Löschen</a>
....................

Was ja bedeutet das bei dem Klick auf löschen die tan und nicht die kid übergeben wird , wenn mich nicht alles täuscht.
Was dazu führt das kein Banner gelöscht wird egal wie oft man den Link klickt.

Gab also 2 möglichkeiten:

Einmal leichteste:
In den link für löschen tan durch kid ersetzen wäre aber schwachsinn weil dann alle banner mit der selben kid gelöscht würden .

Habe folgendes gemacht :

Die Inanfrage von zeile 7 geändert in:

if ($_GET['loeschen']=='true' && $_GET['tan']) {
db_query("DELETE FROM ".$db_prefix."_gebuchte_werbung WHERE tan='".$_GET['tan']."' and sponsor='webmasterlose' LIMIT 1");
}

Nun wird endlich die tan zum löschen (identifizieren des zu löschenden Banners Herrangezogen)


Danke für die Meldung, wird beim nächsten Update berücksichtigt.

Gruß
Gremlin

[Schmuse_Kater40]

Eben diese PN erhalten:



Danke für die Meldung, wird beim nächsten Update berücksichtigt.

Gruß
Gremlin

Habe das gerade mal überprüft. Das ist bei allen anderen Interfacebannerdateien auch der Fall die im VMS1.2.2 integriert sind. Das bitte beim nächsten Update berücksichtigen.

[hecke1987]

Handelte sich auch ums vms 1.2.2 ^^ nur hatte ich nicht alle geprüft ^^aber giebt ja auch andere Leute die aufmerksam mithelfen*gg*

[jpwfour]

und sollte da nicht auch irgendwo ein link "hinzufügen zur blacklist" oder so sein?

[hecke1987]

Sollte wohl mir wäre dabei nur noch nicht ganz klar nach welchem kriterium man es genau darein setzt und wie genau ^^ wenn jemand da ne lösung hat wäre super *gg*

[jpwfour]

naja, framebrecher sind wohl der hauptgrund, und evtl pornographische inhalte etc.

den link hab ich so eingebaut:

<td align="center" width="50%">
<a href="?content=/interfacebanner/gollox&amp;loeschen=true&amp;tan='.$anzeigen['tan'].'" target="_self">Löschen</a> /
<a href="?content=/interfacebanner/gollox&amp;blacklist=true&amp;tan='.$anzeigen['tan'].'" target="_self">Blacklist</a>
</td>

da wo gollox steht natürlich dann den entsprechenden sponsor/dateinamen einsetzen.

und bei

if ($_GET['blacklist']=='true' && $_GET['kid']) {
db_query("UPDATE ".$db_prefix."_gebuchte_werbung SET status = '2' WHERE kid='".$_GET['kid']."' and sponsor='........' LIMIT 1");
}

hab ich auch alles auf tan geändert:


if ($_GET['blacklist']=='true' && $_GET['tan'])
{
db_query("UPDATE ".$db_prefix."_gebuchte_werbung SET status = '2' WHERE tan='".$_GET['tan']."' and sponsor='......' LIMIT 1");
}