Merge pull request #508 from contactashish13/issue-507

Increase minimum requirement to PHP 5.6

Author: contactashish13

.travis.yml

@@ -8,17 +8,12 @@ matrix:
     - php: 7.1
     - php: 7.0
     - php: '5.6'
+    - name: Wraith
+      php: '5.6'
       install: true
       before_script: chmod +x bin/wraith.sh
       env: TEST_SUITE=Wraith_Visual_Regression_Testing WRAITH_FAIL=5
       script: "./bin/wraith.sh"
-    - php: 5.5
-      env: WP_VERSION=5.1
-    - php: 5.4
-      env: WP_VERSION=5.1
-    - php: 5.3
-      env: WP_VERSION=5.1
-      dist: precise
   allow_failures:
     - env: TEST_SUITE=Wraith_Visual_Regression_Testing WRAITH_FAIL=5
 branches:

classes/Visualizer/Render/Sidebar.php

@@ -270,7 +270,12 @@ private static function is_excel_enabled() {
 			include_once( $vendor_file );
 		}
 
-		return class_exists( 'PhpOffice\PhpSpreadsheet\Spreadsheet' ) && extension_loaded( 'zip' ) && extension_loaded( 'xml' ) && extension_loaded( 'fileinfo' ) && version_compare( PHP_VERSION, '5.6.0', '>' );
+		if ( version_compare( phpversion(), '5.6.0', '<' ) ) {
+			do_action( 'themeisle_log_event', Visualizer_Plugin::NAME, sprintf( 'PHP version %s not supported', phpversion() ), 'error', __FILE__, __LINE__ );
+			return false;
+		}
+
+		return class_exists( 'PhpOffice\PhpSpreadsheet\Spreadsheet' ) && extension_loaded( 'zip' ) && extension_loaded( 'xml' ) && extension_loaded( 'fileinfo' );
 	}
 
 	/**

classes/Visualizer/Source.php

@@ -329,6 +329,7 @@ public static final function get_date_formats_if_exists( $series, $data ) {
 	 */
 	private static final function determine_date_format( $value, $type ) {
 		if ( version_compare( phpversion(), '5.3.0', '<' ) ) {
+			do_action( 'themeisle_log_event', Visualizer_Plugin::NAME, sprintf( 'PHP version %s not supported', phpversion() ), 'error', __FILE__, __LINE__ );
 			return null;
 		}
 

readme.md

@@ -3,6 +3,7 @@
 **Tags:** chart, charts, charting, graph, graphs, graphing, visualisation, visualise data, visualization, visualize data, HTML5, canvas, pie chart, line chart, bar chart, column chart, gauge chart, area chart, scatter chart, candlestick chart, geo chart, google visualization api, data tables, chartjs  
 **Requires at least:** 3.5  
 **Tested up to:** 5.2  
+**Requires PHP:** 5.6
 **Stable tag:** trunk  
 **License:** GPL v2.0 or later  
 **License URI:** http://www.opensource.org/licenses/gpl-license.php  

readme.txt

@@ -3,6 +3,7 @@ Contributors:  codeinwp,marius2012,marius_codeinwp,hardeepasrani,themeisle,Madal
 Tags: chart, charts, charting, graph, graphs, graphing, visualisation, visualise data, visualization, visualize data, HTML5, canvas, pie chart, line chart, bar chart, column chart, gauge chart, area chart, scatter chart, candlestick chart, geo chart, google visualization api, data tables, chartjs
 Requires at least: 3.5
 Tested up to: 5.2
+Requires PHP: 5.6
 Stable tag: trunk
 License: GPL v2.0 or later
 License URI: http://www.opensource.org/licenses/gpl-license.php