On Wed, Jan 4, 2012 at 12:10 AM, IAlex <ialex.wiki(a)gmail.com> wrote:
I added those fields to the Title class to avoid some
DB queries and avoid
code
duplication.
*nod* they're just not generally applicable to many circumstances, and
really are page-data not title-data to begin with.
The problem is that currently the WikiPage instance
created in
Wiki.php is not stored in the context object (you can grep for
"WikiPage::factory"
to see how many times that method is called). This means that every time
another
instance of that class is created there will be a new database query. I
think we
need to store the WikiPage object in RequestContext to be able to share
that
object and thus avoiding those database queries.
That sounds like it would be reasonable; any objections?
-- brion