Merge pull request #1138 from HifiExperiments/emptyField

fix accidentally clearing url fields when you don't have view permission
overte-org · Sep 22, 2024 · 542b231 · 542b231
2 parents 808135b + 79cfa68
commit 542b231
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/scripts/system/create/entityProperties/html/js/entityProperties.js b/scripts/system/create/entityProperties/html/js/entityProperties.js
@@ -2365,6 +2365,11 @@ function updateMultiDiffProperties(propertiesMapToUpdate, onlyUpdateEntity) {
 
 function createEmitTextPropertyUpdateFunction(property) {
     return function() {
+        // If we don't have canViewAssetURLs permissions, ignore clearing URLs
+        if (!canViewAssetURLs && property.data.placeholder === "URL" && this.value === "") {
+            return;
+        }
+
         property.elInput.classList.remove('multi-diff');
         updateProperty(property.name, this.value, property.isParticleProperty);
     };