Infrastructure/ghost
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2025-01-27 22:49:56 -05:00
Code Issues Activity

Cleaned up data validation module

Browse source 
no-issue

Removes overly verbose repetition of identifiers making this easier to read
This commit is contained in:
Fabien O'Carroll 2020-06-30 16:41:12 +02:00 • committed by Fabien 'egg' O'Carroll
parent 774187ced5
commit 38fce4efc5
1 changed files with 13 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
core/server/data/validation/index.js
View file

@ -8,10 +8,6 @@ const {i18n} = require('../../lib/common');
const errors = require('@tryghost/errors');
const settingsCache = require('../../services/settings/cache');
const urlUtils = require('../../../shared/url-utils');
let validatePassword;
let validateSchema;
let validateSettings;
let validate;
function assertString(input) {
assert(typeof input === 'string', 'Validator js validates strings only');
@ -92,7 +88,7 @@ validator.extend('isSlug', function isSlug(str) {
* invalid password: `validationResult: {isValid: false, message: 'Sorry, you cannot use an insecure password.'}`
* valid password: `validationResult: {isValid: true}`
*/
validatePassword = function validatePassword(password, email, blogTitle) {
function validatePassword(password, email, blogTitle) {
const validationResult = {isValid: true};
const disallowedPasswords = ['password', 'ghost', 'passw0rd'];
let blogUrl = urlUtils.urlFor('home', true);
@ -159,7 +155,7 @@ validatePassword = function validatePassword(password, email, blogTitle) {
}
return validationResult;
};
}
/**
* Validate model against schema.
@ -176,7 +172,7 @@ validatePassword = function validatePassword(password, email, blogTitle) {
* ## on model add
* - validate everything to catch required fields
*/
validateSchema = function validateSchema(tableName, model, options) {
function validateSchema(tableName, model, options) {
options = options || {};
const columns = _.keys(schema[tableName]);
@ -271,12 +267,12 @@ validateSchema = function validateSchema(tableName, model, options) {
}
return Promise.resolve();
};
}
// Validation for settings
// settings are checked against the validation objects
// form default-settings.json
validateSettings = function validateSettings(defaultSettings, model) {
function validateSettings(defaultSettings, model) {
const values = model.toJSON();
let validationErrors = [];
const matchingDefault = defaultSettings[values.key];
@ -290,7 +286,7 @@ validateSettings = function validateSettings(defaultSettings, model) {
}
return Promise.resolve();
};
}
/**
* Validate keys using the validator module.
@ -313,7 +309,7 @@ validateSettings = function validateSettings(defaultSettings, model) {
* @param {String} tableName (optional) the db table of the value to validate, used for error message.
* @return {Array} returns an Array including the found validation errors (empty if none found);
*/
validate = function validate(value, key, validations, tableName) {
function validate(value, key, validations, tableName) {
const validationErrors = [];
let translation;
value = _.toString(value);
@ -356,12 +352,12 @@ validate = function validate(value, key, validations, tableName) {
}, this);
return validationErrors;
};
}
module.exports = {
validate: validate,
validator: validator,
validatePassword: validatePassword,
validateSchema: validateSchema,
validateSettings: validateSettings
validate,
validator,
validatePassword,
validateSchema,
validateSettings
};