Declare compatibility with WordPress 5.4, update dependencies, add PHP 7.4 to Travis build (#16)

* Declare compatibility with WordPress 5.4, update dependencies, add PHP 7.4 to Travis build

* Improve code style

* Improve code style of JavaScript file

Author: patrickrobrecht

.travis.yml

@@ -1,10 +1,11 @@
 language: php
 
 php:
-  - '5.6'
-  - '7.1'
-  - '7.2'
-  - '7.3'
+  - 5.6
+  - 7.1
+  - 7.2
+  - 7.3
+  - 7.4
   - nightly
 
 matrix:

CHANGELOG.md

@@ -5,10 +5,15 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 
-## Version 1.1.2
+## Unreleased
 
 ### Changed
+* Enhancement: Compliance with latest WP coding guidelines and other code style improvements
+
 
+## Version 1.1.2
+
+### Changed
 * Enhancement: Compliance with latest WP coding guidelines
 
 

assets/functions.js

@@ -1,32 +1,32 @@
-function exportTableToCSV($table, filename) {
-	var tmpColDelim = String.fromCharCode(11), tmpRowDelim = String.fromCharCode(0), // Temporary delimiters unlikely to be typed by keyboard to avoid accidentally splitting the actual contents
-	colDelim = '","', rowDelim = '"\r\n"', // actual delimiters for CSV
-	$rows = $table.find('tr'),
-	csv = '"' + $rows.map(function(i, row) {
-		var $row = jQuery(row), $cols = $row.find('td,th');
-		return $cols.map(function(j, col) {
-			var $col = jQuery(col), text = $col.text();
-			return text.replace(/"/g, '""'); // escape double quotes
-		}).get().join(tmpColDelim);
-	}).get().join(tmpRowDelim).split(tmpRowDelim)
-			.join(rowDelim).split(tmpColDelim)
-			.join(colDelim) + '"',
-	csvData = 'data:application/csv;charset=utf-8,' + encodeURIComponent(csv);
-	jQuery(this).attr({
-		'download' : filename,
-		'href' : csvData
-	});
+function posts_and_users_stats_export_table_to_csv(table, filename) {
+    const tmpColDelim = String.fromCharCode(11), tmpRowDelim = String.fromCharCode(0), // Temporary delimiters unlikely to be typed by keyboard to avoid accidentally splitting the actual contents
+        colDelim = '","', rowDelim = '"\r\n"', // actual delimiters for CSV
+        rows = table.find('tr'),
+        csv = '"' + rows.map(function (i, row) {
+            const $row = jQuery(row), $cols = $row.find('td,th');
+            return $cols.map(function (j, col) {
+                const $col = jQuery(col), text = $col.text();
+                return text.replace(/"/g, '""'); // escape double quotes
+            }).get().join(tmpColDelim);
+        }).get().join(tmpRowDelim).split(tmpRowDelim)
+            .join(rowDelim).split(tmpColDelim)
+            .join(colDelim) + '"',
+        csvData = 'data:application/csv;charset=utf-8,' + encodeURIComponent(csv);
+    jQuery(this).attr({
+        'download': filename,
+        'href': csvData
+    });
 }
 
 function posts_and_users_stats_bar_chart(div, xData, yData, xAxisTitle, yAxisTitle) {
-    var data = {
+    const data = {
         labels: xData,
-    	series: [
-        	yData
-		]
-	};
+        series: [
+            yData
+        ]
+    };
 
-    var options = {
+    const options = {
         seriesBarDistance: 20,
         chartPadding: {
             top: 20,
@@ -61,7 +61,7 @@ function posts_and_users_stats_bar_chart(div, xData, yData, xAxisTitle, yAxisTit
         ]
     };
 
-    var responsiveOptions = [
+    const responsiveOptions = [
         ['screen and (max-width: 640px)', {
             seriesBarDistance: 5,
             axisX: {
@@ -76,7 +76,7 @@ function posts_and_users_stats_bar_chart(div, xData, yData, xAxisTitle, yAxisTit
 }
 
 function posts_and_users_stats_time_line_chart(div, seriesData, dateFormat, xAxisTitle, yAxisTitle) {
-    var data = {
+    const data = {
         series: [
             {
                 name: 'series-1',
@@ -85,7 +85,7 @@ function posts_and_users_stats_time_line_chart(div, seriesData, dateFormat, xAxi
         ]
     };
 
-    var options = {
+    const options = {
         chartPadding: {
             top: 20,
             right: 30,
@@ -95,7 +95,7 @@ function posts_and_users_stats_time_line_chart(div, seriesData, dateFormat, xAxi
         axisX: {
             type: Chartist.FixedScaleAxis,
             divisor: 5,
-            labelInterpolationFnc: function(value) {
+            labelInterpolationFnc: function (value) {
                 return moment(value).format(dateFormat);
             }
         },

composer.json

@@ -17,16 +17,16 @@
   "require": {
     "php": ">=5.6",
     "npm-asset/chartist": "^0.11.4",
-    "npm-asset/chartist-plugin-axistitle": "^0.0.6",
+    "npm-asset/chartist-plugin-axistitle": "^0.0.7",
     "npm-asset/moment": "^2.24.0"
   },
   "require-dev": {
-    "dealerdirect/phpcodesniffer-composer-installer": "^0.4",
-    "matthiasmullie/minify": "^1.3",
-    "phpcompatibility/php-compatibility": "^9.3",
-    "slowprog/composer-copy-file": "^0.3",
-    "squizlabs/php_codesniffer": "^3.5",
-    "wp-coding-standards/wpcs": "^2.1"
+    "dealerdirect/phpcodesniffer-composer-installer": "^0.6.2",
+    "matthiasmullie/minify": "^1.3.63",
+    "phpcompatibility/php-compatibility": "^9.3.5",
+    "slowprog/composer-copy-file": "^0.3.1",
+    "squizlabs/php_codesniffer": "^3.5.4",
+    "wp-coding-standards/wpcs": "^2.2.1"
   },
   "repositories": [
     {