We should add a drush command to view the summary.

Comments

malc0mn’s picture

malc0mn commented

Yes, we need Drush integration! First implementation done (clone the 7.x-1.x branch to have a peek). Still need to:

- fix these weird errors (don't even seem relevant to what I'm doing!?):

  Invalid argument supplied for foreach() tablesort.inc:204                                                                                   [warning]
  reset() expects parameter 1 to be array, null given tablesort.inc:218                                                                       [warning]
  Invalid argument supplied for foreach() tablesort.inc:204                                                                                   [warning]
  reset() expects parameter 1 to be array, null given tablesort.inc:218                                                                       [warning]
  Invalid argument supplied for foreach() tablesort.inc:245                                                                                   [warning]

- add detail page
- clean up the code so as not to duplicate the whole page rendering functions if possible. They are almost the same as in the performance.module file.

malc0mn’s picture

malc0mn commented
Assigned: Unassigned » malc0mn
Status: Active » Needs work

Aaah what the heck, finished it off... Clone 7.x-1.x branch to test.

Still need to do cleanup...

malc0mn’s picture

malc0mn commented
Status: Needs work » Fixed

Finished implementation for D7 and made fully compatible for D6 so only one performance.drush.inc file needs to be maintained.

Available in the next release. Will be a beta to allow ppl to check the drush integration first.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.