X-Git-Url: http://git.polytechnique.org/?a=blobdiff_plain;f=classes%2Fcorelogger.php;h=e5784c614e583c66d4709bfd51f3a1b0b2a64c6a;hb=fc10da9a3e2cf019a4c6722288e2f063e3ec1728;hp=5797236d6900a374ab783f8e37bf156e631ea3dd;hpb=e76421d8190c87d81d42df45d662eeb7a5f7fd18;p=platal.git

diff --git a/classes/corelogger.php b/classes/corelogger.php
index 5797236..e5784c6 100644
--- a/classes/corelogger.php
+++ b/classes/corelogger.php
@@ -19,13 +19,14 @@
  */
 
 
-class CoreLogger {
+class CoreLogger
+{
     /** user id */
-    var $uid;
+    public $uid;
     /** id of the session */
-    var $session;
+    private $session;
     /** list of available actions */
-    var $actions;
+    private $actions;
 
     /** The constructor, creates a new entry in the sessions table
      *
@@ -33,7 +34,8 @@ class CoreLogger {
      * @param $suid the id of the administrator who has just su'd to the user
      * @return VOID
      */
-    function CoreLogger($uid, $suid='') {
+    public function __construct($uid, $suid = 0)
+    {
         // write the session entry
         $this->uid     = $uid;
         $this->session = $this->writeSession($uid, $suid);
@@ -52,7 +54,7 @@ class CoreLogger {
      * @param $suid the id of the administrator who has just su'd to the user
      * @return session the session id
      */
-    function writeSession($uid, $suid = null)
+    private function writeSession($uid, $suid = 0)
     {
         $ip      = $_SERVER['REMOTE_ADDR'];
         $host    = strtolower(gethostbyaddr($_SERVER['REMOTE_ADDR']));
@@ -69,7 +71,7 @@ class CoreLogger {
         }
 
         XDB::execute("INSERT INTO logger.sessions
-                     SET uid={?}, host={?}, ip={?}, forward_ip={?}, forward_host={?}, browser={?}, suid={?}, flags={?}",
+                         SET uid={?}, host={?}, ip={?}, forward_ip={?}, forward_host={?}, browser={?}, suid={?}, flags={?}",
                      $uid, $host, $ip, $forward_ip, $forward_host, $browser, $suid, $proxy);
 
         return XDB::insertId();
@@ -79,18 +81,20 @@ class CoreLogger {
     /** Logs an action and its related data.
      *
      * @param $action le type d'action
-     * @param $data les données (id de liste, etc.)
+     * @param $data les données (id de liste, etc.)
      * @return VOID
      */
-    function log($action, $data = null) {
+    public function log($action, $data = null)
+    {
         if (isset($this->actions[$action])) {
             XDB::execute("INSERT INTO logger.events
-                         SET session={?}, action={?}, data={?}",
+                             SET session={?}, action={?}, data={?}",
                          $this->session, $this->actions[$action], $data);
         } else {
-            echo "unknown action : $action<br />";
+            trigger_error("CoreLogger: unknown action, $action", E_USER_WARNING);
         }
     }
 }
 
+// vim:set et sw=4 sts=4 sws=4 foldmethod=marker enc=utf-8:
 ?>