Mailing-List: contact issues-help@maven.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@maven.apache.org
Received-SPF: pass (athena.apache.org: local policy)
Date: Tue, 20 Jan 2015 15:08:18 -0600 (CST)
From: "Michael Osipov (JIRA)" <jira@codehaus.org>
To: issues@maven.apache.org
Message-ID: 
 <JIRA.158651.1419291088735.5039.1421788098167@codehaus01.managed.contegix.com>
In-Reply-To: <JIRA.158651.1419291088735@codehaus01.managed.contegix.com>
References: <JIRA.158651.1419291088735@codehaus01.managed.contegix.com>
Subject: [jira] (MPMD-199) Support PMD functionality on JSP files
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit


    [ https://jira.codehaus.org/browse/MPMD-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=361482#comment-361482 ] 

Michael Osipov commented on MPMD-199:
-------------------------------------

Tom, way better now. A few issues though:

1. {{sample.jsp}} is added twice, ins't it?
2. You copied the javascript apt file to {{jspReport.apt.vm}} but did not alter its content.
3. You either need to add the JSP module to the plugins dependencies and/or make it optional and tell that actually in the docs.

> Support PMD functionality on JSP files
> --------------------------------------
>
>                 Key: MPMD-199
>                 URL: https://jira.codehaus.org/browse/MPMD-199
>             Project: Maven PMD Plugin
>          Issue Type: Improvement
>          Components: PMD
>    Affects Versions: 3.3
>            Reporter: Tom Williamson
>            Priority: Minor
>         Attachments: Jsp_support_201501201341.patch, Patch_for_JSP_support.patch
>
>
> It would be great if this plugin would either support PMD functionality for JSP files, or simply enable them as part of the Java support.   


--
This message was sent by Atlassian JIRA
(v6.1.6#6162)