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

Bob Fields updated MCHECKSTYLE-294:
-----------------------------------
    Attachment: MCHECKSTYLE-294.java.patch

Checks for null value in licenseArtifact before getting the File URL. Logs a 
warning if license does not exist. Changes MalformedURLException to any 
Exception catch and noop/warn, because errors other than MalformedURL could 
occur, just to be safe.

> NPE on missing license artifact, not fixed previously
> -----------------------------------------------------
>
>                 Key: MCHECKSTYLE-294
>                 URL: https://issues.apache.org/jira/browse/MCHECKSTYLE-294
>             Project: Maven Checkstyle Plugin
>          Issue Type: Bug
>          Components: checkstyle:checkstyle
>    Affects Versions: 2.15, 2.16
>         Environment: Windows7 x64 JDK8 Checkstyle 6.2 maven-checkstyle-plugin 
> 2.15 and 2.16-SNAPSHOT as of 5/20/2015.
>            Reporter: Bob Fields
>              Labels: easyfix, patch
>             Fix For: 2.16
>
>         Attachments: MCHECKSTYLE-294.java.patch
>
>
> MCHECKSTYLE-250 fix does not check for null values, only for invalid URL. 
> This patch checks for null values and fixes the original problem, and emits a 
> warning if the license file is missing (but does not kill the checkstyle 
> process or site build). Interestingly - this error only occurs on 'mvn site', 
> not on 'mvn checkstyle:checkstyle'.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to