Fixed issue with 304 caching. Fixed PHP 5.3 compatibility.

flight/Engine.php

@@ -75,6 +75,7 @@ class Engine {
      */
     public function init() {
         static $initialized = false;
+        $self = $this;
 
         if ($initialized) {
             $this->vars = array();
@@ -86,8 +87,8 @@ class Engine {
         $this->loader->register('request', '\flight\net\Request');
         $this->loader->register('response', '\flight\net\Response');
         $this->loader->register('router', '\flight\net\Router');
-        $this->loader->register('view', '\flight\template\View', array(), function($view) {
+        $this->loader->register('view', '\flight\template\View', array(), function($view) use ($self) {
-            $view->path = $this->get('flight.views.path');
+            $view->path = $self->get('flight.views.path');
         });
 
         // Register framework methods
@@ -320,12 +321,13 @@ class Engine {
      *
      * @param int $code HTTP status code
      * @param string $message Response message
+     * @param bool $cache Cache response
      */
-    public function _halt($code = 200, $message = '') {
+    public function _halt($code = 200, $message = '', $cache = false) {
         $this->response(false)
             ->status($code)
             ->write($message)
-            ->cache(false)
+            ->cache($cache)
             ->send();
     }
 
@@ -438,7 +440,7 @@ class Engine {
         $this->response()->header('ETag', $id);
 
         if ($id === getenv('HTTP_IF_NONE_MATCH')) {
-            $this->halt(304);
+            $this->halt(304, '', true);
         }
     }
 
@@ -451,7 +453,7 @@ class Engine {
         $this->response()->header('Last-Modified', date(DATE_RFC1123, $time));
 
         if ($time === strtotime(getenv('HTTP_IF_MODIFIED_SINCE'))) {
-            $this->halt(304);
+            $this->halt(304, '', true);
         }
     }
 }

flight/net/Response.php

@@ -151,19 +151,21 @@ class Response {
      * @return object Self reference
      */
     public function cache($expires) {
-        if ($expires === false) {
+        if ($expires !== true) {
-            $this->headers['Expires'] = 'Mon, 26 Jul 1997 05:00:00 GMT';
+            if ($expires === false) {
-            $this->headers['Cache-Control'] = array(
+                $this->headers['Expires'] = 'Mon, 26 Jul 1997 05:00:00 GMT';
-                'no-store, no-cache, must-revalidate',
+                $this->headers['Cache-Control'] = array(
-                'post-check=0, pre-check=0',
+                    'no-store, no-cache, must-revalidate',
-                'max-age=0'
+                    'post-check=0, pre-check=0',
-            );
+                    'max-age=0'
-            $this->headers['Pragma'] = 'no-cache';
+                );
-        }
+                $this->headers['Pragma'] = 'no-cache';
-        else {
+            }
-            $expires = is_int($expires) ? $expires : strtotime($expires);
+            else {
-            $this->headers['Expires'] = gmdate('D, d M Y H:i:s', $expires) . ' GMT';
+                $expires = is_int($expires) ? $expires : strtotime($expires);
-            $this->headers['Cache-Control'] = 'max-age='.($expires - time());
+                $this->headers['Expires'] = gmdate('D, d M Y H:i:s', $expires) . ' GMT';
+                $this->headers['Cache-Control'] = 'max-age='.($expires - time());
+            }
         }
 
         return $this;