Call date() only once.

Author: sebastianbergmann

PHP/CodeCoverage/Report/HTML.php

@@ -132,14 +132,17 @@ public function process(PHP_CodeCoverage $coverage, $target)
         $report = $coverage->getReport();
         unset($coverage);
 
+        $date = date('D M j G:i:s T Y', $_SERVER['REQUEST_TIME']);
+
         $dashboard = new PHP_CodeCoverage_Report_HTML_Dashboard(
-          $this->templatePath, $this->charset, $this->generator
+          $this->templatePath, $this->charset, $this->generator, $date
         );
 
         $directory = new PHP_CodeCoverage_Report_HTML_Directory(
           $this->templatePath,
           $this->charset,
           $this->generator,
+          $date,
           $this->lowUpperBound,
           $this->highLowerBound
         );
@@ -148,6 +151,7 @@ public function process(PHP_CodeCoverage $coverage, $target)
           $this->templatePath,
           $this->charset,
           $this->generator,
+          $date,
           $this->lowUpperBound,
           $this->highLowerBound,
           $this->highlight,

PHP/CodeCoverage/Report/HTML/Dashboard.php

@@ -72,18 +72,25 @@ class PHP_CodeCoverage_Report_HTML_Dashboard
      */
     protected $generator;
 
+    /**
+     * @var string
+     */
+    protected $date;
+
     /**
      * Constructor.
      *
      * @param string  $templatePath
      * @param string  $charset
      * @param string  $generator
+     * @param string  $date
      */
-    public function __construct($templatePath, $charset, $generator)
+    public function __construct($templatePath, $charset, $generator, $date)
     {
         $this->templatePath = $templatePath;
         $this->charset      = $charset;
         $this->generator    = $generator;
+        $this->date         = $date;
     }
 
     /**
@@ -106,10 +113,7 @@ public function render(PHP_CodeCoverage_Report_Node_Directory $node, $file, $tit
           array(
             'title'                  => $title,
             'charset'                => $this->charset,
-            'date'                   => date(
-                                          'D M j G:i:s T Y',
-                                          $_SERVER['REQUEST_TIME']
-                                        ),
+            'date'                   => $this->date,
             'version'                => '@package_version@',
             'php_version'            => PHP_VERSION,
             'generator'              => $this->generator,

PHP/CodeCoverage/Report/HTML/Directory.php

@@ -72,6 +72,11 @@ class PHP_CodeCoverage_Report_HTML_Directory
      */
     protected $generator;
 
+    /**
+     * @var string
+     */
+    protected $date;
+
     /**
      * @var integer
      */
@@ -88,14 +93,16 @@ class PHP_CodeCoverage_Report_HTML_Directory
      * @param string  $templatePath
      * @param string  $charset
      * @param string  $generator
+     * @param string  $date
      * @param integer $lowUpperBound
      * @param integer $highLowerBound
      */
-    public function __construct($templatePath, $charset, $generator, $lowUpperBound, $highLowerBound)
+    public function __construct($templatePath, $charset, $generator, $date, $lowUpperBound, $highLowerBound)
     {
         $this->templatePath   = $templatePath;
         $this->charset        = $charset;
         $this->generator      = $generator;
+        $this->date           = $date;
         $this->lowUpperBound  = $lowUpperBound;
         $this->highLowerBound = $highLowerBound;
     }
@@ -119,10 +126,7 @@ public function render(PHP_CodeCoverage_Report_Node_Directory $node, $file, $tit
             'charset'          => $this->charset,
             'low_upper_bound'  => $this->lowUpperBound,
             'high_lower_bound' => $this->highLowerBound,
-            'date'             => date(
-                                    'D M j G:i:s T Y',
-                                    $_SERVER['REQUEST_TIME']
-                                  ),
+            'date'             => $this->date,
             'version'          => '@package_version@',
             'php_version'      => PHP_VERSION,
             'generator'        => $this->generator

PHP/CodeCoverage/Report/HTML/File.php

@@ -72,6 +72,11 @@ class PHP_CodeCoverage_Report_HTML_File
      */
     protected $generator;
 
+    /**
+     * @var string
+     */
+    protected $date;
+
     /**
      * @var integer
      */
@@ -98,16 +103,18 @@ class PHP_CodeCoverage_Report_HTML_File
      * @param string  $templatePath
      * @param string  $charset
      * @param string  $generator
+     * @param string  $date
      * @param integer $lowUpperBound
      * @param integer $highLowerBound
      * @param boolean $highlight
      * @param boolean $yui
      */
-    public function __construct($templatePath, $charset, $generator, $lowUpperBound, $highLowerBound, $highlight, $yui)
+    public function __construct($templatePath, $charset, $generator, $date, $lowUpperBound, $highLowerBound, $highlight, $yui)
     {
         $this->templatePath   = $templatePath;
         $this->charset        = $charset;
         $this->generator      = $generator;
+        $this->date           = $date;
         $this->lowUpperBound  = $lowUpperBound;
         $this->highLowerBound = $highLowerBound;
         $this->highlight      = $highlight;
@@ -137,10 +144,7 @@ public function render(PHP_CodeCoverage_Report_Node_File $node, $file, $title =
           array(
             'title'       => $title,
             'charset'     => $this->charset,
-            'date'        => date(
-                               'D M j G:i:s T Y',
-                               $_SERVER['REQUEST_TIME']
-                             ),
+            'date'        => $this->date,
             'version'     => '@package_version@',
             'php_version' => PHP_VERSION,
             'generator'   => $this->generator