[jira] [Commented] (OFBIZ-11127) Code improvement in ShoppingCartEvents.java

2019-07-26 Thread Amit Gadaley (JIRA)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11127?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16894286#comment-16894286
 ] 

Amit Gadaley commented on OFBIZ-11127:
--

Provided updated patch. Thanks [~pawan.verma] for your inputs.

> Code improvement in ShoppingCartEvents.java
> ---
>
> Key: OFBIZ-11127
> URL: https://issues.apache.org/jira/browse/OFBIZ-11127
> Project: OFBiz
>  Issue Type: Improvement
>  Components: order
>Affects Versions: Trunk
>Reporter: Amit Gadaley
>Assignee: Pawan Verma
>Priority: Minor
> Attachments: OFBIZ-11127.patch
>
>
> Method 'getCartObject' of ShoppingCartEvents class accessed incorrectly at 
> some places in ShoppingCartEvents class.
> It should be called by following below syntax:
> getCartObject(parameters);
>  
> But, at some places, it is called incorrectly like this:
> ShoppingCartEvents.getCartObject(parameters)
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)


[jira] [Commented] (OFBIZ-11127) Code improvement in ShoppingCartEvents.java

2019-06-27 Thread Pawan Verma (JIRA)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11127?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16873963#comment-16873963
 ] 

Pawan Verma commented on OFBIZ-11127:
-

Hi [~amit.gadaley],

Thanks for the patch, I have reviewed it and I think we can also improve it for 
destroyCart() method, Please let me know your thoughts on this. Thanks!

> Code improvement in ShoppingCartEvents.java
> ---
>
> Key: OFBIZ-11127
> URL: https://issues.apache.org/jira/browse/OFBIZ-11127
> Project: OFBiz
>  Issue Type: Improvement
>  Components: order
>Affects Versions: Trunk
>Reporter: Amit Gadaley
>Assignee: Pawan Verma
>Priority: Minor
> Attachments: OFBIZ-11127.patch
>
>
> Method 'getCartObject' of ShoppingCartEvents class accessed incorrectly at 
> some places in ShoppingCartEvents class.
> It should be called by following below syntax:
> getCartObject(parameters);
>  
> But, at some places, it is called incorrectly like this:
> ShoppingCartEvents.getCartObject(parameters)
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (OFBIZ-11127) Code improvement in ShoppingCartEvents.java

2019-06-26 Thread Amit Gadaley (JIRA)


[ 
https://issues.apache.org/jira/browse/OFBIZ-11127?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16873862#comment-16873862
 ] 

Amit Gadaley commented on OFBIZ-11127:
--

Provided patch.

> Code improvement in ShoppingCartEvents.java
> ---
>
> Key: OFBIZ-11127
> URL: https://issues.apache.org/jira/browse/OFBIZ-11127
> Project: OFBiz
>  Issue Type: Improvement
>  Components: order
>Affects Versions: Trunk
>Reporter: Amit Gadaley
>Assignee: Amit Gadaley
>Priority: Minor
> Attachments: OFBIZ-11127.patch
>
>
> Method 'getCartObject' of ShoppingCartEvents class accessed incorrectly at 
> some places in ShoppingCartEvents class.
> It should be called by following below syntax:
> getCartObject(parameters);
>  
> But, at some places, it is called incorrectly like this:
> ShoppingCartEvents.getCartObject(parameters)
>  
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)