[ https://issues.apache.org/jira/browse/OFBIZ-4728?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14501315#comment-14501315 ]
Deepak Dixit edited comment on OFBIZ-4728 at 4/18/15 11:51 AM: --------------------------------------------------------------- Based on Ravi comments, closing the issue. was (Author: deepak.dixit): As per the Ravi comments, closing the issue. > productdetail.ftl shows only primaryProductCategoryId even though a > productCategoryId is handed over > ---------------------------------------------------------------------------------------------------- > > Key: OFBIZ-4728 > URL: https://issues.apache.org/jira/browse/OFBIZ-4728 > Project: OFBiz > Issue Type: Bug > Components: specialpurpose/ecommerce > Affects Versions: Trunk > Reporter: Markus M. May > Priority: Minor > > On the productsummary.ftl the productUrl is constructed like: > <#assign productUrl><@ofbizCatalogAltUrl productId=product.productId > productCategoryId=categoryId/></#assign> > This works fine, unfortunately in the ProductDetail.groovy the following > assignment returns null if the primaryProductCategoryId is not set on the > product: > // get next/previous information for category > categoryId = parameters.category_id ?: product.primaryProductCategoryId; > This leads to a nasty NullPointerException in the CatalogScreens.xml. > I would propose the following solution in the > applications/order/webapp/ordermgr/WEB-INF/actions/entry/catalog/ProductDetail.groovy: > // get next/previous information for category > categoryId = parameters.productCategoryId ?: parameters.category_id ?: > product.primaryProductCategoryId; > I am unsure, why the parameter is named productCategoryId after the call of > the ofbizCatalogAltUrl, whereas it is named category throughout the rest of > the application. Another fix could be to rename this parameter in the > ofbizCatalogAltUrl. > What do you think? -- This message was sent by Atlassian JIRA (v6.3.4#6332)