X-Git-Url: http://git.roojs.org/?p=Pman.Core;a=blobdiff_plain;f=DataObjects%2FEvents.php;h=c89d84ec4c014f00a7bd7353ab7c52ab600c73f4;hp=372533cbbaffb5a43aad1d52a8ca021e509c7be4;hb=bbbf9583c92462e5ba51a489fb0870981b602245;hpb=5c458c0ecb482b5c19fbf977e71718ac334e3f93 diff --git a/DataObjects/Events.php b/DataObjects/Events.php index 372533cb..c89d84ec 100644 --- a/DataObjects/Events.php +++ b/DataObjects/Events.php @@ -107,13 +107,6 @@ class Pman_Core_DataObjects_Events extends DB_DataObject } */ - if (!$au->hasPerm("Admin.Admin_Tab", 'S')) { - //DB_DataObject::DebugLevel(1); - // they can only view their changes.. - $this->whereAdd("($tn.person_id = {$au->id} OR $tn.person_id = 0)"); -// $this->person_id = $au->id; - - } // _join = tablename,tablename... /// on_table=cohead @@ -218,7 +211,7 @@ class Pman_Core_DataObjects_Events extends DB_DataObject $roo->jerr("ontable is invalid"); } - if (!method_exists($obj,'relatedWhere') && !method_exists($obj,'relatedEventsWhere')) { + if (!method_exists($obj,'relatedWhere')) { $roo->jerr( $q['_related_on_table'] . " Does not have method relatedWhere or relatedEventsWhere:" . implode(',', get_class_methods($obj))); } @@ -243,23 +236,8 @@ class Pman_Core_DataObjects_Events extends DB_DataObject $this->whereAdd(implode(' OR ' , $w)); } - if ($obj && method_exists($obj,'relatedEventsWhere')) { - $a = $obj->relatedEventsWhere($q,$roo); - if($a){ - $this->whereAdd($a); - } - } - - - + - } - if (isset($q['_who'])) { - - $e = PDO_DataObject::factory('ext_data'); - $this->joinAdd($e,'LEFT'); - //$this->autoJoinCorePerson(); - $this->selectAddWho(); } // since roo does not support autojoin yet.. @@ -278,8 +256,16 @@ class Pman_Core_DataObjects_Events extends DB_DataObject } } - + function applyPermissionFilters($q, $au ,$roo) + { + if (!$au->hasPerm("Admin.Admin_Tab", 'S')) { + //DB_DataObject::DebugLevel(1); + // they can only view their changes.. + $this->whereAdd("($tn.person_id = {$au->id} OR $tn.person_id = 0)"); +// $this->person_id = $au->id; + } + } /** * check who is trying to access this. false == access denied.. @@ -322,13 +308,13 @@ class Pman_Core_DataObjects_Events extends DB_DataObject $ff = HTML_FlexyFramework::get(); $pg = $ff->page; $au = $pg->getAuthUser(); - + if ($ff->cli) { // && empty($au) && isset($obj->person_id)) { $au = false; // $au = DB_DataObject::Factory('Person'); // not always a person.. // $au->get($obj->person_id); } - + $this->person_name = $au && !empty($au->name) ? $au->name : ''; //print_r($au); //exit; @@ -361,7 +347,11 @@ class Pman_Core_DataObjects_Events extends DB_DataObject $rem[] = $obj->toEventString(); } } - $rem[] = $remarks; + + if(!empty($remarks)){ + $rem[] = $remarks; + } + $this->remarks = implode(' : ', $rem); } @@ -497,14 +487,16 @@ class Pman_Core_DataObjects_Events extends DB_DataObject $user = getenv('USERNAME'); // windows. } - // DEPRICATED... + + + if (!empty($ff->Pman['storedir'])) { + return $ff->Pman['storedir'] .'/Events/'.$user; + } + // DEPRICATED... if (!empty($ff->Pman['event_log_dir'])) { return $ff->Pman['event_log_dir'] . '/'.$user; } - if (!empty($ff->Pman['storedir'])) { - return $ff->Pman['storedir'] .'/Events/'.$user; - } return false; } @@ -513,6 +505,7 @@ class Pman_Core_DataObjects_Events extends DB_DataObject function writeEventLog($extra_data = false) { $logdir = $this->logDir(); + if (!$logdir) { return false; } @@ -524,7 +517,8 @@ class Pman_Core_DataObjects_Events extends DB_DataObject @mkdir(dirname($file),0700,true); // this might fail if it does not have correct permissions.. if (!file_exists(dirname($file))) { - die("could not create $file - permissons are not correct"); // fatal, otherwise we loop!? + //print_r($this); + die("Events:: writeEventLog: could not create $file - permissons are not correct\n"); // fatal, otherwise we loop!? } } @@ -627,7 +621,7 @@ class Pman_Core_DataObjects_Events extends DB_DataObject function retrieveEventLog() { - + $logdir = $this->logDir(); if (!$logdir) { return false; @@ -723,25 +717,5 @@ class Pman_Core_DataObjects_Events extends DB_DataObject } - function autoJoinExtData() - { - $this->_join .= "LEFT JOIN - ext_data AS ext_data_id - ON - Events.modx_users_id = ext_data_id.userdata_id"; - } - function selectAddWho() - { - $this->selectAdd("join_person_id_userdata_id.in_firstname, join_person_id_userdata_id.in_middlename, join_person_id_userdata_id.in_lastname"); - - //$this->selectAdd(" - // CASE - // WHEN ext_data_id.in_middlename='' THEN - // CONCAT_WS(' ', ext_data_id.in_firstname,ext_data_id.in_lastname) - // ELSE - // CONCAT_WS(' ', ext_data_id.in_firstname,ext_data_id.in_middlename,ext_data_id.in_lastname) - // END AS ext_data_full_name - //"); - } }