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

Michael Vorburger commented on FINERACT-1245:
---------------------------------------------

Oh, I forgot, this is just because of that {{options.compilerArgs -= 
["-Werror"]}} in {{fineract-client/build.gradle}}... which you had to put there 
because _the generated code contains some warnings in 
build/generated/java/src/main/java/org/apache/fineract/client/auth/OAuthOkHttpClient.java#L71
 that won't pass_.

A better solution than removing {{-Werror}} could be just to remove 
{{deprecation}}. I've tried that like this, but it didn't work:

{{options.compilerArgs -= ["-Xlint:deprecation"]}}
{{options.compilerArgs += ["-Xlint:-deprecation"]}}

> Error Prone Bug Pattern JavaTimeDefaultTimeZone should be Error not just 
> Warning
> --------------------------------------------------------------------------------
>
>                 Key: FINERACT-1245
>                 URL: https://issues.apache.org/jira/browse/FINERACT-1245
>             Project: Apache Fineract
>          Issue Type: Bug
>            Reporter: Michael Vorburger
>            Assignee: Aleksandar Vidakovic
>            Priority: Major
>             Fix For: 1.5.0
>
>
> This should be configured to be an Error, not just a Warning, and the 
> occurrence fixed:
> {noformat}> Task :fineract-client:compileTestJava                             
>                                                                   
> /home/vorburger/dev/Mifos/fineract/fineract-client/src/test/java/org/apache/fineract/integrationtests/newstyle/OfficeTest.java:40:
>  warning: [JavaTimeDefaultTimeZone] LocalDate.now() is not allowed because it 
> silently uses the system default time-zone. You must pass an explicit 
> time-zone (e.g., ZoneId.of("America/Los_Angeles")) to this method.            
>                                               
>                 
> .openingDate(LocalDate.now()).dateFormat(dateFormat()).locale("en_US"))).getOfficeId()).isGreaterThan(0);
>              
>                                           ^                                   
>                                                         
>     (see https://errorprone.info/bugpattern/JavaTimeDefaultTimeZone)          
>                                                         
>   Did you mean 
> '.openingDate(LocalDate.now(ZoneId.systemDefault())).dateFormat(dateFormat()).locale("en_US"))).getOfficeId()).isGreaterThan(0);'?
>                                                                               
>                                              
> 1 warning{noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to