wp-advads 1.0.4 → 1.0.6

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "wp-advads",
-  "version": "1.0.4",
+  "version": "1.0.6",
   "description": "Create a Advanced Ads wordPress plugin eco system.",
   "author": {
     "name": "Shakeeb Ahmed",

package/src/commands/setup/create-plugin.js

@@ -148,7 +148,7 @@ class CreatePlugin {
             'async',
             'browser-sync',
             'browser-sync-webpack-plugin',
-            'chalk',
+            'chalk@4.1.2',
             'eslint-plugin-prettier',
             'husky',
             'laravel-mix',

package/template/configs/.phpcs.xml

@@ -57,9 +57,13 @@
         <type>error</type>
     </rule>
 
-    <rule ref="Generic.Arrays.DisallowLongArraySyntax">
-        <type>error</type>
-    </rule>
+    <!-- Disable short arrays: -->
+	<rule ref="WordPress-Extra">
+		<exclude name="Generic.Arrays.DisallowShortArraySyntax"/>
+	</rule>
+
+	<!-- Enforce short arrays: -->
+	<rule ref="Generic.Arrays.DisallowLongArraySyntax"/>
 
 	<!-- Method names MUST NOT be prefixed with a single underscore to indicate protected or private visibility. That is, an underscore prefix explicitly has no meaning. -->
     <rule ref="PSR2.Methods.MethodDeclaration.Underscore">

package/template/tools/wp-glotpress.js

@@ -1,68 +1,101 @@
 #!/usr/bin/env node
+/* eslint-disable no-console */
 
 // Packages.
-const chalk = require("chalk");
-const fs = require("fs");
-const { resolve } = require("path");
-const async = require("async");
+const chalk = require('chalk');
+const fs = require('fs');
+const { resolve } = require('path');
+const async = require('async');
 
 // Settings.
-const package = require("../package.json");
+const packageDetails = require('../package.json');
 const BASEURL = 'https://translate.wpadvancedads.com/api/projects';
 
 // Validation of settings.
-if ( undefined === package.glotpress || Object.keys(package.glotpress).length < 1 ) {
-	console.log( chalk.bgRed.bold("Error:") + " The GlotPress settings is not defined." );
-	return;
+function validateSettings() {
+	if (
+		undefined === packageDetails.glotpress ||
+		Object.keys(packageDetails.glotpress).length < 1
+	) {
+		console.log(
+			chalk.bgRed.bold('Error:') +
+				' The GlotPress settings is not defined.'
+		);
+		return false;
+	}
+
+	if (
+		undefined === packageDetails.glotpress.project ||
+		('' === undefined) === packageDetails.glotpress.project
+	) {
+		console.log(
+			chalk.bgRed.bold('Error:') +
+				' The GlotPress project name is not defined.'
+		);
+		return false;
+	}
+
+	if (
+		undefined === packageDetails.glotpress.destination ||
+		('' === undefined) === packageDetails.glotpress.destination
+	) {
+		console.log(
+			chalk.bgRed.bold('Error:') +
+				' The destination directory is not defined.'
+		);
+		return false;
+	}
+
+	return true;
 }
 
-if ( undefined === package.glotpress.project || '' === undefined === package.glotpress.project ) {
-	console.log( chalk.bgRed.bold("Error:") + " The GlotPress project name is not defined." );
-	return;
-}
-
-if ( undefined === package.glotpress.destination || '' === undefined === package.glotpress.destination ) {
-	console.log( chalk.bgRed.bold("Error:") + " The destination directory is not defined." );
-	return;
-}
-
-const { glotpress } = package
-const { project } = glotpress
-let { potPrefix, destination } = glotpress
-potPrefix = potPrefix ?? project
-destination =resolve(destination)
+const { glotpress } = packageDetails;
+const { project } = glotpress;
+let { potPrefix, destination } = glotpress;
+potPrefix = potPrefix ?? project;
+destination = resolve(destination);
 
 if (!fs.existsSync(destination)) {
 	fs.mkdirSync(destination);
 }
 
 async function getGlotPressData() {
-	const response = await fetch(`${BASEURL}/${package.glotpress.project}`);
+	const response = await fetch(
+		`${BASEURL}/${packageDetails.glotpress.project}`
+	);
 	const data = await response.json();
-	const sets = {}
+	const sets = {};
+
+	if (undefined !== data.success && !data.success) {
+		console.log(chalk.bgRed.bold('Error:') + ' ' + data.error);
+		return false;
+	}
+
 	data.translation_sets.map((set) => {
-		if ( set.current_count > 0 ) {
+		if (set.current_count > 0) {
 			let id = set.wp_locale;
-			if ( 'default' !== set.slug ) {
+			if ('default' !== set.slug) {
 				id += '_' + set.slug;
 			}
 
-			sets[ id ] = set
+			sets[id] = set;
 		}
-	})
 
-	return sets
+		return false;
+	});
+
+	return sets;
 }
 
 // Download Handler.
 async function downloadFile(locale, data, format) {
-	console.log( chalk.bold("Downloading: " + format));
+	console.log(chalk.bold('Downloading: ' + format));
 
-	const target = `${destination}/${potPrefix}-${locale}.${format}`
-	const endpoint = `${BASEURL}/${project}/${data.locale}/${data.slug}/export-translations?format=${format}`
+	const target = `${destination}/${potPrefix}-${locale}.${format}`;
+	const endpoint = `${BASEURL}/${project}/${data.locale}/${data.slug}/export-translations?format=${format}`;
 
 	// Download.
-	const response = await fetch(endpoint)
+	const response = await fetch(endpoint);
 	const content = await response.text();
 
 	fs.writeFileSync(target, content);
@@ -71,25 +104,37 @@ async function downloadFile(locale, data, format) {
 async function runCommand() {
 	const locales = await getGlotPressData();
 
+	if (false === locales) {
+		return;
+	}
+
 	// Download files.
 	async.map(
 		Object.keys(locales),
 		(locale, callback) => {
-			const localeData = locales[locale]
-
-			console.log("");
-			console.log( chalk.bgGreen.bold("Updating Language:") + ` ${chalk.italic(localeData.name)}`);
-			async.map(
-				['mo', 'po'],
-				(format) => downloadFile(locale, localeData, format)
-			)
-
-			callback(null, true)
+			const localeData = locales[locale];
+
+			console.log('');
+			console.log(
+				chalk.bgGreen.bold('Updating Language:') +
+					` ${chalk.italic(localeData.name)}`
+			);
+			async.map(['mo', 'po'], (format) =>
+				downloadFile(locale, localeData, format)
+			);
+
+			callback(null, true);
 		},
-		function(err, results) {
-			console.log("");
-			console.log(chalk.bgGreen.bold("Success:") + " All files has been downloaded.");
+		function () {
+			console.log('');
+			console.log(
+				chalk.bgGreen.bold('Success:') +
+					' All files has been downloaded.'
+			);
 		}
 	);
 }
-runCommand()
+
+if (validateSettings()) {
+	runCommand();
+}