phpOpenTracker/phpOpenTracker/API Plugin.php,1.33,1.34

Sebastian Bergmann <[email protected]> Thu, 27 May 2004 05:53:54 +0000
Newsgroups gmane.comp.web.phpopentracker.cvs
Message-ID <[email protected]>
Update of /cvsroot/phpopencounter/phpOpenTracker/phpOpenTracker/API
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv16214/phpOpenTracker/API

Modified Files:
	Plugin.php 
Log Message:
De-normalize database schema.

Index: Plugin.php
===================================================================
RCS file: /cvsroot/phpopencounter/phpOpenTracker/phpOpenTracker/API/Plugin.php,v
retrieving revision 1.33
retrieving revision 1.34
diff -u -d -r1.33 -r1.34
--- Plugin.php	1 Mar 2004 08:56:57 -0000	1.33
+++ Plugin.php	27 May 2004 05:53:51 -0000	1.34
@@ -39,108 +39,15 @@
         $this->database      = phpOpenTracker_Database_Factory::getInstance();
     }
 
-    protected function getConstraintClause($constraints, $selfJoinPossiblyRequired = false) {
-        $constraint       = '';
-        $selfJoinRequired = false;
-
-        foreach ($constraints as $field => $value) {
-            switch ($field) {
-                case 'document': {
-                    $constraint .= sprintf(
-                      " AND accesslog%s.document_id = %d",
-
-                      ($selfJoinPossiblyRequired) ? '2' : '',
-                      $value,
-                      ($selfJoinPossiblyRequired) ? '2' : ''
-                    );
-
-                    if ($selfJoinPossiblyRequired) {
-                        $selfJoinRequired = true;
-                    }
-                }
-                break;
-
-                case 'entry_document': {
-                    $constraint .= sprintf(
-                      " AND accesslog%s.document_id = %d AND accesslog%s.entry_document = 1",
-
-                      ($selfJoinPossiblyRequired) ? '2' : '',
-                      $value,
-                      ($selfJoinPossiblyRequired) ? '2' : ''
-                    );
-
-                    if ($selfJoinPossiblyRequired) {
-                        $selfJoinRequired = true;
-                    }
-                }
-                break;
-
-                case 'exit_document': {
-                    $constraint .= sprintf(
-                      ' AND accesslog%s.document_id = %d AND accesslog%s.exit_target_id <> 0',
-
-                      ($selfJoinPossiblyRequired) ? '2' : '',
-                      $value,
-                      ($selfJoinPossiblyRequired) ? '2' : ''
-                    );
-
-                    if ($selfJoinPossiblyRequired) {
-                        $selfJoinRequired = true;
-                    }
-                }
-                break;
-
-                case 'exit_target': {
-                    $constraint .= sprintf(
-                      ' AND accesslog%s.exit_target_id = %d',
-
-                      ($selfJoinPossiblyRequired) ? '2' : '',
-                      $value
-                    );
-
-                    if ($selfJoinPossiblyRequired) {
-                        $selfJoinRequired = true;
-                    }
-                }
-                break;
-
-                case 'host':
-                case 'operating_system':
-                case 'referer':
-                case 'user_agent': {
-                    $constraint .= sprintf(
-                      ' AND visitors.%s_id = %d',
-
-                      $field,
-                      $value
-                    );
-                }
-                break;
-
-                case 'hour':
-                case 'weekday': {
-                    $constraint .= sprintf(
-                      ' AND accesslog.%s = %d',
+    protected function getConstraintClause($constraints) {
+        $constraint = '';
 
-                      $field,
-                      $value
-                    );
-                }
-                break;
-            }
-        }
+        // ...
 
-        if ($selfJoinPossiblyRequired) {
-            return array(
-              $constraint,
-              $selfJoinRequired
-            );
-        } else {
-            return $constraint;
-        }
+        return $constraint;
     }
 
-    protected function getTimerangeClause($start, $end, $table = 'visitors') {
+    protected function getTimerangeClause($start, $end, $table = 'visits') {
         $table    .= '.';
         $timerange = ' AND ';
 



-------------------------------------------------------
This SF.Net email is sponsored by: Oracle 10g
Get certified on the hottest thing ever to hit the market... Oracle 10g. 
Take an Oracle 10g class now, and we'll give you the exam FREE.
http://ads.osdn.com/?ad_id=3149&alloc_id=8166&op=click