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

[Jon Daley]

 	Hrm.  Just saw the bug again.  I thought this was taken care of. 
There is probably another spot that a similar bug exists.

On Wed, 3 Oct 2007, Jon Daley wrote:

> 	Yeah, I agree.  I don't see how my fix can break anything else,
> since it just postpones the article getting into the cache for some other
> operation, but I don't really know why it fixes it either.
> 	I made this change probably last Saturday or so on the two sites
> that were having this issue, and they have been fine since then.  Where,
> usually, they would have gotten a messed up cache file in one or two days.
>
> On Wed, 3 Oct 2007, Oscar Renalias wrote:
>> I'm not so sure... There's nothing wrong with the fix, it's just that
>> I don't understand it :)
>>
>> On 10/3/07, jondaley at devel.lifetype.net <jondaley at devel.lifetype.net> wrote:
>>> Author: jondaley
>>> Date: 2007-10-03 14:15:27 -0400 (Wed, 03 Oct 2007)
>>> New Revision: 5964
>>>
>>> Modified:
>>>    plog/branches/lifetype-1.2/class/dao/articles.class.php
>>> Log:
>>> This fixes the corrupt cache problem.
>>>
>>> I don't think this is the real fix, as I think this code was probably okay,
>>> but I couldn't figure out where $article was being set to an array()
>>>
>>>
>>>
>>> Modified: plog/branches/lifetype-1.2/class/dao/articles.class.php
>>> ===================================================================
>>> --- plog/branches/lifetype-1.2/class/dao/articles.class.php     2007-10-02 18:55:05 UTC (rev 5963)
>>> +++ plog/branches/lifetype-1.2/class/dao/articles.class.php     2007-10-03 18:15:27 UTC (rev 5964)
>>> @@ -991,7 +991,8 @@
>>>                         RecentArticles::resetRecentArticlesCache( $article->getBlogId());
>>>              $this->_cache->removeData( $article->getBlogId(), CACHE_ARTICLESPERMONTH );
>>>                         $this->_cache->removeData( $article->getId(), CACHE_ARTICLETEXT );
>>> -            $this->_cache->setData( $article->getId(), CACHE_ARTICLES, $article );
>>> +            $this->_cache->removeData( $article->getId(), CACHE_ARTICLES );
>>> +//            $this->_cache->setData( $article->getId(), CACHE_ARTICLES, $article );
>>>                         $this->_cache->removeData( $article->getPostSlug(), CACHE_ARTICLES_BYNAME );
>>>
>>>              return true;
>>>
>>> _______________________________________________
>>> 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
>>
>
> -- 
> Jon Daley
> http://jon.limedaley.com/
>
> Never mess up an apology with an excuse.
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://limedaley.com/mailman/listinfo/plog-svn
>

-- 
Jon Daley
http://jon.limedaley.com/

To understand recursion, you must first understand recursion.