Logdatei für einen Datensatz schreiben

imsadi

Guten Morgen,

ich habe einen AFTER trigger auf die Relation reklamationen_fuer_reklamationsaufnahme mit folgendem Inhalt erstellt:

$BODY$
DECLARE
textsonst_dateien TEXT;
status_update TEXT;
sonst_dateien TEXT;
login_id TEXT;
timestamp text := current_timestamp;

BEGIN
SELECT text_sonst_dateien into textsonst_dateien
from {xxx}.[relationen_fuer_reklamationsaufnahme];
SELECT status into status_update
from {xxx}.[relationen_fuer_reklamationsaufnahme];
SELECT (lsw_organizations.contact.surname) INTO login_id
FROM {lsw_organizations}.[contact]
where id = (
select instance.get_login_id()
from {xxx}.[relationen_fuer_reklamationsaufnahme]
Where (xxx.relationen_fuer_reklamationsaufnahme.id) = NEW.(xxx.relationen_fuer_reklamationsaufnahme.id));
Insert into {xxx}.[logfile_update] (login_id, timestamp) VALUES (login_id, timestamp);

RETURN NEW;

END;
$BODY$

Das funktioniert soweit, jedoch wird das logfile für alle eingegebenen Datensätze angezeigt und nicht explizit für den, der vorher editiert wurde. Weiterhin wird die login_id ausgelesen, aber ein anderer Name angezeigt, als der, der eingeloggt ist und die Änderung vorgenommen hat.

Wo muss ich anders ansetzen ?

gabriel

Da sind einige Punkte zu adressieren:

SELECTs von der Relation, wo der Trigger zieht (also reklamationen_fuer_reklamationsaufnahme) sind nicht notwendig. Alle Daten findest du bereits in der Variable NEW.
- NEW.text_sonst_dateien beinhaltet den entsprechenden Attributwert vom betroffenen Datensatz.
- Obwohl du hier auf jeden Fall zum Platzhalter wechseln solltest. Der sollte so aussehen (xxx.relationen_fuer_reklamationsaufnahme.text_sonst_dateien) und wird automatisch ausgewählt, wenn du das Platzhaltermenü auf der rechten Seite nutzt. Ohne Platzhalter kann deine Funktion kaputt gehen, wenn du Attribute umbenennst.
SELECTs ohne WHERE holen ALLE(!) Daten aus der Datenbank. Das ist nicht hilfreich, weil du die Daten schon in NEW hast (z. B. in NEW.(xxx.relationen_fuer_reklamationsaufnahme.text_sonst_dateien)) und weil du einen bestimmten Datensatz haben möchtest. Hier müsstest du auf die ID des Datensatzes filtern.
Beim Organisationskontakt kriegst du den falschen Kontakt weil du die ID vom Kontakt mit der Login-ID vergleichst. Das richtige Attribute wäre (lsw_organizations.contact.login).

Wenn du lernen möchtest, wie Backendfunktionen arbeiten, hier sind ein paar Tutorials / die offizielle Dokumentation:

imsadi

gabriel

Hallo Gabriel,

