438707 - Make Cake query debug output prettier

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Justin Scott [:fligtar]]

Attachment: patch, v1

While reviewing bug 427094's patch, I decided Cake's query output is ugly and needed some additional info, so this patch overwrites the default methods with our own.

It makes the query log much prettier and makes it collapsed by default and also displays the database config that each query was run on.

Comment 1

[Justin Scott [:fligtar]]

Attachment: patch, v2

Meh, part of morgamic's patch was in that one.

Comment 2

[Michael Morgan [:morgamic]]

Comment on attachment 324698 [details] [diff] [review]
patch, v2

+r=morgamic - extra points if you can figure out why cake has a hard limit in query debug view of 200.  For example, if we have 294 queries executed, it'll only list 200 of them below even if the total is > 200.  I know that's not your fault though.

Comment 3

[Fred Wenzel [:wenzel]]

I seem to remember it starts over with a second list then? So it shows one list with 200, then another with 94. No?

Comment 4

[Justin Scott [:fligtar]]

From http://svn.mozilla.org/addons/trunk/site/cake/libs/model/datasources/datasource.php

/**
 * Maximum number of items in query log, to prevent query log taking over
 * too much memory on large amounts of queries -- I we've had problems at
 * >6000 queries on one system.
 *
 * @var int Maximum number of queries in the queries log.
 * @access private
 */
	var $_queriesLogMax = 200;

But I'll just remove the if() in my overwrite logQuery() so that it doesn't check that var at all.

Fred - From what I can tell, it creates a list per db config used, so it'll have a list for queries run on master and one for default. We just had no way of knowing which was which.

Comment 5

[Justin Scott [:fligtar]]

Checked in, r15385.