User bestätigt mehr Mails als verschickt wurden!?

[Siggi84]

Hallo VMS'ler,

ich habe wieder mal ein Problemchen, mal kurz ausgeholt...

Letzte Woche haben mich 2 Mitglieder angeschrieben das seit 1 Woche ein User immer eine Mail mehr in der Mailrally hat als sie, obwohl Sie immer alle bestätigt haben.

Ist natürlich unfair, somit ist ihm immer der erste Platz sicher.

Nachdem ich unzähliche Backups durchsucht habe und Mail für Mail zusammen gerechnet habe, habe ich bei dem User die Mailanzahl um 1 veringert wie bei den anderen, somit war wieder "Gleichstand".

Da ich über die Feiertage keine verschickt habe, hab ich heute umso genauer hingeschaut, siehe da, der User hat wieder 1 mehr.

Habe 8 Stück verschickt und er hat heute 9 in den Buchungen drin stehen!

Wo kann hier der Fehler liegen bzw. wird hier vielleicht manipulliert ?

Es kann sich auch nicht um eine "alte" Mail handeln, da ich gestern die Datenbank "gesäubert" habe und alles Leer war, die Mails haben eh nur eine Gültigkeit von 2 Tagen, wäre somit auch vorbei.

Ich hoffe das Ihr mir helfen könnt und bedanke mich im vorraus für eure Hilfe.

Gruß

Falk alias Siggi84

[jpwfour]

Hm, wegen 1er Mail

Mail is ja immer bei jedem anders, daher wäre noch wichtig:
Welche Mailrallye?
Wie werden Mails verschickt?

Habe 8 Stück verschickt und er hat heute 9 in den Buchungen drin stehen!

Wie genau stehen die in den Buchungen drin? (TANs, ...)

Evtl. an der Stelle der Rallyefunktion die Daten für den entsprechenden User mitloggen lassen (da er das ja vermutlich weiterhin so macht):

if($mail['uid']==123456){
file_put_contents("adminforce/log.txt", date('d.m. H:i:s')." - ".$_GET['tan']." - ".$mail['verdienst']."\r\n", FILE_APPEND);
} 

Bspw. für den normalen Mailversand und Bestätigung in der pcheck.php

[Siggi84]

Hm, wegen 1er Mail

Je nur wegen einer, aber meine User passen da recht gut auf, vorallem wenn es um Platz 1 geht Sie haben aber ja auch recht!

Mail is ja immer bei jedem anders, daher wäre noch wichtig:
Welche Mailrallye?

Diese Hier, gleich der 2te Artikel (Nr.:197)

Wie werden Mails verschickt?

Manuell direkt über das VMS, also kein Automailer, etc...

Wie genau stehen die in den Buchungen drin? (TANs, ...)

Etwas genaues steht in den Buchungen nicht drin, nur als Verwendungszweck "Paidmailverdienst", kann man hier noch die TAN hinzufügen? Aktuell sieht es so aus:

buchungsliste (create_code(14),$mail['verdienst'],'Paidmailverdienst',$mail['uid']); 

Evtl. an der Stelle der Rallyefunktion die Daten für den entsprechenden User mitloggen lassen (da er das ja vermutlich weiterhin so macht):

if($mail['uid']==123456){
file_put_contents("adminforce/log.txt", date('d.m. H:i:s')." - ".$_GET['tan']." - ".$mail['verdienst']."\r\n", FILE_APPEND);
} 

Bspw. für den normalen Mailversand und Bestätigung in der pcheck.php

Das werde ich morgen gleich mal ausprobieren, wenn ich die nächsten mails verschicke, nur bin ich mir gerade etwas unsicher wo der Code hin muss...

In die pcheck.php unter die ganzen Sachen wie, Rallys, Buchung, APs, Bilanz, etc... ?

Alle außer der eine haben 8 bestätigt, er 9, da ist doch was faul, da es ja bereits letzte Woche genau das gleiche war

Danke wieder im Vorraus !!!

Falk

[Siggi84]

Also mit der Log habe ich schon mal hinbekommen! DANKE!

Jetzt muss ich erst mal abwarten bis der "Herr" die Mails bestätigt.

[Siggi84]

Guten Morgen,

so heute hats geklappt...

Hier die Log des Users:

