From e6b29068c3082f7b8fb12f16ab06ec9e76b8d257 Mon Sep 17 00:00:00 2001
From: Martii <martii@users.noreply.github.com>
Date: Sat, 1 Mar 2025 19:59:36 -0700
Subject: [PATCH] Restore previous naming conventions

* Revert prior fix as it broke things instead of clearing up identifier naming.
* Change also how the horizontal ellipsis shows up.

Post #2056 #2048

* We'll try this again.
---
 controllers/scriptStorage.js | 22 +++++++++++++---------
 libs/modelParser.js          |  4 ++--
 2 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/controllers/scriptStorage.js b/controllers/scriptStorage.js
index 53e5c7d84..99a38ceb8 100644
--- a/controllers/scriptStorage.js
+++ b/controllers/scriptStorage.js
@@ -1201,6 +1201,8 @@ exports.storeScript = function (aUser, aMeta, aBuf, aUpdate, aCallback) {
   var userName = findMeta(aMeta, 'OpenUserJS.author.0.value') || aUser.name;
   var scriptName = null;
   var scriptDescription = null;
+  var thisName = null;
+  var thisDescription = null;
   var hasInvalidKey = null;
 
   async.series([
@@ -1251,7 +1253,8 @@ exports.storeScript = function (aUser, aMeta, aBuf, aUpdate, aCallback) {
       // Check for non-localized presence
       name.forEach(function (aElement, aIndex, aArray) {
         if (!name[aIndex].key) {
-          scriptName = aElement.value;
+          thisName = aElement.value;
+          scriptName = cleanFilename(thisName, '');
         }
       });
 
@@ -1311,7 +1314,8 @@ exports.storeScript = function (aUser, aMeta, aBuf, aUpdate, aCallback) {
       if (description) {
         description.forEach(function (aElement, aIndex, aArray) {
           if (!description[aIndex].key) {
-            scriptDescription = aElement.value;
+            thisDescription = aElement.value;
+            scriptDescription = cleanFilename(thisDescription, '');
           }
         });
       }
@@ -2083,15 +2087,15 @@ exports.storeScript = function (aUser, aMeta, aBuf, aUpdate, aCallback) {
 
 
           storeDescriptionLength = settings.scriptSearchQueryStoreMaxDescription;
-          storeDescriptionLength = rLogographic.test(scriptDescription)
+          storeDescriptionLength = rLogographic.test(thisDescription)
             ? parseInt(storeDescriptionLength / logographicDivisor)
             : storeDescriptionLength;
 
           aScript = new Script({
-            name: scriptName,
+            name: thisName,
             _description: (
-              scriptDescription
-                ? scriptDescription.substr(0, storeDescriptionLength).trim()
+              thisDescription
+                ? thisDescription.substr(0, storeDescriptionLength).trim()
                 : ''
             ),
             author: aUser.name,
@@ -2133,13 +2137,13 @@ exports.storeScript = function (aUser, aMeta, aBuf, aUpdate, aCallback) {
           }
 
           storeDescriptionLength = settings.scriptSearchQueryStoreMaxDescription;
-          storeDescriptionLength = rLogographic.test(scriptDescription)
+          storeDescriptionLength = rLogographic.test(thisDescription)
             ? parseInt(storeDescriptionLength / logographicDivisor)
             : storeDescriptionLength;
 
           aScript._description = (
-            scriptDescription
-              ? scriptDescription.substr(0, storeDescriptionLength).trim()
+            thisDescription
+              ? thisDescription.substr(0, storeDescriptionLength).trim()
               : ''
           );
           aScript.meta = aMeta;
diff --git a/libs/modelParser.js b/libs/modelParser.js
index 228026164..83fcb825d 100644
--- a/libs/modelParser.js
+++ b/libs/modelParser.js
@@ -351,8 +351,8 @@ var parseScript = function (aScript) {
     if (script.description && script._description
       && script.description.length && script.description.length > logographicDivisor
         && script.description.length > (logographic
-          ? parseInt(script._description.length / logographicDivisor)
-          : script._description.length)) {
+          ? parseInt(settings.scriptSearchQueryStoreMaxDescription / logographicDivisor)
+          : settings.scriptSearchQueryStoreMaxDescription)) {
             script.hasLongDescription = true;
     }
   }