danke für die Hinweise - ich erhalte nun die korrekten Daten aus den Tabellen, lediglich der Part mit
"und weil du einen bestimmten Datensatz haben möchtest. Hier müsstest du auf die ID des Datensatzes filtern."
verstehe ich nicht so ganz.
In einem Erfassungsformular - reklamationsbericht - wurden Daten erhoben und als Datensätze in (relationen_fuer_reklamationsaufnahme) abgespeichert, Klickt ein User auf einen Datensatz , dann öffnet sich ein neues Formular, in dem er Daten nachträglich eingeben kann, die vorher nicht bekannt waren. Da ich das protokollieren möchte in der Relation (log_update) ist mir nicht klar, wie ich das Protokoll lediglich mit diesem geänderten Datensatz "verbinde", damit die Änderungen auch nur bei diesem speziellen Datensatz angezeigt wird.
Bsp:
.....
BEGIN
NEW.(xxx.relationen_fuer_reklamationsaufnahme.text_sonst_dateien) into inhalt_textsonstdateien --> liefert den korrekten Inhalt.
wenn ich das so erweitere:
NEW.(xxx.relationen_fuer_reklamationsaufnahme.text_sonst_dateien) into inhalt_textsonstdateien
where id = (
select (xxx.relationen_fuer_reklamationsaufnahme.id)
from (xxx.relationen_fuer_reklamationsaufnahme);
--> kommt eine Fehlermeldung : request data set failure (login id19, Error column "id" does not exist.
Oder muss das noch erweitert werden mit bspw.:
Where (xxx.relationen_fuer_reklamationsaufnahme.id) = NEW.(xxx.relationen_fuer_reklamationsaufnahme.id));

Fragen über Fragen ...

gabriel

Wenn du NEW verwendest, brauchst du keine WHERE-Abfrage, weil du bereits den aktuell betroffnen Datensatz in der Hand hast. Das wäre aber bei einigen deiner SELECT-Abfragen in deinem ursprünglichen Post notwendig gewesen - weil du dort alle Daten aus der Datenbank abgefragt hast (weil kein WHERE-Filter). Wie gesagt, mit NEW (was den Datensatz der aktuellen Relation beinhaltet), brauchst du das nicht.

Wie du dein Log mit dem betroffenen Datensatz verbindest, ist entsprechend einfach: Mit einer n:1-Beziehung (Beziehungsattribut auf deiner Log-Relation, hin zu deiner Datenrelation). Wenn du in der Beziehung dann auch die Option CASCADE verwendest, werden Logs auch sauber aufgeräumt, wenn der originale Datensatz gelöscht wird - natürlich nur, wenn das gewollt ist.

imsadi

Danke für Deine Geduld Gabriel,

ich hatte das jetzt so versucht:
$BODY$
DECLARE
inhalt_textsonstdateien TEXT;
inhalt_status Integer;

inhalt_user Integer;
BEGIN
Select new.(xxx..relationen_fuer_reklamationsaufnahme.text_sonst_dateien) into inhalt_textsonstdateien;
Select new.(xxx.relationen_fuer_reklamationsaufnahme.status) into inhalt_status;
Select instance.get_login_id() into inhalt_user;
insert into {xxx}.[log_update] (log_sonst_dateien, status, log_user) VALUES (inhalt_textsonstdateien, inhalt_status,
inhalt_user);
RETURN NEW;
END;
$BODY$

dann habe ich von der log_update relation quasi rückwärts auf die relation_fuer_reklamationsaufnahme eine n:1 Verbindung mit dem Feld status gesetzt (wenn ich dich ober richtig verstanden habe). Die anderen Felder werden oben über den trigger gefüllt. Bin mir nicht sicher, ob die Felder in der log_update Relation identisch lauten müssen zu der in der Datenrelation.

Da es leider nicht so funktioniert , wie ich mir das vorstelle, gebe das Vorhaben mit dem log auf - es muss auch so gehen.

gabriel

Das wäre die korrekte Syntax für einen Trigger:

BEGIN
INSERT INTO {xxx}.[log_update] (
  log_sonst_dateien,
  status,
  log_user
)
VALUES (
  NEW.(xxx.relationen_fuer_reklamationsaufnahme.text_sonst_dateien),
  NEW.(xxx.relationen_fuer_reklamationsaufnahme.status),
  instance.get_login_id()
);
RETURN NEW;
END;
$BODY$

Wie schon ein paar mal erwähnt - NEW beinhaltet alle Werte des neuen/geänderten Datensatzes, kein SELECT notwendig. 😉

imsadi

Danke Gabriel,

das hat mir sehr weitergeholfen - ich hätte das nie so umsetzen können.
Ich musste jedoch noch bei dem Formular der Einzeldatensatzanzeige einen Filter auf die ID des Datensatzes setzen, damit er mir nur die Logeinträge dieses Datensatzes anzeigt. Jetzt muss ich nur noch herausfinden, wie ich den status und die login_id im Log als lesbare Beschreibung umsetzen kann, bspw. für die login_id 19 = Hans und die Status ID 1 = erledigt

gabriel

Du kannst in der Liste für Logeinträge entweder per Join (Relation verbinden) oder per SubQuery Daten wie Vor-/Nachname anzeigen.

imsadi

Hallo Gabriel,

danke für die Hilfe - es funktioniert nun alles so, wie es sein soll :-)