- Adding new config option: amp_exclusions_enable (default is enabl…

…ed). This improves compatibility with [Accelerated Mobile Pages](https://www.ampproject.org/). When this option is enabled and the URI being compressed ends with `/amp/`, or the document contains a top-level `<html ⚡>` tag (`<html amp>` is accepted as well), then features which are incompatible with [Accelerated Mobile Pages](https://www.ampproject.org/) will be disabled accordingly, regardless of your other settings. See [Issue #695](wpsharks/comet-cache#695) in the Comet Cache repo. - Adding `isDocAmpd()` conditional check against current URI & document. - Adding automatic AMP feature exclusions for improved AMP compatibility. - Optimizing for speed by removing unnecessary calls to `unset()`. - Enhancing unicode compatibility by taking full advantage of all `mb_*()` functions. - Adding multibyte compatible `pregQuote()`. - Adding multibyte compatible `replaceOnce()`. - Adding multibyte compatible `substrReplace()`. - Updating all regex patterns to add the `/u` flag for unicode compatibility. - Updating minimum PHP requirement. Now requires PHP v5.4+ in support of short array syntax `[]`.
wpsharks · Dec 7, 2016 · 3543679 · 3543679
1 parent 424dc37
commit 3543679
Show file tree

Hide file tree

Showing 7 changed files with 444 additions and 422 deletions.
diff --git a/.build.props b/.build.props
@@ -11,5 +11,5 @@ project_sub_namespace            = HtmlCompressor
 
 project_version                  = %y%m%d
 
-project_php_required_version     = 5.3.2
+project_php_required_version     = 5.4
 project_php_tested_up_to_version = ${php.version}
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,16 @@
+## $v
+
+- Adding new config option: `amp_exclusions_enable` (default is enabled). This improves compatibility with [Accelerated Mobile Pages](https://www.ampproject.org/). When this option is enabled and the URI being compressed ends with `/amp/`, or the document contains a top-level `<html ⚡>` tag (`<html amp>` is accepted as well), then features which are incompatible with [Accelerated Mobile Pages](https://www.ampproject.org/) will be disabled accordingly, regardless of your other settings. See [Issue #695](https://github.com/websharks/comet-cache/issues/695) in the Comet Cache repo.
+- Adding `isDocAmpd()` conditional check against current URI & document.
+- Adding automatic AMP feature exclusions for improved AMP compatibility.
+- Optimizing for speed by removing unnecessary calls to `unset()`.
+- Enhancing unicode compatibility by taking full advantage of all `mb_*()` functions.
+- Adding multibyte compatible `pregQuote()`.
+- Adding multibyte compatible `replaceOnce()`.
+- Adding multibyte compatible `substrReplace()`.
+- Updating all regex patterns to add the `/u` flag for unicode compatibility.
+- Updating minimum PHP requirement. Now requires PHP v5.4+ in support of short array syntax `[]`.
+
 ## v161108
 
 - **Enhancing Exclusion Tests:** Now testing exclusion rules against full open tag instead of only the CSS or JS URL and content. This allows for exclusion rules that exclude CSS or JS based on an opening tag attribute; e.g., `<style id='rs-plugin-settings-inline-css'`. See also: [this Comet Cache issue](https://github.com/websharks/comet-cache/issues/614).

diff --git a/README.md b/README.md
@@ -346,6 +346,8 @@ e.g., `new WebSharks\HtmlCompressor\Core($options);`
  please set this to a FALSE value. This can be helpful if your site (for whatever reason) is incompatible with the HTML compression routines.
  NOTE: if you disable this due to an incompatibility, please report it via GitHub so the issue can be resolved for everyone.
 
+- (boolean)`amp_exclusions_enable` TRUE by default. This improves compatibility with [Accelerated Mobile Pages](https://www.ampproject.org/). When this option is enabled and the URI being compressed ends with `/amp/`, or the document contains a top-level `<html ⚡>` tag (`<html amp>` is accepted as well), then features which are incompatible with [Accelerated Mobile Pages](https://www.ampproject.org/) will be disabled accordingly, regardless of your other settings.
+
 ----
 
 ##### Other misc. options. These don't really fall into any specific category yet.

diff --git a/composer.json b/composer.json
@@ -28,7 +28,7 @@
   },
 
   "require": {
-    "php": ">=5.3",
+    "php": ">=5.4",
     "ext-openssl": "*",
     "ext-curl": "*",
     "ext-mbstring": "*",

diff --git a/src/includes/classes/Benchmark.php b/src/includes/classes/Benchmark.php
@@ -16,18 +16,18 @@ class Benchmark // For debugging.
      */
 
     /**
-     * @type array An array of times.
+     * @var array An array of times.
      *
      * @since 150315 Enhancing debugger.
      */
-    protected $times = array();
+    protected $times = [];
 
     /**
-     * @type array An array of data.
+     * @var array An array of data.
      *
      * @since 150315 Enhancing debugger.
      */
-    protected $data = array();
+    protected $data = [];
 
     /*
      * Public API Methods