Validation in api did accept nulls but did not handle them correctly …

…in case of trimming input
openstad · Nov 9, 2023 · 9ab202a · 9ab202a · github-actions · Nov 9, 2023
1 parent ba0fcbc
commit 9ab202a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/models/Idea.js b/src/models/Idea.js
@@ -210,7 +210,7 @@ module.exports = function (db, sequelize, DataTypes) {
         validate: {
           textLength(value) {
             // We need to undo the sanitization before we can check the length
-            let len = htmlToText.fromString(value).length;
+            let len = htmlToText.fromString(value || '').length;
             let summaryMinLength =
               (this.config &&
                 this.config.ideas &&
@@ -241,7 +241,7 @@ module.exports = function (db, sequelize, DataTypes) {
         allowNull: !this.publishDate,
         validate: {
           textLength(value) {
-            let len = sanitize.summary(value.trim()).length;
+            let len = sanitize.summary(value || ''.trim()).length;
             let descriptionMinLength =
               (this.config &&
                 this.config.ideas &&