[pLog-svn] r4572 - plog/branches/lifetype-1.1.5/class/summary/dao

[Oscar Renalias]

Is revision 4640 better?

http://devel.lifetype.net/svn/plog/plog/branches/lifetype-1.1.5/class/ 
summary/dao/summarystats.class.php

On 29 Jan 2007, at 01:01, Oscar Renalias wrote:

> I need to take a look, I guess we're not filtering based on the
> status of the blog...
>
> On 28 Jan 2007, at 21:08, BalearWeb wrote:
>
>> I think there is something wrong with this patch.
>> I tested it (having applied all the previous patches too),
>> and in the list of most active blogs I just get the 10 most
>> new blogs, including those which have not yet been confirmed,
>> that is with status=3
>>
>> oscar at devel.lifetype.net wrote:
>>> Author: oscar
>>> Date: 2007-01-24 21:12:48 +0000 (Wed, 24 Jan 2007)
>>> New Revision: 4572
>>>
>>> Modified:
>>>    plog/branches/lifetype-1.1.5/class/summary/dao/
>>> summarystats.class.php
>>> Log:
>>> This is a simplified version of the getMostActiveBlogs() method,
>>> as apparently it hasn't been generating data properiy. It's not as
>>> complex as before but I wasn't able to come up with something as
>>> complex as before... don't know, perhaps it isn't even needed as
>>> long as the last_update_date field is updated accordingly (this
>>> new version is useless without that field...)
>>>
>>> Modified: plog/branches/lifetype-1.1.5/class/summary/dao/
>>> summarystats.class.php
>>> ===================================================================
>>> --- plog/branches/lifetype-1.1.5/class/summary/dao/
>>> summarystats.class.php	2007-01-24 21:10:38 UTC (rev 4571)
>>> +++ plog/branches/lifetype-1.1.5/class/summary/dao/
>>> summarystats.class.php	2007-01-24 21:12:48 UTC (rev 4572)
>>> @@ -179,7 +179,7 @@
>>>  			include_once( PLOG_CLASS_PATH."class/dao/blogs.class.php" );
>>>  		
>>>  			$prefix = $this->getPrefix();
>>> -            $query = "SELECT num_posts, last_update_date FROM
>>> {$prefix}blogs
>>> +            $query = "SELECT id, num_posts, last_update_date FROM
>>> {$prefix}blogs
>>>                        WHERE show_in_summary = '1'
>>>  					  AND last_update_date >= ".$this->_startTime." AND
>>> last_update_date <= ".$this->_now."
>>>  					  ORDER BY last_update_date DESC, num_posts DESC";
>>> @@ -198,7 +198,7 @@
>>>              $blogs = Array();
>>>  			$blogsDao = new Blogs();
>>>              while( $row = $result->FetchRow()) {
>>> -                $blog = $blogsDao->getBlogInfo( $row["blog_id"] );
>>> +                $blog = $blogsDao->getBlogInfo( $row["id"] );
>>>                  $blogs[$blog->getId()] = $blog;
>>>              }
>>>
>>
>>
>> _______________________________________________
>> pLog-svn mailing list
>> pLog-svn at devel.lifetype.net
>> http://limedaley.com/mailman/listinfo/plog-svn
>>
>
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://limedaley.com/mailman/listinfo/plog-svn
>