2011/4/29 A.L.E.C alec@alec.pl:
On 26.04.2011 22:01, Digital Dog wrote:
Tutaj proponuję również to ujednolicić i dodać np. element ['status'] typu int zawierający info czy węzeł/węzły zostały włączone czy wyłączone.
Czekamy na patcha. Zgłoś to do BTSu.
Patch jest gotowy, niestety BTS nie przysyła maila z potwierdzeniem rejestracji :(
$cancel = $LMS->ExecHook('node_del_before', $plugin_data); if ($cancel === 1) { $LMS->DeleteNode($nodeid); $LMS->ExecHook('node_del_after', $plugin_data); }
Ja bym node_del_after wyciągnął poza ifa, ale idea jest ok. Zgłoś do BTSu.
Powyżej oczywiście pomylony operator, miało być if ($cancel !== 1)
Jeśli samo usunięcie nie miało miejsca, to nie można wykonać akcji 'after' która (słusznie) założy, że obiekt jednak został usunięty, np.:
function del_before($vars) { if (...) return 1; // anulowanie usunięcia - załóżmy, że komp widnieje w bazie 'ważnych' }
$LMS->DeleteNode() nie jest wywołane
więc komunikat
function del_after($vars) { foreach ($vars['nodes'] as $node) error_log("Usunięto komputer $node"); }
nie miałby sensu.
Proszę o pomoc z tym BTSem.