[pLog-svn] r1720 - in plog/branches/plog-1.1-ben/class: action action/admin cache

[Benjamin Krause]

Oscar Renalias wrote:
> I don't quite see the point of the BlogInfoManager class. Why can't it
> be merged with the current Blogs class?
> 
> For example the job of BlogInfoManager::getBlogInfo() could be done by
> Blogs::getBlogInfo() which would take into account the cache and so
> on.
> Any comments?

yes.. i expected that comment :)

the problem with blogs() is, that it extends Model, and calls model(). 
this starts a hugh include process and connects to the db. i want to 
avoid the includes and the connection to the db.. now with the extra 
BlogInfoManager we do not have a connection to the db.

we might add the method to blogs, but then we must remove the

$this->_db =& Db::getDb();

line from the Model() constructor.. but a lot of classes depend on the 
existence of a $this->_db. so removing this line would make things a bit 
  more complicated.. :)  we can look into it if you want to, adding the 
new BlogInfoManager is faster and left the current code untouched.

Ben