[pLog-svn] r4606 - plog/branches/lifetype-1.2/class/dao

[oscar at devel.lifetype.net]

Author: oscar
Date: 2007-01-28 08:30:52 -0500 (Sun, 28 Jan 2007)
New Revision: 4606

Modified:
   plog/branches/lifetype-1.2/class/dao/articles.class.php
Log:
Fixed an issue with Articles::updateArticleNumReads()

Modified: plog/branches/lifetype-1.2/class/dao/articles.class.php
===================================================================
--- plog/branches/lifetype-1.2/class/dao/articles.class.php	2007-01-28 11:39:17 UTC (rev 4605)
+++ plog/branches/lifetype-1.2/class/dao/articles.class.php	2007-01-28 13:30:52 UTC (rev 4606)
@@ -1011,18 +1011,12 @@
         {
             require_once( PLOG_CLASS_PATH . 'class/database/db.class.php' );
 
-            $article  = $this->getArticle( $articleId );
-            $numReads = $article->getNumReads() + 1;
-            $article->setNumReads( $numReads );
+            $query = "UPDATE ".$this->getPrefix()."articles SET ".
+                     " num_reads = num_reads+1, date = date".
+			         " WHERE article_id = '".Db::qstr( $articleId )."'";
 
-			$query = "UPDATE ".$this->getPrefix()."articles SET num_reads = {$numReads} ".
-			         "WHERE article_id = ".Db::qstr( $articleId )."'";
-
             $result = $this->Execute( $query );
 
-            if( $result )
-                $this->_cache->setData( $articleId, CACHE_ARTICLES, $article );
-
             return $result;
         }
 		
@@ -1036,8 +1030,7 @@
         {
             // we have to build up the query, which will be pretty long...
             $query = "UPDATE ".$this->getPrefix()."articles SET ".
-                     " num_reads = num_reads+1 ".
-                     ", date = date ".
+                     " num_reads = num_reads+1, date = date".
                      " WHERE slug = '".Db::qstr($articleName)."'";
 
             $result = $this->Execute( $query );