30.12. 09:27:12 - I1svNXXS14kFku - 7000.00
30.12. 09:27:12 - I1svNXXS14kFku - 7000.00
30.12. 09:27:15 - LDWsWKLhBlRa74 - 7000.00
30.12. 09:27:16 - ejzFlQhAyzZPK1 - 7000.00
30.12. 09:27:18 - 5S6Tkn8t96yUZZ - 7000.00
30.12. 09:27:20 - DZuJE1KoSa3327 - 7000.00
30.12. 09:27:22 - vevWBqyKbaTmWk - 7000.00
30.12. 09:27:25 - CXxVsmkuxELtZE - 7000.00
30.12. 09:27:26 - 0RQ3puDlFOPxU8 - 7000.00

Wie man erkennen kann, wurde die erste Mail 2 mal, exakt zur gleichen Zeit bestätigt und auch Gebucht!

Bin jetzt mal so ziemlich alle User durch gegangen die, die Mails bestätigen, das passiert nur bei diesen "Herrn".

Dachte schon es ist vielleicht ein anderer Fehler der Seite, aber da würde es ja nicht nur bei Ihm passieren!?

Jemand eine Idee wie man das unterbinden kann, bzw. wie das eigentlich möglich ist?

Vielen Dank wieder im Vorraus!

[jpwfour]

Hat der User zu dieser Tan 2 Einträge in der vms_paidmails_empfaenger?
Dann gibts evtl schon beim verschicken ein Problem, dass der User die Mail 2x bekommt.

Welchen Aufenthalt hat die Mail?

[Siggi84]

Hat der User zu dieser Tan 2 Einträge in der vms_paidmails_empfaenger?
Dann gibts evtl schon beim verschicken ein Problem, dass der User die Mail 2x bekommt.

Welchen Aufenthalt hat die Mail?

Mit der Tan, das weiß ich nicht, muss ich morgen schauen wenn ich wieder verschicke. Sind ja nach der Bestätigung weg.

Kann ich mir nur schlecht vorstellen, ist das überhaupt möglich, 2 mal die gleichen Einträge?

Meine Mails haben immer 50sek Aufenthalt!

[jpwfour]

Der Vollständigkeithalber:
es besteht theoretisch die Möglichkeit, sofern das PHP Skript mehrmals "gleichzeitig" aufgerufen wird, dass die Vergütung mehrfach erfolgt, sofern in 2 Fällen das Auslesen der Daten aus der db vor dem Update der entsprechenden Einträge (im jeweils anderen Skriptaufruf) erfolgt (und damit die if Abfrage: "Vergüten ja/nein" beide Male positiv ausfällt).

Das ist aber extrem unwahrscheinlich und kann definitiv nicht "gewollt" herbeigeführt werden, insofern würde sowas nur ein gelegenliches Auftreten einer Doppelbestätigung erklären, und da kann man schon Größenordnung von 1er aus 100.000 mind. annehmen

Sollte tatsächlich ein doppelter Eintrag die Ursache sein, könnte man das denke ich fixen, indem man aus:

db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=1 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid']." LIMIT 1"); 

das Limit 1 weglässt, also:

db_query("UPDATE ".$db_prefix."_paidmails_empfaenger SET status=1 WHERE tan='".$_GET['tan']."' and uid=".$_GET['uid'].""); 

Dann würde bei doppelten TANs jede auf "bestätigt" gesetzt werden (nur für die jewelige UID natürlich), einen Versuch ist es wert (pcheck.php)

[Siggi84]

Danke für deine Hilfe

Da ich jetzt leider auf Arbeit muss, kann ich heute keine Mails verschicken.

Wird also erst im neuen Jahr

Melde mich morgen wieder!

Einen guten rutsch ins Jahr 2011!!!

[Siggi84]

Ein gesundes neues Jahr 2011 !!!

Habe soeben wieder Mails verschickt und jeden einzelnen Eintrag in...

vms_paidmails_empfaenger

... Kontrolliert, sind insgesamt 791 Einträge und keiner davon ist doppelt.

Jetzt heißt es abwarten bis die Mails wieder bestätigt werden, melde mich sofort, sobald sich was ergibt.

Die Änderung an der pcheck.php habe ich vorerst noch nicht gemacht, da es ja keinen doppelten Eintrag gibt, aber theoretisch kann sich das nicht negativ auswirken, eher zum vorteil, oder?

Könnte man ja somit generell abändern, oder?

Schönes ausnüchtern noch