From c562a316817091b80ac9c7d3efa31067850b03e6 Mon Sep 17 00:00:00 2001 From: Aleksander Machniak <alec@alec.pl> Date: Wed, 18 Dec 2013 15:41:56 -0500 Subject: [PATCH] Improved minified files handling, added css minification (#1486988) --- program/include/rcmail_output_html.php | 43 +++++++++++++++++++++++++------------------ 1 files changed, 25 insertions(+), 18 deletions(-) diff --git a/program/include/rcmail_output_html.php b/program/include/rcmail_output_html.php index 7ff85e0..e4059b7 100644 --- a/program/include/rcmail_output_html.php +++ b/program/include/rcmail_output_html.php @@ -662,23 +662,33 @@ // add file modification timestamp if (preg_match('/\.(js|css)$/', $file, $m)) { - $fs = false; - $ext = $m[1]; - - // use minified file if exists (not in development mode) - if (!$this->devel_mode && !preg_match('/\.min\.' . $ext . '$/', $file)) { - $minified_file = substr($file, 0, strlen($ext) * -1) . 'min.' . $ext; - if ($fs = @filemtime($minified_file)) { - $file = $minified_file . '?s=' . $fs; - } - } - - if (!$fs && ($fs = @filemtime($file))) { - $file .= '?s=' . $fs; - } + $file = $this->file_mod($file); } return $matches[1] . '=' . $matches[2] . $file . $matches[4]; + } + + /** + * Modify file by adding mtime indicator + */ + protected function file_mod($file) + { + $fs = false; + $ext = substr($file, strrpos($file, '.') + 1); + + // use minified file if exists (not in development mode) + if (!$this->devel_mode && !preg_match('/\.min\.' . $ext . '$/', $file)) { + $minified_file = substr($file, 0, strlen($ext) * -1) . 'min.' . $ext; + if ($fs = @filemtime($minified_file)) { + return $minified_file . '?s=' . $fs; + } + } + + if ($fs = @filemtime($file)) { + $file .= '?s=' . $fs; + } + + return $file; } /** @@ -1225,10 +1235,7 @@ public function include_script($file, $position='head') { if (!preg_match('|^https?://|i', $file) && $file[0] != '/') { - $file = $this->scripts_path . $file; - if ($fs = @filemtime($file)) { - $file .= '?s=' . $fs; - } + $file = $this->file_mod($this->scripts_path . $file); } if (!is_array($this->script_files[$position])) { -- Gitblit v1.9.1