[ 
https://issues.apache.org/jira/browse/OFBIZ-3984?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Jacques Le Roux closed OFBIZ-3984.
----------------------------------

    Resolution: Fixed
      Assignee: Jacques Le Roux

Sascha,

Your patch is in trunk at r1021344

I wondered why you removed the _if (UtilValidate.isNotEmpty(delegatorName)) {_ 
check, but found that anyway _DelegatorFactory.getDelegator(_ will pick 
_"default"_ in such case

> Server Hit Bin, counterHit receive delegator
> --------------------------------------------
>
>                 Key: OFBIZ-3984
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-3984
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ALL COMPONENTS
>    Affects Versions: SVN trunk
>            Reporter: Sascha Rodekamp
>            Assignee: Jacques Le Roux
>             Fix For: SVN trunk
>
>         Attachments: OFBIZ-3984_serverHitBin.patch
>
>
> Hi,
> here is a little patch to improve the delegator call in the countHit method.
> The method got the delegator name from the session and get with this 
> delegator name over the delegator factory the delegator.
> But when the delegator name is not available in the session the method 
> crashes. 
> I try to get the delegator from the request, and if their is no delegator 
> object i go the factory way.
> I think it's a little bit saver to get an delegator object over this way
> Have a nice day
> Sascha

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to