dual api support for http client api in work item handler

Actually it should be seen as a constant from outside, the only reason it is not final it's to be able to easy test both options. And any way it won't be seen from api so I opt to leave it as is. 

When it comes to formatting it turned out I need to reapply templates after upgrade of IDE. 


Dnia 21 lis 2014 o godz. 14:59 Petr Široký <notifications@github.com> napisał(a):

> In jbpm-workitems/src/main/java/org/jbpm/process/workitem/rest/RESTWorkItemHandler.java:
> 
> > @@ -89,11 +97,23 @@
> >  	private String password;
> >  	private AuthenticationType type;
> >  	private String authUrl;	
> > +	// protected for test purpose
> > +	protected static boolean HTTP_CLIENT_API_43 = true;
> I guess this can't be final because it is needed for the testing (as comment says). I think it would be better to call it httpClientApi43 as the UPPER_CASE by convention suggests this is a constant, which in fact is not. WDYT?
> 
> —
> Reply to this email directly or view it on GitHub.
>