[
https://issues.apache.org/jira/browse/RAT-148?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Philipp Ottlinger resolved RAT-148.
-----------------------------------
Resolution: Fixed
Checked in as rev1612959. Thanks for reporting this issue.
> LicenseAddingReport#report has useless call to metaData.getData
> ---------------------------------------------------------------
>
> Key: RAT-148
> URL: https://issues.apache.org/jira/browse/RAT-148
> Project: Apache Rat
> Issue Type: Bug
> Components: reports
> Reporter: Chris A. Mattmann
> Assignee: Philipp Ottlinger
> Fix For: 0.11
>
> Attachments: RAT-148.Mattmann.082813.patch.txt
>
>
> While perusing the current trunk code, I noticed that
> LicenseAddingReport.java#report has a useless call to metaData.getData:
> {code:java}
> public class LicenseAddingReport extends AbstractReport {
> private final AbstractLicenceAppender appender;
> public LicenseAddingReport(String pCopyrightMsg, boolean pForced) {
> appender = pCopyrightMsg == null ? new ApacheV2LicenceAppender() :
> new ApacheV2LicenceAppender(pCopyrightMsg);
> appender.setForce(pForced);
> }
> @Override
> public void report(org.apache.rat.api.Document document) throws
> RatException {
> final MetaData metaData = document.getMetaData();
> final Datum licenseHeader =
> metaData.get(MetaData.RAT_URL_HEADER_CATEGORY);
> if (licenseHeader == null
> ||
> MetaData.RAT_LICENSE_FAMILY_CATEGORY_DATUM_UNKNOWN.getValue().equals(licenseHeader.getValue()))
> {
> final File file = new File(document.getName());
> if (file.isFile()) {
> try {
> appender.append(file);
> } catch (IOException e) {
> throw new RatException(e.getMessage(), e);
> }
> }
> }
> metaData.getData();
> }
> }
> {code}
> Looks to me that metaData is a local final variable, and that the call to
> getData returns a MetaData object, which in turn is never used.
--
This message was sent by Atlassian JIRA
(v6.2#6252)
