jacobemerick / pqp

tests/unit/DisplayTest.php

Indentation +22 added lines, -22 removed lines

@@ -5,29 +5,29 @@
 class DisplayTest extends PHPUnit_Framework_TestCase
 {
 
-    public function testGetReadableTime()
-    {
-        $test_input = array(
-            '.032432' => '32.432 ms',
-            '24.3781' => '24.378 s',
-            '145.123' => '2.419 m'
-        );
+	public function testGetReadableTime()
+	{
+		$test_input = array(
+			'.032432' => '32.432 ms',
+			'24.3781' => '24.378 s',
+			'145.123' => '2.419 m'
+		);
 
-        foreach ($test_input as $input => $expected_return) {
-            $this->assertEquals($expected_return, Display::getReadableTime($input));
-        }
-    }
+		foreach ($test_input as $input => $expected_return) {
+			$this->assertEquals($expected_return, Display::getReadableTime($input));
+		}
+	}
 
-    public function testGetReadableMemory()
-    {
-        $test_input = array(
-            '314'     => '314 b',
-            '7403'    => '7.23 k',
-            '2589983' => '2.47 M'
-        );
+	public function testGetReadableMemory()
+	{
+		$test_input = array(
+			'314'     => '314 b',
+			'7403'    => '7.23 k',
+			'2589983' => '2.47 M'
+		);
 
-        foreach ($test_input as $input => $expected_return) {
-            $this->assertEquals($expected_return, Display::getReadableMemory($input));
-        }
-    }
+		foreach ($test_input as $input => $expected_return) {
+			$this->assertEquals($expected_return, Display::getReadableMemory($input));
+		}
+	}
 }

src/Display.php

Indentation +139 added lines, -139 removed lines

@@ -14,154 +14,154 @@
 class Display
 {
 
-    /** @var  array */
-    protected $output;
+	/** @var  array */
+	protected $output;
 
-    public function __construct()
-    {
-    }
+	public function __construct()
+	{
+	}
 
-    public function setConsole(Console $console)
-    {
-        $console_data = array(
-            'messages' => array(),
-            'count'    => array(
-                'log'    => 0,
-                'memory' => 0,
-                'error'  => 0,
-                'speed'  => 0
-            )
-        );
-        foreach ($console->getLogs() as $log) {
-            switch($log['type']) {
-                case 'log':
-                    $message = array(
-                        'data' => print_r($log['data'], true),
-                        'type' => 'log'
-                    );
-                    $console_data['count']['log']++;
-                    break;
-                case 'memory':
-                    $message = array(
-                        'name' => $log['name'],
-                        'data' => self::getReadableMemory($log['data']),
-                        'type' => 'memory'
-                    );
-                    if (!empty($log['data_type'])) {
-                        $message['data_type'] = $log['data_type'];
-                    }
-                    $console_data['count']['memory']++;
-                    break;
-                case 'error':
-                    $message = array(
-                        'data' => $log['data'],
-                        'file' => $log['file'],
-                        'line' => $log['line'],
-                        'type' => 'error'
-                    );
-                    $console_data['count']['error']++;
-                    break;
-                case 'speed':
-                    $elapsedTime = $log['data'] - $this->startTime;
-                    $message = array(
-                        'name' => $log['name'],
-                        'data' => self::getReadableTime($elapsedTime),
-                        'type' => 'speed'
-                    );
-                    $console_data['count']['speed']++;
-                    break;
-                default:
-                    $message = array(
-                        'data' => "Unrecognized console log type: {$log['type']}",
-                        'type' => 'error'
-                    );
-                    $console_data['count']['error']++;
-                    break;
-            }
-            array_push($console_data['messages'], $message);
-        }
-        $this->output['console'] = $console_data;
-    }
+	public function setConsole(Console $console)
+	{
+		$console_data = array(
+			'messages' => array(),
+			'count'    => array(
+				'log'    => 0,
+				'memory' => 0,
+				'error'  => 0,
+				'speed'  => 0
+			)
+		);
+		foreach ($console->getLogs() as $log) {
+			switch($log['type']) {
+				case 'log':
+					$message = array(
+						'data' => print_r($log['data'], true),
+						'type' => 'log'
+					);
+					$console_data['count']['log']++;
+					break;
+				case 'memory':
+					$message = array(
+						'name' => $log['name'],
+						'data' => self::getReadableMemory($log['data']),
+						'type' => 'memory'
+					);
+					if (!empty($log['data_type'])) {
+						$message['data_type'] = $log['data_type'];
+					}
+					$console_data['count']['memory']++;
+					break;
+				case 'error':
+					$message = array(
+						'data' => $log['data'],
+						'file' => $log['file'],
+						'line' => $log['line'],
+						'type' => 'error'
+					);
+					$console_data['count']['error']++;
+					break;
+				case 'speed':
+					$elapsedTime = $log['data'] - $this->startTime;
+					$message = array(
+						'name' => $log['name'],
+						'data' => self::getReadableTime($elapsedTime),
+						'type' => 'speed'
+					);
+					$console_data['count']['speed']++;
+					break;
+				default:
+					$message = array(
+						'data' => "Unrecognized console log type: {$log['type']}",
+						'type' => 'error'
+					);
+					$console_data['count']['error']++;
+					break;
+			}
+			array_push($console_data['messages'], $message);
+		}
+		$this->output['console'] = $console_data;
+	}
 
-    public function setFileData(array $file_data)
-    {
-        $this->output['files'] = $file_data['files'];
-        $this->output['fileTotals'] = $file_data['fileTotals'];
-    }
+	public function setFileData(array $file_data)
+	{
+		$this->output['files'] = $file_data['files'];
+		$this->output['fileTotals'] = $file_data['fileTotals'];
+	}
 
-    /**
-     * Sets memory data
-     *
-     * @param array $data
-     */
-    public function setMemoryData(array $data)
-    {
-        $this->output['memory'] = array(
-            'used'    => self::getReadableMemory($data['used']),
-            'allowed' => $data['allowed']
-        );
-    }
+	/**
+	 * Sets memory data
+	 *
+	 * @param array $data
+	 */
+	public function setMemoryData(array $data)
+	{
+		$this->output['memory'] = array(
+			'used'    => self::getReadableMemory($data['used']),
+			'allowed' => $data['allowed']
+		);
+	}
 
-    public function setQueryData(array $query_data)
-    {
-        // the void
-    }
+	public function setQueryData(array $query_data)
+	{
+		// the void
+	}
 
-    /**
-     * Sets speed data
-     *
-     * @param array $data
-     */
-    public function setSpeedData(array $data)
-    {
-        $this->output['speed'] = array(
-            'elapsed' => self::getReadableTime($data['elapsed']),
-            'allowed' => self::getReadableTime($data['allowed'], 0)
-        );
-    }
+	/**
+	 * Sets speed data
+	 *
+	 * @param array $data
+	 */
+	public function setSpeedData(array $data)
+	{
+		$this->output['speed'] = array(
+			'elapsed' => self::getReadableTime($data['elapsed']),
+			'allowed' => self::getReadableTime($data['allowed'], 0)
+		);
+	}
 
-    /**
-     * Static formatter for human-readable time
-     * Only handles time up to 60 minutes gracefully
-     *
-     * @param double  $time
-     * @param integer $decimals
-     * @return string
-     */
-    public static function getReadableTime($time, $decimals = 3)
-    {
-        $unit = 's';
-        if ($time < 1) {
-            $time *= 1000;
-            $unit = 'ms';
-        } else if ($time > 60) {
-            $time /= 60;
-            $unit = 'm';
-        }
-        $time = number_format($time, $decimals);
-        return "{$time} {$unit}";
-    }
+	/**
+	 * Static formatter for human-readable time
+	 * Only handles time up to 60 minutes gracefully
+	 *
+	 * @param double  $time
+	 * @param integer $decimals
+	 * @return string
+	 */
+	public static function getReadableTime($time, $decimals = 3)
+	{
+		$unit = 's';
+		if ($time < 1) {
+			$time *= 1000;
+			$unit = 'ms';
+		} else if ($time > 60) {
+			$time /= 60;
+			$unit = 'm';
+		}
+		$time = number_format($time, $decimals);
+		return "{$time} {$unit}";
+	}
 
-    /**
-     * Static formatter for human-readable memory
-     *
-     * @param double  $size
-     * @param integer $decimals
-     */
-    public static function getReadableMemory($size, $decimals = 2)
-    {
-        $unitOptions = array('b', 'k', 'M', 'G');
+	/**
+	 * Static formatter for human-readable memory
+	 *
+	 * @param double  $size
+	 * @param integer $decimals
+	 */
+	public static function getReadableMemory($size, $decimals = 2)
+	{
+		$unitOptions = array('b', 'k', 'M', 'G');
 
-        $base = log($size, 1024);
+		$base = log($size, 1024);
 
-        $memory = round(pow(1024, $base - floor($base)), $decimals);
-        $unit = $unitOptions[floor($base)];
-        return "{$memory} {$unit}";
-    }
+		$memory = round(pow(1024, $base - floor($base)), $decimals);
+		$unit = $unitOptions[floor($base)];
+		return "{$memory} {$unit}";
+	}
  
-    public function __invoke()
-    {
-        $output = $this->output;
-        require_once __DIR__ .'/../asset/display.tpl.php';
-    }
+	public function __invoke()
+	{
+		$output = $this->output;
+		require_once __DIR__ .'/../asset/display.tpl.php';
+	}
 }	

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
             )
         );
         foreach ($console->getLogs() as $log) {
-            switch($log['type']) {
+            switch ($log['type']) {
                 case 'log':
                     $message = array(
                         'data' => print_r($log['data'], true),
@@ -162,6 +162,6 @@ 
     public function __invoke()
     {
         $output = $this->output;
-        require_once __DIR__ .'/../asset/display.tpl.php';
+        require_once __DIR__.'/../asset/display.tpl.php';
     }
 }	

src/PhpQuickProfiler.php

Indentation +143 added lines, -143 removed lines

@@ -15,96 +15,96 @@ 
 class PhpQuickProfiler
 {
 
-    /** @var  Particletree\Pqp\Console */
-    protected $console;
-
-    /** @var  integer */
-    protected $startTime;
-
-    /**
-     * @param Particletree\Pqp\Console $console
-     * @param integer                  $startTime
-     */
-    public function __construct(Console $console, $startTime = null)
-    {
-        $this->console = $console;
-
-        if (is_null($startTime)) {
-            $startTime = microtime(true);
-        }
-        $this->startTime = $startTime;
-    }
+	/** @var  Particletree\Pqp\Console */
+	protected $console;
+
+	/** @var  integer */
+	protected $startTime;
+
+	/**
+	 * @param Particletree\Pqp\Console $console
+	 * @param integer                  $startTime
+	 */
+	public function __construct(Console $console, $startTime = null)
+	{
+		$this->console = $console;
+
+		if (is_null($startTime)) {
+			$startTime = microtime(true);
+		}
+		$this->startTime = $startTime;
+	}
  
   /*-------------------------------------------
       AGGREGATE DATA ON THE FILES INCLUDED
   -------------------------------------------*/
   
   public function gatherFileData() {
-    $files = get_included_files();
-    $fileList = array();
-    $fileTotals = array(
-      "count" => count($files),
-      "size" => 0,
-      "largest" => 0,
-    );
-
-    foreach($files as $file) {
-      $size = filesize($file);
-      $fileList[] = array(
-          'name' => $file,
-          'size' => $this->getReadableFileSize($size)
-        );
-      $fileTotals['size'] += $size;
-      if($size > $fileTotals['largest']) $fileTotals['largest'] = $size;
-    }
+	$files = get_included_files();
+	$fileList = array();
+	$fileTotals = array(
+	  "count" => count($files),
+	  "size" => 0,
+	  "largest" => 0,
+	);
+
+	foreach($files as $file) {
+	  $size = filesize($file);
+	  $fileList[] = array(
+		  'name' => $file,
+		  'size' => $this->getReadableFileSize($size)
+		);
+	  $fileTotals['size'] += $size;
+	  if($size > $fileTotals['largest']) $fileTotals['largest'] = $size;
+	}
     
-    $fileTotals['size'] = $this->getReadableFileSize($fileTotals['size']);
-    $fileTotals['largest'] = $this->getReadableFileSize($fileTotals['largest']);
+	$fileTotals['size'] = $this->getReadableFileSize($fileTotals['size']);
+	$fileTotals['largest'] = $this->getReadableFileSize($fileTotals['largest']);
 
-    return array(
-      'files' => $fileList,
-      'fileTotals' => $fileTotals
-    );
+	return array(
+	  'files' => $fileList,
+	  'fileTotals' => $fileTotals
+	);
   }
 
-    /**
-     * Get data about memory usage of the application
-     *
-     * @returns array
-     */
-    public function gatherMemoryData()
-    {
-        $usedMemory = memory_get_peak_usage();
-        $allowedMemory = ini_get('memory_limit');
-        return array(
-            'used'    => $usedMemory,
-            'allowed' => $allowedMemory
-        );
-    }
+	/**
+	 * Get data about memory usage of the application
+	 *
+	 * @returns array
+	 */
+	public function gatherMemoryData()
+	{
+		$usedMemory = memory_get_peak_usage();
+		$allowedMemory = ini_get('memory_limit');
+		return array(
+			'used'    => $usedMemory,
+			'allowed' => $allowedMemory
+		);
+	}
   
   /*--------------------------------------------------------
        QUERY DATA -- DATABASE OBJECT WITH LOGGING REQUIRED
   ----------------------------------------------------------*/
   
   public function gatherQueryData() {
-    $queryTotals = array();
-    $queryTotals['count'] = 0;
-    $queryTotals['time'] = 0;
-    $queries = array();
+	$queryTotals = array();
+	$queryTotals['count'] = 0;
+	$queryTotals['time'] = 0;
+	$queries = array();
     
-    if($this->db != '') {
-      $queryTotals['count'] += $this->db->queryCount;
-      foreach($this->db->queries as $query) {
-        $query = $this->attemptToExplainQuery($query);
-        $queryTotals['time'] += $query['time'];
-        $query['time'] = $this->getReadableTime($query['time']);
-        $queries[] = $query;
-      }
-    }
+	if($this->db != '') {
+	  $queryTotals['count'] += $this->db->queryCount;
+	  foreach($this->db->queries as $query) {
+		$query = $this->attemptToExplainQuery($query);
+		$queryTotals['time'] += $query['time'];
+		$query['time'] = $this->getReadableTime($query['time']);
+		$queries[] = $query;
+	  }
+	}
     
-    $queryTotals['time'] = $this->getReadableTime($queryTotals['time']);
-    $this->output['queries'] = $queries;
-    $this->output['queryTotals'] = $queryTotals;
+	$queryTotals['time'] = $this->getReadableTime($queryTotals['time']);
+	$this->output['queries'] = $queries;
+	$this->output['queryTotals'] = $queryTotals;
   }
   
   /*--------------------------------------------------------
@@ -112,90 +112,90 @@ 
   ----------------------------------------------------------*/
   
   function attemptToExplainQuery($query) {
-    try {
-      $sql = 'EXPLAIN '.$query['sql'];
-      $rs = $this->db->query($sql);
-    }
-    catch(Exception $e) {}
-    if($rs) {
-      $row = mysql_fetch_array($rs, MYSQL_ASSOC);
-      $query['explain'] = $row;
-    }
-    return $query;
+	try {
+	  $sql = 'EXPLAIN '.$query['sql'];
+	  $rs = $this->db->query($sql);
+	}
+	catch(Exception $e) {}
+	if($rs) {
+	  $row = mysql_fetch_array($rs, MYSQL_ASSOC);
+	  $query['explain'] = $row;
+	}
+	return $query;
   }
 
-    /**
-     * Get data about speed of the application
-     *
-     * @returns array
-     */
-    public function gatherSpeedData()
-    {
-        $elapsedTime = microtime(true) - $this->startTime;
-        $allowedTime = ini_get('max_execution_time');
-        return array(
-            'elapsed' => $elapsedTime,
-            'allowed' => $allowedTime
-        );
-    }
+	/**
+	 * Get data about speed of the application
+	 *
+	 * @returns array
+	 */
+	public function gatherSpeedData()
+	{
+		$elapsedTime = microtime(true) - $this->startTime;
+		$allowedTime = ini_get('max_execution_time');
+		return array(
+			'elapsed' => $elapsedTime,
+			'allowed' => $allowedTime
+		);
+	}
   
   /*-------------------------------------------
        HELPER FUNCTIONS TO FORMAT DATA
   -------------------------------------------*/
   
   public function getReadableFileSize($size, $retstring = null) {
-          // adapted from code at http://aidanlister.com/repos/v/function.size_readable.php
-         $sizes = array('bytes', 'kB', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB');
+		  // adapted from code at http://aidanlister.com/repos/v/function.size_readable.php
+		 $sizes = array('bytes', 'kB', 'MB', 'GB', 'TB', 'PB', 'EB', 'ZB', 'YB');
 
-         if ($retstring === null) { $retstring = '%01.2f %s'; }
+		 if ($retstring === null) { $retstring = '%01.2f %s'; }
 
-    $lastsizestring = end($sizes);
+	$lastsizestring = end($sizes);
 
-    foreach ($sizes as $sizestring) {
-          if ($size < 1024) { break; }
-             if ($sizestring != $lastsizestring) { $size /= 1024; }
-         }
-         if ($sizestring == $sizes[0]) { $retstring = '%01d %s'; } // Bytes aren't normally fractional
-         return sprintf($retstring, $size, $sizestring);
+	foreach ($sizes as $sizestring) {
+		  if ($size < 1024) { break; }
+			 if ($sizestring != $lastsizestring) { $size /= 1024; }
+		 }
+		 if ($sizestring == $sizes[0]) { $retstring = '%01d %s'; } // Bytes aren't normally fractional
+		 return sprintf($retstring, $size, $sizestring);
   }
 
-    /**
-     * Static formatter for human-readable time
-     * Only handles time up to 60 minutes gracefully
-     *
-     * @param integer $time time in seconds
-     * @return string
-     */
-    public static function getReadableTime($time)
-    {
-        $unit = 's';
-
-        if ($time < 1) {
-            $time *= 1000;
-            $unit = 'ms';
-        } else if ($time > 60) {
-            $time /= 60;
-            $unit = 'm';
-        }
-
-        $time = number_format($time, 3);
-        return "{$time} {$unit}";
-    }
+	/**
+	 * Static formatter for human-readable time
+	 * Only handles time up to 60 minutes gracefully
+	 *
+	 * @param integer $time time in seconds
+	 * @return string
+	 */
+	public static function getReadableTime($time)
+	{
+		$unit = 's';
+
+		if ($time < 1) {
+			$time *= 1000;
+			$unit = 'ms';
+		} else if ($time > 60) {
+			$time /= 60;
+			$unit = 'm';
+		}
+
+		$time = number_format($time, 3);
+		return "{$time} {$unit}";
+	}
   
 
-    /**
-     * Triggers end display of the profiling data
-     *
-     * @param Display $display
-     */
-    public function display(Display $display)
-    {
-        $display->setConsole($this->console);
-        $display->setFileData($this->gatherFileData());
-        $display->setMemoryData($this->gatherMemoryData());
-        $display->setQueryData($this->gatherQueryData());
-        $display->setSpeedData($this->gatherSpeedData());
-
-        $display();
-    }
+	/**
+	 * Triggers end display of the profiling data
+	 *
+	 * @param Display $display
+	 */
+	public function display(Display $display)
+	{
+		$display->setConsole($this->console);
+		$display->setFileData($this->gatherFileData());
+		$display->setMemoryData($this->gatherMemoryData());
+		$display->setQueryData($this->gatherQueryData());
+		$display->setSpeedData($this->gatherSpeedData());
+
+		$display();
+	}
 }

Spacing +6 added lines, -6 removed lines

@@ -48,14 +48,14 @@ 
       "largest" => 0,
     );
 
-    foreach($files as $file) {
+    foreach ($files as $file) {
       $size = filesize($file);
       $fileList[] = array(
           'name' => $file,
           'size' => $this->getReadableFileSize($size)
         );
       $fileTotals['size'] += $size;
-      if($size > $fileTotals['largest']) $fileTotals['largest'] = $size;
+      if ($size > $fileTotals['largest']) $fileTotals['largest'] = $size;
     }
     
     $fileTotals['size'] = $this->getReadableFileSize($fileTotals['size']);
@@ -92,9 +92,9 @@ 
     $queryTotals['time'] = 0;
     $queries = array();
     
-    if($this->db != '') {
+    if ($this->db != '') {
       $queryTotals['count'] += $this->db->queryCount;
-      foreach($this->db->queries as $query) {
+      foreach ($this->db->queries as $query) {
         $query = $this->attemptToExplainQuery($query);
         $queryTotals['time'] += $query['time'];
         $query['time'] = $this->getReadableTime($query['time']);
@@ -116,8 +116,8 @@ 
       $sql = 'EXPLAIN '.$query['sql'];
       $rs = $this->db->query($sql);
     }
-    catch(Exception $e) {}
-    if($rs) {
+    catch (Exception $e) {}
+    if ($rs) {
       $row = mysql_fetch_array($rs, MYSQL_ASSOC);
       $query['explain'] = $row;
     }