ofbiz-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jler...@apache.org
Subject svn commit: r1088034 - /ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy
Date Sat, 02 Apr 2011 14:19:25 GMT
Author: jleroux
Date: Sat Apr  2 14:19:24 2011
New Revision: 1088034

URL: http://svn.apache.org/viewvc?rev=1088034&view=rev
A patch from "Product page meta-tag does not contain i18n content" https://issues.apache.org/jira/browse/OFBIZ-4208

the groovy script that produces the content meta information is only partially using i18n
information from the Content Wrapper. Namely, it does NOT use the Product or Category ContentWrapper
to receive internationalized content for Product Name and Description or Category Description
but rather the direct properties of the product entity.
Using the ContentWrapper will overcome this shortcoming.

The fix needs to be applied in applications/order/webapp/ordermgr/WEB-INF/actions/cart/catalog/Product.groovy

It is assumed that ContentWrappers will handle primitive over related Content properly, ie.
if there is no related Content entity but only primitive content, it will return that. And
if there is i18n content missing in the desired (current session) locale, it will return the
default locale translation.


Modified: ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy
URL: http://svn.apache.org/viewvc/ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy?rev=1088034&r1=1088033&r2=1088034&view=diff
--- ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy
+++ ofbiz/branches/release09.04/applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/Product.groovy
Sat Apr  2 14:19:24 2011
@@ -86,8 +86,10 @@ if (productId) {
         members = delegator.findByAndCache("ProductCategoryMember", [productId : productId]);
         members.each { member ->
             category = member.getRelatedOneCache("ProductCategory");
-            if (category.description) {
-                keywords.add(category.description);
+            categoryContentWrapper = new CategoryContentWrapper(category, request);
+            categoryDescription = categoryContentWrapper.DESCRIPTION;
+            if (categoryDescription) {
+                keywords.add(categoryDescription);
         context.metaKeywords = StringUtil.join(keywords, ", ");

View raw message