PR feedback

This commit is contained in:
Steve Walsh 2021-07-13 18:25:05 +01:00
parent 3596c030c8
commit 4a9002afa5

View File

@ -94,25 +94,21 @@ module.exports = class PayloadValidator {
* @returns * @returns
*/ */
ensureOrganizationNotModified(after) { ensureOrganizationNotModified(after) {
try { variablesToCheck.forEach((location) => {
variablesToCheck.forEach((location) => { const beforeValue = this.getValue(this.before, location);
const beforeValue = this.getValue(this.before, location); const afterValue = this.getValue(after, location);
const afterValue = this.getValue(after, location); if (beforeValue !== afterValue) {
if (beforeValue !== afterValue) { this.logException(beforeValue, afterValue, location);
this.logException(beforeValue, afterValue, location);
// attempt to set the value back to its correct one // attempt to set the value back to its correct one
after = this.setValue(after, location, beforeValue); after = this.setValue(after, location, beforeValue);
if (!_.has(after, location, beforeValue)) { if (!_.has(after, location, beforeValue)) {
this.logger.error(`Cant set value as ${location} is no longer assessable in after`); this.logger.error(`Cant set value as ${location} is no longer assessable in after`);
}
} }
}); }
} catch (e) { });
console.log('Error while trying to verify variable changes');
console.log(e);
}
return after; return after;
} }