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('"http://"') != 0) {
- url = '"http://"' + 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
}