This is an automated email from the ASF dual-hosted git repository.

mridulpathak pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 52a26bec5d Fixed: CategoryContentServicesScript related-URL, SEO, and 
download-update regressions (OFBIZ-13548)
52a26bec5d is described below

commit 52a26bec5d4de5d4a6890e8815570d37574b6a09
Author: Mridul Pathak <[email protected]>
AuthorDate: Mon Sep 7 15:22:42 2026 +0530

    Fixed: CategoryContentServicesScript related-URL, SEO, and download-update 
regressions (OFBIZ-13548)
    
    - createRelatedUrlContentForCategory: fixed corrupted URL prefix (literal 
XML-escape artifact instead of http://), misspelled DataResource field names, 
and a call to a nonexistent service
    - updateContentSEOForCategory: fixed typo'd metaDescription parameter name 
and a malformed ElectronicText lookup condition
    - updateDownloadContentForCategory: fixed a misspelled service: keyword 
that crashed every metadata update after the file had already been replaced
---
 .../category/CategoryContentServicesScript.groovy        | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git 
a/applications/product/src/main/groovy/org/apache/ofbiz/product/product/category/CategoryContentServicesScript.groovy
 
b/applications/product/src/main/groovy/org/apache/ofbiz/product/product/category/CategoryContentServicesScript.groovy
index b61eb31d34..8093e5c722 100644
--- 
a/applications/product/src/main/groovy/org/apache/ofbiz/product/product/category/CategoryContentServicesScript.groovy
+++ 
b/applications/product/src/main/groovy/org/apache/ofbiz/product/product/category/CategoryContentServicesScript.groovy
@@ -80,7 +80,7 @@ Map createSimpleTextContentForCategory() {
 Map updateContentSEOForCategory() {
     updateContent('title', 'PAGE_TITLE')
     updateContent('metaKeyword', 'META_KEYWORD')
-    updateContent('metaDiscription', 'META_DESCRIPTION')
+    updateContent('metaDescription', 'META_DESCRIPTION')
 }
 
 /**
@@ -93,7 +93,7 @@ Map updateContent(String param, String typeId) {
             .queryList()
         if (productCategoryContents) {
             Map productCategoryContent = 
EntityUtil.getFirst(productCategoryContents)
-            Map electronicText = 
from('ElectronicText').where('dataResourceId', 
productCategoryContent).queryOne()
+            Map electronicText = 
from('ElectronicText').where('dataResourceId', 
productCategoryContent.dataResourceId).queryOne()
             if (electronicText) {
                 electronicText.textData = parameters."${param}"
                 electronicText.store()
@@ -115,12 +115,12 @@ Map updateContent(String param, String typeId) {
 Map createRelatedUrlContentForCategory() {
     String url = parameters.url
     url = url.trim()
-    if (url.indexOf('&quot;http://&quot;') != 0) {
-        url = '&quot;http://&quot;' + url
+    if (!url.startsWith('http://') && !url.startsWith('https://')) {
+        url = 'http://' + url
     }
     Map dataResource = [
-        dataRescourceName: parameters.title,
-        dataRescourceTypeId: 'URL_RESOURCE',
+        dataResourceName: parameters.title,
+        dataResourceTypeId: 'URL_RESOURCE',
         mimeTypeId: 'text/plain',
         objectInfo: url,
         localeString: parameters.localeString
@@ -138,7 +138,7 @@ Map createRelatedUrlContentForCategory() {
     Map contRes = run service: 'createContent', with: content
     parameters.contentId = contRes.contentId
     Map createCategoryContentMap = 
dispatcher.getDispatchContext().makeValidContext('createCategoryContent', 
ModelService.IN_PARAM, parameters)
-    run service: 'createContentCategory', with: createCategoryContentMap
+    run service: 'createCategoryContent', with: createCategoryContentMap
 }
 
 /**
@@ -209,5 +209,5 @@ Map updateDownloadContentForCategory() {
     run service: 'attachUploadToDataResource', with: attachMap
 
     Map updateCategoryContent = 
dispatcher.getDispatchContext().makeValidContext('updateCategoryContent', 
ModelService.IN_PARAM, parameters)
-    run sevrice: 'updateCategoryContent', with: updateCategoryContent
+    run service: 'updateCategoryContent', with: updateCategoryContent
 }

Reply via email to