[MySQL][PHP]logi Opcje

[kamilo818]

Mym pytanie na temat logow.
Chciałbym aby zapisywały mi się wszystkie działania na bazie danych. Tzn jeśli użytkownik dodał rekord -> log, jeśli usunął -> log, jeśli jakaś operacja się nie powiodła -> log.

Aktualnie robię to tak ze w funkcji która wykonuje operacje na bazie mysql mam kolejna funkcje addLog() która w zależności od tego czy operacja zwróciła false czy true dodaje odpowiedni rekord do bazy.
Dodawana tam są informacje : kto, kiedy, do jakiej tabeli, jaka akcja ( dodanie, usunięcie ), wartość na jaka miała być zmieniona.

W sumie działa to jak oczekuje. Ale może jest jakaś wbudowana funkcja mysql czy php która dzięki której lepiej można gromadzić logi. Albo czy można jakoś to lepiej napisać ? W sensie czy moje podejście do tematu nie jest błędne .

[rad11]

Uzyj do tego wyzwalaczy.

[PHP] pobierz, plaintext 
DROP TRIGGER IF EXISTS `NAZWA_WYZWALACZA`;
SET @OLDTMP_SQL_MODE=@@SQL_MODE, SQL_MODE='STRICT_TRANS_TABLES,NO_ENGINE_SUBSTITUTION';
DELIMITER //
CREATE TRIGGER `NAZWA_WYZWALACZA` AFTER UPDATE ON `NAZWA_SLEDZONEJ_TABELI` FOR EACH ROW BEGIN
 
IF (@user_id IS NULL ) THEN
SET @user_id=0;
END IF;
 
IF OLD.NAZWA_SLEDZONEGO_POLA1<> NEW.NAZWA_SLEDZONEGO_POLA1 THEN
INSERT INTO logs (user_id,object,object_field,object_id,new_value,old_value,date) VALUES (@user_id,'rounds_country','name',OLD.id,NEW.name,OLD.name,NOW()); -- DEFINICJA CO ZAPISUJESZ W TABELI DZIENNIKA
END IF;	
 
END//
DELIMITER ;
SET SQL_MODE=@OLDTMP_SQL_MODE;
[PHP] pobierz, plaintext 

w phpie ustawiasz tylko user_id

[PHP] pobierz, plaintext 
set @user_id = TUTAJ NUMER ID USERA
[PHP] pobierz, plaintext 

Ten post edytował rad11 1.02.2015, 15:19:14

[kamilo818]

Dobra super opcja.

Nie wiem jak ustawic w php set @user_id = TUTAJ NUMER ID USERA. Po wipisaniu tego po prostu

[PHP] pobierz, plaintext 
<?php
set @user_id = TUTAJ NUMER ID USERA
?>
[PHP] pobierz, plaintext 

wywala błędy

Ten post edytował kamilo818 1.02.2015, 20:16:49

[rad11]

Stworz sobie tabele:

[PHP] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `logs` (
 `id` int(10) unsigned NOT NULL AUTO_INCREMENT,
 `user_id` smallint(5) unsigned NOT NULL,
 `object` varchar(50) COLLATE utf8_unicode_ci NOT NULL COMMENT 'nazwa tabeli',
 `object_field` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
 `object_id` int(10) unsigned NOT NULL COMMENT 'id rekordu',
 `new_value` mediumtext COLLATE utf8_unicode_ci,
 `old_value` mediumtext COLLATE utf8_unicode_ci,
 `date` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,
 PRIMARY KEY (`id`),
 KEY `object` (`object`,`object_field`,`object_id`,`date`,`user_id`),
 KEY `user_id` (`user_id`)
) ENGINE=InnoDB  DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci AUTO_INCREMENT=1 ;
[PHP] pobierz, plaintext 

potem uruchom wyzwalacz w phpmyadmin np:

[PHP] pobierz, plaintext 
DROP TRIGGER IF EXISTS `NAZWA_WYZWALACZA`;
SET @OLDTMP_SQL_MODE=@@SQL_MODE, SQL_MODE='STRICT_TRANS_TABLES,NO_ENGINE_SUBSTITUTION';
DELIMITER //
CREATE TRIGGER `NAZWA_WYZWALACZA` AFTER UPDATE ON `NAZWA_SLEDZONEJ_TABELI` FOR EACH ROW BEGIN
 
IF (@user_id IS NULL ) THEN
SET @user_id=0;
END IF;
 
IF OLD.NAZWA_SLEDZONEGO_POLA1<> NEW.NAZWA_SLEDZONEGO_POLA1 THEN
INSERT INTO logs (user_id,object,object_field,object_id,new_value,old_value,date) VALUES (@user_id,'rounds_country','name',OLD.id,NEW.name,OLD.name,NOW()); -- DEFINICJA CO ZAPISUJESZ W TABELI DZIENNIKA
END IF;
 
END//
DELIMITER ;
SET SQL_MODE=@OLDTMP_SQL_MODE;
[PHP] pobierz, plaintext 

a to linijke:

[PHP] pobierz, plaintext 
set @user_id = TUTAJ NUMER ID USERA
[PHP] pobierz, plaintext 

ustawiaj w przed miejscem w ktorym robisz update, insert itd.

Ten post edytował rad11 1.02.2015, 20:22:37

[kamilo818]

[PHP] pobierz, plaintext 
	set @user_id = $_SESSION['nick'];	
	mysqli_query($db, "UPDATE $tabela SET $rekord_zmiana = '$wartosc_zmiana' WHERE $rekord = '$wartosc'");
[PHP] pobierz, plaintext 

tak?
nie działa :/

[rad11]

Oczywiscie ta linijke z set ustaw jako zwykle zapytanie z poziomu php.

[PHP] pobierz, plaintext 
mysqli_query($db, "set @user_id = $_SESSION['nick']");
[PHP] pobierz, plaintext 

tylko czy nick to jest id usera? I przed albo za zapytaniem ale chyba raczej za sorry jak pisalem przed.

Ten post edytował rad11 1.02.2015, 20:56:05

[kamilo818]

Dzięki

A jeszcze pytanie.
Czy mogę zamiast @user_id usawić całą nazwę? Bo w tym przypadku tylko cyfry da sie zapisać pod @user_id

[rad11]

Lepiej jak masz powiazanie poprzez id usera, ale tak mozesz tez zrobic jako nick.

[kamilo818]

ok, ale jak bo w taki sposób nie wychodzi.
pod $_SESSION['nick'] jest np user@o2.pl

[PHP] pobierz, plaintext 
mysqli_query($db, "set @user_id = $_SESSION['nick']");
[PHP] pobierz, plaintext 

//edit

juz wiem

[rad11]

No musisz zmienic w strukturze tabeli user_id na nick i jako varchar.

[kamilo818]

A jest możliwość aby trigger dodawał rekord gdy operacja sie nie powiedzie? Bo teraz zbieram tylko logi z udanymi zmianami w tabeli. A chciałbym jeszcze gdy coś sie nie powiedzie.