From d6ec474f78b3b22e23a7f194f9dce42382fee408 Mon Sep 17 00:00:00 2001 From: James Date: Sun, 11 Jun 2017 20:12:29 +1000 Subject: [PATCH] Fix case sensitivity --- .../screenshots/{1.PNG => 1.png} | Bin .../screenshots/{2.PNG => 2.png} | Bin .../screenshots/{3.PNG => 3.png} | Bin .../screenshots/{4.PNG => 4.png} | Bin .../screenshots/{5.PNG => 5.png} | Bin .../screenshots/{6.PNG => 6.png} | Bin .../screenshots/{7.PNG => 7.png} | Bin .../screenshots/{8.PNG => 8.png} | Bin _validation/app.js | 39 +++++++++++++++--- 9 files changed, 34 insertions(+), 5 deletions(-) rename Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/{1.PNG => 1.png} (100%) rename Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/{2.PNG => 2.png} (100%) rename Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/{3.PNG => 3.png} (100%) rename Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/{4.PNG => 4.png} (100%) rename Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/{5.PNG => 5.png} (100%) rename Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/{6.PNG => 6.png} (100%) rename Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/{7.PNG => 7.png} (100%) rename Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/{8.PNG => 8.png} (100%) diff --git a/Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/1.PNG b/Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/1.png similarity index 100% rename from Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/1.PNG rename to Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/1.png diff --git a/Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/2.PNG b/Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/2.png similarity index 100% rename from Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/2.PNG rename to Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/2.png diff --git a/Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/3.PNG b/Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/3.png similarity index 100% rename from Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/3.PNG rename to Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/3.png diff --git a/Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/4.PNG b/Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/4.png similarity index 100% rename from Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/4.PNG rename to Phoenix-Wright-Ace-Attorney-Dual-Destinies/screenshots/4.png diff --git a/Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/5.PNG b/Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/5.png similarity index 100% rename from Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/5.PNG rename to Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/5.png diff --git a/Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/6.PNG b/Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/6.png similarity index 100% rename from Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/6.PNG rename to Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/6.png diff --git a/Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/7.PNG b/Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/7.png similarity index 100% rename from Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/7.PNG rename to Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/7.png diff --git a/Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/8.PNG b/Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/8.png similarity index 100% rename from Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/8.PNG rename to Phoenix-Wright-Ace-Attorney-Spirit-Of-Justice/screenshots/8.png diff --git a/_validation/app.js b/_validation/app.js index 1024109..2ebac55 100644 --- a/_validation/app.js +++ b/_validation/app.js @@ -12,6 +12,9 @@ const toml = require('toml'); let currentGame = null; let errors = []; +// Catch non-formatting errors +let miscError = false; + function getDirectories(srcpath) { return fs.readdirSync(srcpath) .filter(file => fs.lstatSync(path.join(srcpath, file)).isDirectory()) @@ -22,6 +25,12 @@ function getFiles(srcpath) { .filter(file => fs.lstatSync(path.join(srcpath, file)).isFile()) } +/// Check that a filename matches the following: +/// [any of a-z, A-Z, 0-9, a '-' or a '_'](one or more) . [a-z](two or three) +function isValidFilename(name) { + return name.match(/^([a-zA-Z0-9_\-])+\.([a-z]){2,3}$/); +} + /// Validates that a image is correctly sized and of the right format. function validateImage(path, config) { if (fs.existsSync(path) === false) { @@ -63,6 +72,12 @@ function validateDirImages(path, config) { if (fs.existsSync(path)) { const files = getFiles(path); + files.forEach(file => { + if (!isValidFilename(file)) { + validationError(`File \"${file}\" contains bad characters!`); + } + }); + for (let i = 0; i < files.length; i++) { const file = files[i]; validateImage(`${path}/${file}`, config); @@ -113,7 +128,7 @@ function validateIsBoolean(struct, name) { /// Validates pattern of YYYY-MM-DD in a field of a structure. function validateIsDate(struct, name) { validateContents(struct, name, field => { - if (!field.match(/[0-9]{4}-((0[1-9])|(1[0-2]))-((0[1-9])|([1-2][0-9])|(3[0-1]))/)) { + if (!field.match(/^[0-9]{4}-((0[1-9])|(1[0-2]))-((0[1-9])|([1-2][0-9])|(3[0-1]))$/)) { validationError(`\"${name}\" is not a valid date (\"${field}\").`); } }); @@ -189,7 +204,7 @@ function validateTOML(path) { validateContents(release, "title", field => { if (field.length !== 16) { validationError(`Release #${i + 1}: Game title ID has an invalid length`); - } else if (!field.match(/([a-zA-Z0-9]){16}/)) { + } else if (!field.match(/^([a-zA-Z0-9]){16}$/)) { validationError(`Release #${i + 1}: Game title ID is not a hexadecimal ID`); } }); @@ -216,7 +231,7 @@ function validateTOML(path) { validateContents(testcase, "title", field => { if (field.length !== 16) { validationError(`Testcase #${i + 1}: Game title ID has an invalid length`); - } else if (!field.match(/([a-zA-Z0-9]){16}/)) { + } else if (!field.match(/^([a-zA-Z0-9]){16}$/)) { validationError(`Testcase #${i + 1}: Game title ID is not a hexadecimal ID`); } }); @@ -273,7 +288,7 @@ function validateSaveTOML(path) { validateContents(tomlDoc, "title_id", field => { if (field.length !== 16) { validationError(`Game save data: Game title ID has an invalid length`); - } else if (!field.match(/([a-zA-Z0-9]){16}/)) { + } else if (!field.match(/^([a-zA-Z0-9]){16}$/)) { validationError(`Game save data: Game title ID is not a hexadecimal ID`); } }); @@ -299,6 +314,12 @@ function validateSaves(dir) { const files = getFiles(dir); + files.forEach(file => { + if (!isValidFilename(file)) { + validationError(`File \"${file}\" contains bad characters!`); + } + }); + // Strip extensions, so we know what save 'groups' we are dealing with const strippedFiles = files.map(file => { return file.substr(0, file.lastIndexOf(".")) @@ -334,6 +355,13 @@ getDirectories(config.directory).forEach(function (game) { let inputDirectoryGame = `${config.directory}/${game}`; currentGame = game; + // Check that everything is lowercase. + getFiles(inputDirectoryGame).forEach(file => { + if (!isValidFilename(file)) { + validationError(`File \"${file}\" contains bad characters!`); + } + }); + // Verify the game's boxart. validateImage(`${inputDirectoryGame}/${config.boxart.filename}`, config.boxart); @@ -353,10 +381,11 @@ getDirectories(config.directory).forEach(function (game) { } catch (ex) { console.warn(`${game} has encountered an unexpected error.`); console.error(ex); + miscError = true; } }); -if (errors.length > 0) { +if (errors.length > 0 || miscError) { console.warn('Validation completed with errors.'); const groups = groupBy(errors, "game");