Verständnisfrage php

[Darezettl]

Code: Alles auswählen

if ($this->uid == $userdata['user_id'])
	{
		$sql = "UPDATE " . USERS_TABLE . "
			SET user_new_album_imgx = 1
			WHERE user_allow_new_img_popupx = 1
			AND user_id <> ".$userdata['user_id'];
		if (!$db->sql_query($sql))
		{
			message_die(GENERAL_ERROR, 'Could not update popup status for gb entry', '', __LINE__, __FILE__, $sql);
		}}

hilfe!
ich versuche gerade diesen code zu verstehen, bitte helft mir mal und sagt mir ob das so stimmt:


also: "wenn die user id und die id des users, in dessen gästebuch eine neue nachricht geschrieben wurde die selbe sind, dann
zeige diesem user beim nächsten login eine popup-benachrichtigung darüber, sofern er das so will." stimmt das bisher so?
nur das verstehe ich nicht: AND user_id <> ".$userdata['user_id'];
was macht das?

und wäre der code auch an der richtigen stelle (siehe datei)?

[ Attachment gelöscht am Fr 02.Feb, 2007 15:01 von oxpus ]

[oxpus]

Die SQL-Anfrage passt nicht so recht zur Anforderung.
Denn "user_new_album_imgx" erscheint mir kein Eintrag von einem Gästebuch zu sein...

Ansonsten wäre die Logik auch an sich nicht korrekt:
Der User des Gästebuchs will ja informiert werden, also müsste die Bedingung ja <> und nicht == lauten, denn der aktuelle User soll ja eben nicht über das Gästebuch eines anderen Users informiert werden.
Dazu ist in der Abfrage selber user_id <> $userdata['user_id'] auch eher durch user_id = $this_uid zu ersetzen, denn nur der eine User soll ja über neue Einträge informiert werden, oder?

[Darezettl]

ja also ich hab mir halt deinen MOD "Popup on new album images" zur Vorlage genommen und nehme exakt die selben Variablen /Dateien her, nur immer noch ein "x" hinter jeder!

Und der Code den ich da oben gepostet hab, ist eben der, der bei deinem Popup MOD in die album_upload.php / album_modcp.php kommt.
Nur muss der eben für mein Vorhaben in die Datei von dem Profil Gästebuch.
Also der Code soll halt folgendes machen, ich sags mal in Worten:

"Wenn die UserID und die ID des Users, in dessen Gästebuch ein neuer Eintrag geschrieben wurde, gleich sind und dieser User in seinem Profil aktiviert hat, dass er bei neuen Einträgen in seinem Gästebuch ein PopUp erhalten möchte, dann setze bei einem neuen Eintrag auf seinem Gästebuch das PopUp auf *aktiv*"
Genau das will ich haben. Kannst mir des schnell schreiben oxpus?

Wie gesagt ich nehm die selben Variablen wie bei deinem andern PopUp MOD her, nur mit einem x noch dahinter..
Und kuck bitte in die angehängte Datei, ob ich den (noch falschen) Code aber wenigsten schon an der richtigen Stelle platziert hab!

thanks

[oxpus]

Kannst mir des schnell schreiben oxpus?

Lass einfach die Bedingung weg, ersetze dann in der Abfrage die Zeile

Code: Alles auswählen

		AND user_id <> ".$userdata['user_id'];

durch

Code: Alles auswählen

		AND user_id == " . $this_uid . "
		AND user_id <> " . $userdata['user_id'];

Dann ist es fertig

[Darezettl]

ok, also hab nun den mod komplett eingebaut.
es funktioniert auch prima, man kann schön einstellen ob man das popup will oder nicht, nur wenn man in einem gästebuch was schreibt kommt nach dem absenden des posts Could not update popup status for new gb entry, also die fehlermeldung für den code.
(die nachricht an sich die man ins gäsebuch schreiben wollte wird aber in die DB eingetragen!)
ich habs aber genauso gemacht wie du gesagt hast oxpus. bitte schau dir die datei mal an, und sag mir was ich falsch gemacht habe!

ahja: statt der erweiterung "x" habe ich jetzt doch lieber überall "2" genommen.

[ Attachment gelöscht am Fr 02.Feb, 2007 15:02 von oxpus ]

[oxpus]

Kleiner Lapsus von mir:
Zeile

Code: Alles auswählen

AND user_id == " . $this_uid . "

muss natürlich so lauten:

Code: Alles auswählen

AND user_id = " . $this->uid . "

[Darezettl]

super, jetzt gehts einwandfrei!
ich danke dir!

dein album popup mod ist klasse als vorlage wenn man noch für andere sachen popups machen will, hehe

[oxpus]

dein album popup mod ist klasse als vorlage wenn man noch für andere sachen popups machen will, hehe

Meine Vorlage war die PM-Popup-Funktion des phpBB

[AmigaLink]

dein album popup mod ist klasse als vorlage wenn man noch für andere sachen popups machen will, hehe

Meine Vorlage war die PM-Popup-Funktion des phpBB