[
https://issues.apache.org/jira/browse/OFBIZ-4375?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13091875#comment-13091875
]
Atul Vani commented on OFBIZ-4375:
----------------------------------
I'm not sure how much people here care about cleaner and understandable code,
so this patch might never get committed, but if it does get it'll inspire me
more to do the good work. Two patches attached, latest one has got only the
changes supposed to be committed, previous one has comments along with the
changes to explain them.
> TYPO in CategoryTree.groovy, lot's of un-used/un-wanted variables and
> un-clean & really hard to understand code
> ---------------------------------------------------------------------------------------------------------------
>
> Key: OFBIZ-4375
> URL: https://issues.apache.org/jira/browse/OFBIZ-4375
> Project: OFBiz
> Issue Type: Bug
> Components: product
> Reporter: Atul Vani
> Fix For: Release Branch 11.04, SVN trunk
>
> Attachments: OFBIZ-4375.patch, OFBIZ-4375.patch
>
>
> TYPO in CategoryTree.groovy:
> <<prodCatalogMap.put("isCategoryType", true);>> should be
> <<prodCatalogMap2.put("isCategoryType", true);>>
> lot's of un-used/un-wanted variables:
> i, prodCatalogTree2, prodCatalogCategories, prodCatalog, prodCatalogId
> un-clean and really hard to understand code.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira