pmd / pmd / 440

Committed 24 Mar 2026 06:03PM UTC coverage: 79.047% (+0.007%) from 79.04%

Build # 440

Build Type

push

github

Committed by

web-flow

Commit Message

[core] Fix #6503: Don't escape externalInfoUrl in reports (#6504)

Run Details

18607 of 24437 branches covered (76.14%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

40602 of 50467 relevant lines covered (80.45%)

0.81 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

94.12

/pmd-core/src/main/java/net/sourceforge/pmd/renderers/HTMLRenderer.java

/*
 * BSD-style license; for more info see http://pmd.sourceforge.net/license.html
 */

package net.sourceforge.pmd.renderers;

import java.io.IOException;
import java.io.PrintWriter;
import java.io.Writer;
import java.net.URI;
import java.util.Iterator;
import java.util.List;
import java.util.Optional;

import org.apache.commons.lang3.StringEscapeUtils;
import org.apache.commons.lang3.StringUtils;

import net.sourceforge.pmd.lang.document.FileId;
import net.sourceforge.pmd.lang.rule.Rule;
import net.sourceforge.pmd.properties.PropertyDescriptor;
import net.sourceforge.pmd.properties.PropertyFactory;
import net.sourceforge.pmd.reporting.Report;
import net.sourceforge.pmd.reporting.Report.ConfigurationError;
import net.sourceforge.pmd.reporting.RuleViolation;

/**
 * Renderer to basic HTML format.
 *
 * FIXME: this class should just work with the XMLRenderer and then apply an
 * XSLT transformation + stylesheet. No need to hard-code HTML markup here.
 */
public class HTMLRenderer extends AbstractIncrementingRenderer {

    public static final String NAME = "html";

    public static final PropertyDescriptor<Optional<String>> LINE_PREFIX =
        PropertyFactory.stringProperty("linePrefix")
                       .desc("Prefix for line number anchor in the source file.")
                       .toOptional("<none>")
                       .defaultValue(Optional.empty())
                       .build();

    public static final PropertyDescriptor<String> LINK_PREFIX =
        PropertyFactory.stringProperty("linkPrefix").desc("Path to HTML source.").defaultValue("").build();

    public static final PropertyDescriptor<Boolean> HTML_EXTENSION =
        PropertyFactory.booleanProperty("htmlExtension")
                       .desc("Replace file extension with .html for the links.")
                       // default value is false - to have the old (pre 6.23.0) behavior, this needs to be set to true.
                       .defaultValue(false)
                       .build();

    private String linkPrefix;
    private String linePrefix;
    private boolean replaceHtmlExtension;

    boolean colorize = true;

    public HTMLRenderer() {
        super(NAME, "HTML format");

        definePropertyDescriptor(LINK_PREFIX);
        definePropertyDescriptor(LINE_PREFIX);
        definePropertyDescriptor(HTML_EXTENSION);
    }

    @Override
    public String defaultFileExtension() {
        return "html";
    }

    private static String escape(String s) {
        return StringEscapeUtils.escapeHtml4(s);
    }

    /**
     * Write the body of the main body of the HTML content.
     */
    public void renderBody(PrintWriter writer, Report report) throws IOException {
        linkPrefix = getProperty(LINK_PREFIX);
        linePrefix = getProperty(LINE_PREFIX).orElse(null);
        replaceHtmlExtension = getProperty(HTML_EXTENSION);

        writer.write("<center><h3>PMD report</h3></center>");
        writer.write("<center><h3>Problems found</h3></center>");
        writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");
        writer.println("<th>#</th><th>File</th><th>Line</th><th>Problem</th></tr>");
        setWriter(writer);
        renderFileReport(report);
        writer.write("</table>");
        glomProcessingErrors(writer, errors);
        if (showSuppressedViolations) {
            glomSuppressions(writer, suppressed);
        }
        glomConfigurationErrors(writer, configErrors);
    }

    @Override
    public void start() throws IOException {
        linkPrefix = getProperty(LINK_PREFIX);
        linePrefix = getProperty(LINE_PREFIX).orElse(null);
        replaceHtmlExtension = getProperty(HTML_EXTENSION);

        writer.println("<html><head><title>PMD</title></head><body>");
        writer.write("<center><h3>PMD report</h3></center>");
        writer.write("<center><h3>Problems found</h3></center>");
        writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");
        writer.println("<th>#</th><th>File</th><th>Line</th><th>Problem</th></tr>");
    }

    @Override
    public void renderFileViolations(Iterator<RuleViolation> violations) throws IOException {
        glomRuleViolations(writer, violations);
    }

    @Override
    public void end() throws IOException {
        writer.write("</table>");
        glomProcessingErrors(writer, errors);
        if (showSuppressedViolations) {
            glomSuppressions(writer, suppressed);
        }
        glomConfigurationErrors(writer, configErrors);
        writer.println("</body></html>");
    }

    private void glomRuleViolations(Writer writer, Iterator<RuleViolation> violations) throws IOException {
        int violationCount = 1;

        StringBuilder buf = new StringBuilder(500);

        while (violations.hasNext()) {
            RuleViolation rv = violations.next();
            buf.setLength(0);
            buf.append("<tr");
            if (colorize) {
                buf.append(" bgcolor=\"lightgrey\"");
            }
            colorize = !colorize;
            buf.append("> ").append(System.lineSeparator());
            buf.append("<td align=\"center\">").append(violationCount).append("</td>").append(System.lineSeparator());
            buf.append("<td width=\"*%\">")
               .append(renderFileNameEscaped(rv.getFileId(), rv.getBeginLine()))
               .append("</td>")
                .append(System.lineSeparator());
            buf.append("<td align=\"center\" width=\"5%\">").append(rv.getBeginLine()).append("</td>").append(System.lineSeparator());

            String d = escape(rv.getDescription());

            String infoUrl = rv.getRule().getExternalInfoUrl();
            if (StringUtils.isNotBlank(infoUrl)) {
                URI uri = URI.create(infoUrl);
                d = "<a href=\"" + uri + "\">" + d + "</a>";
            }
            buf.append("<td width=\"*\">")
               .append(d)
               .append("</td>")
               .append(System.lineSeparator())
               .append("</tr>")
                .append(System.lineSeparator());
            writer.write(buf.toString());
            violationCount++;
        }
    }

    private String renderFileNameEscaped(FileId fileId, int beginLine) {
        return maybeWrap(escape(determineFileName(fileId)),
                linePrefix == null || beginLine < 0 ? "" : linePrefix + beginLine);
    }

    private String renderRuleNameEscaped(Rule rule) {
        String name = escape(rule.getName());
        String infoUrl = rule.getExternalInfoUrl();
        if (StringUtils.isNotBlank(infoUrl)) {
            return "<a href=\"" + infoUrl + "\">" + name + "</a>";
        }
        return name;
    }

    private void glomProcessingErrors(PrintWriter writer, List<Report.ProcessingError> errors) throws IOException {

        if (errors.isEmpty()) {
            return;
        }

        writer.write("<hr/>");
        writer.write("<center><h3>Processing errors</h3></center>");
        writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");
        writer.println("<th>File</th><th>Problem</th></tr>");

        StringBuilder buf = new StringBuilder(500);
        boolean colorize = true;
        for (Report.ProcessingError pe : errors) {
            buf.setLength(0);
            buf.append("<tr");
            if (colorize) {
                buf.append(" bgcolor=\"lightgrey\"");
            }
            colorize = !colorize;
            buf.append("> ").append(System.lineSeparator());
            buf.append("<td>").append(renderFileNameEscaped(pe.getFileId(), -1)).append("</td>").append(System.lineSeparator());
            buf.append("<td><pre>").append(escape(pe.getDetail())).append("</pre></td>").append(System.lineSeparator());
            buf.append("</tr>").append(System.lineSeparator());
            writer.write(buf.toString());
        }
        writer.write("</table>");
    }

    private void glomSuppressions(PrintWriter writer, List<Report.SuppressedViolation> suppressed) throws IOException {
        if (suppressed.isEmpty()) {
            return;
        }

        writer.write("<hr/>");
        writer.write("<center><h3>Suppressed warnings</h3></center>");
        writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");
        writer.println("<th>File</th><th>Line</th><th>Rule</th><th>NOPMD or Annotation</th><th>Reason</th></tr>");

        StringBuilder buf = new StringBuilder(500);
        boolean colorize = true;
        for (Report.SuppressedViolation sv : suppressed) {
            buf.setLength(0);
            buf.append("<tr");
            if (colorize) {
                buf.append(" bgcolor=\"lightgrey\"");
            }
            colorize = !colorize;
            buf.append("> ").append(System.lineSeparator());
            RuleViolation rv = sv.getRuleViolation();
            String userMessage = Optional.ofNullable(sv.getUserMessage()).orElse("");
            buf.append("<td align=\"left\">").append(renderFileNameEscaped(rv.getFileId(), rv.getBeginLine())).append("</td>").append(System.lineSeparator());
            buf.append("<td align=\"center\">").append(rv.getBeginLine()).append("</td>").append(System.lineSeparator());
            buf.append("<td align=\"center\">").append(renderRuleNameEscaped(rv.getRule())).append("</td>").append(System.lineSeparator());
            buf.append("<td align=\"center\">").append(escape(sv.getSuppressor().getId())).append("</td>").append(System.lineSeparator());
            buf.append("<td align=\"center\">").append(escape(userMessage)).append("</td>").append(System.lineSeparator());
            buf.append("</tr>").append(System.lineSeparator());
            writer.write(buf.toString());
        }
        writer.write("</table>");
    }

    private void glomConfigurationErrors(final PrintWriter writer, final List<ConfigurationError> configErrors) throws IOException {
        if (configErrors.isEmpty()) {
            return;
        }

        writer.write("<hr/>");
        writer.write("<center><h3>Configuration errors</h3></center>");
        writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");
        writer.println("<th>Rule</th><th>Problem</th></tr>");

        StringBuilder buf = new StringBuilder(500);
        boolean colorize = true;
        for (Report.ConfigurationError ce : configErrors) {
            buf.setLength(0);
            buf.append("<tr");
            if (colorize) {
                buf.append(" bgcolor=\"lightgrey\"");
            }
            colorize = !colorize;
            buf.append("> ").append(System.lineSeparator());
            buf.append("<td>").append(renderRuleNameEscaped(ce.rule())).append("</td>").append(System.lineSeparator());
            buf.append("<td>").append(escape(ce.issue())).append("</td>").append(System.lineSeparator());
            buf.append("</tr>").append(System.lineSeparator());
            writer.write(buf.toString());
        }
        writer.write("</table>");
    }

    private String maybeWrap(String filenameEscaped, String line) {
        if (StringUtils.isBlank(linkPrefix)) {
            return filenameEscaped;
        }
        String newFileName = filenameEscaped.replace('\\', '/');

        if (replaceHtmlExtension) {
            int index = filenameEscaped.lastIndexOf('.');
            if (index >= 0) {
                newFileName = filenameEscaped.substring(0, index);
            }
        }

        return "<a href=\"" + linkPrefix + newFileName + (replaceHtmlExtension ? ".html#" : "#") + line + "\">"
            + newFileName + "</a>";
    }
}

1	/*
2	* BSD-style license; for more info see http://pmd.sourceforge.net/license.html
3	*/
4
5	package net.sourceforge.pmd.renderers;
6
7	import java.io.IOException;
8	import java.io.PrintWriter;
9	import java.io.Writer;
10	import java.net.URI;
11	import java.util.Iterator;
12	import java.util.List;
13	import java.util.Optional;
14
15	import org.apache.commons.lang3.StringEscapeUtils;
16	import org.apache.commons.lang3.StringUtils;
17
18	import net.sourceforge.pmd.lang.document.FileId;
19	import net.sourceforge.pmd.lang.rule.Rule;
20	import net.sourceforge.pmd.properties.PropertyDescriptor;
21	import net.sourceforge.pmd.properties.PropertyFactory;
22	import net.sourceforge.pmd.reporting.Report;
23	import net.sourceforge.pmd.reporting.Report.ConfigurationError;
24	import net.sourceforge.pmd.reporting.RuleViolation;
25
26	/**
27	* Renderer to basic HTML format.
28	*
29	* FIXME: this class should just work with the XMLRenderer and then apply an
30	* XSLT transformation + stylesheet. No need to hard-code HTML markup here.
31	*/
32	public class HTMLRenderer extends AbstractIncrementingRenderer {
33
34	public static final String NAME = "html";
35
36	public static final PropertyDescriptor<Optional<String>> LINE_PREFIX =	1✔
37	PropertyFactory.stringProperty("linePrefix")	1✔
38	.desc("Prefix for line number anchor in the source file.")	1✔
39	.toOptional("<none>")	1✔
40	.defaultValue(Optional.empty())	1✔
41	.build();	1✔
42
43	public static final PropertyDescriptor<String> LINK_PREFIX =	1✔
44	PropertyFactory.stringProperty("linkPrefix").desc("Path to HTML source.").defaultValue("").build();	1✔
45
46	public static final PropertyDescriptor<Boolean> HTML_EXTENSION =	1✔
47	PropertyFactory.booleanProperty("htmlExtension")	1✔
48	.desc("Replace file extension with .html for the links.")	1✔
49	// default value is false - to have the old (pre 6.23.0) behavior, this needs to be set to true.
50	.defaultValue(false)	1✔
51	.build();	1✔
52
53	private String linkPrefix;
54	private String linePrefix;
55	private boolean replaceHtmlExtension;
56
57	boolean colorize = true;	1✔
58
59	public HTMLRenderer() {
60	super(NAME, "HTML format");	1✔
61
62	definePropertyDescriptor(LINK_PREFIX);	1✔
63	definePropertyDescriptor(LINE_PREFIX);	1✔
64	definePropertyDescriptor(HTML_EXTENSION);	1✔
65	}	1✔
66
67	@Override
68	public String defaultFileExtension() {
69	return "html";	×
70	}
71
72	private static String escape(String s) {
73	return StringEscapeUtils.escapeHtml4(s);	1✔
74	}
75
76	/**
77	* Write the body of the main body of the HTML content.
78	*/
79	public void renderBody(PrintWriter writer, Report report) throws IOException {
80	linkPrefix = getProperty(LINK_PREFIX);	1✔
81	linePrefix = getProperty(LINE_PREFIX).orElse(null);	1✔
82	replaceHtmlExtension = getProperty(HTML_EXTENSION);	1✔
83
84	writer.write("<center><h3>PMD report</h3></center>");	1✔
85	writer.write("<center><h3>Problems found</h3></center>");	1✔
86	writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");	1✔
87	writer.println("<th>#</th><th>File</th><th>Line</th><th>Problem</th></tr>");	1✔
88	setWriter(writer);	1✔
89	renderFileReport(report);	1✔
90	writer.write("</table>");	1✔
91	glomProcessingErrors(writer, errors);	1✔
92	if (showSuppressedViolations) {	1✔
93	glomSuppressions(writer, suppressed);	1✔
94	}
95	glomConfigurationErrors(writer, configErrors);	1✔
96	}	1✔
97
98	@Override
99	public void start() throws IOException {
100	linkPrefix = getProperty(LINK_PREFIX);	1✔
101	linePrefix = getProperty(LINE_PREFIX).orElse(null);	1✔
102	replaceHtmlExtension = getProperty(HTML_EXTENSION);	1✔
103
104	writer.println("<html><head><title>PMD</title></head><body>");	1✔
105	writer.write("<center><h3>PMD report</h3></center>");	1✔
106	writer.write("<center><h3>Problems found</h3></center>");	1✔
107	writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");	1✔
108	writer.println("<th>#</th><th>File</th><th>Line</th><th>Problem</th></tr>");	1✔
109	}	1✔
110
111	@Override
112	public void renderFileViolations(Iterator<RuleViolation> violations) throws IOException {
113	glomRuleViolations(writer, violations);	1✔
114	}	1✔
115
116	@Override
117	public void end() throws IOException {
118	writer.write("</table>");	1✔
119	glomProcessingErrors(writer, errors);	1✔
120	if (showSuppressedViolations) {	1!
121	glomSuppressions(writer, suppressed);	1✔
122	}
123	glomConfigurationErrors(writer, configErrors);	1✔
124	writer.println("</body></html>");	1✔
125	}	1✔
126
127	private void glomRuleViolations(Writer writer, Iterator<RuleViolation> violations) throws IOException {
128	int violationCount = 1;	1✔
129
130	StringBuilder buf = new StringBuilder(500);	1✔
131
132	while (violations.hasNext()) {	1✔
133	RuleViolation rv = violations.next();	1✔
134	buf.setLength(0);	1✔
135	buf.append("<tr");	1✔
136	if (colorize) {	1✔
137	buf.append(" bgcolor=\"lightgrey\"");	1✔
138	}
139	colorize = !colorize;	1✔
140	buf.append("> ").append(System.lineSeparator());	1✔
141	buf.append("<td align=\"center\">").append(violationCount).append("</td>").append(System.lineSeparator());	1✔
142	buf.append("<td width=\"*%\">")	1✔
143	.append(renderFileNameEscaped(rv.getFileId(), rv.getBeginLine()))	1✔
144	.append("</td>")	1✔
145	.append(System.lineSeparator());	1✔
146	buf.append("<td align=\"center\" width=\"5%\">").append(rv.getBeginLine()).append("</td>").append(System.lineSeparator());	1✔
147
148	String d = escape(rv.getDescription());	1✔
149
150	String infoUrl = rv.getRule().getExternalInfoUrl();	1✔
151	if (StringUtils.isNotBlank(infoUrl)) {	1!
152	URI uri = URI.create(infoUrl);	1✔
153	d = "<a href=\"" + uri + "\">" + d + "</a>";	1✔
154	}
155	buf.append("<td width=\"*\">")	1✔
156	.append(d)	1✔
157	.append("</td>")	1✔
158	.append(System.lineSeparator())	1✔
159	.append("</tr>")	1✔
160	.append(System.lineSeparator());	1✔
161	writer.write(buf.toString());	1✔
162	violationCount++;	1✔
163	}	1✔
164	}	1✔
165
166	private String renderFileNameEscaped(FileId fileId, int beginLine) {
167	return maybeWrap(escape(determineFileName(fileId)),	1✔
168	linePrefix == null \|\| beginLine < 0 ? "" : linePrefix + beginLine);	1✔
169	}
170
171	private String renderRuleNameEscaped(Rule rule) {
172	String name = escape(rule.getName());	1✔
173	String infoUrl = rule.getExternalInfoUrl();	1✔
174	if (StringUtils.isNotBlank(infoUrl)) {	1!
175	return "<a href=\"" + infoUrl + "\">" + name + "</a>";	1✔
176	}
UNCOV 177	return name;	×
178	}
179
180	private void glomProcessingErrors(PrintWriter writer, List<Report.ProcessingError> errors) throws IOException {
181
182	if (errors.isEmpty()) {	1✔
183	return;	1✔
184	}
185
186	writer.write("<hr/>");	1✔
187	writer.write("<center><h3>Processing errors</h3></center>");	1✔
188	writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");	1✔
189	writer.println("<th>File</th><th>Problem</th></tr>");	1✔
190
191	StringBuilder buf = new StringBuilder(500);	1✔
192	boolean colorize = true;	1✔
193	for (Report.ProcessingError pe : errors) {	1✔
194	buf.setLength(0);	1✔
195	buf.append("<tr");	1✔
196	if (colorize) {	1!
197	buf.append(" bgcolor=\"lightgrey\"");	1✔
198	}
199	colorize = !colorize;	1!
200	buf.append("> ").append(System.lineSeparator());	1✔
201	buf.append("<td>").append(renderFileNameEscaped(pe.getFileId(), -1)).append("</td>").append(System.lineSeparator());	1✔
202	buf.append("<td><pre>").append(escape(pe.getDetail())).append("</pre></td>").append(System.lineSeparator());	1✔
203	buf.append("</tr>").append(System.lineSeparator());	1✔
204	writer.write(buf.toString());	1✔
205	}	1✔
206	writer.write("</table>");	1✔
207	}	1✔
208
209	private void glomSuppressions(PrintWriter writer, List<Report.SuppressedViolation> suppressed) throws IOException {
210	if (suppressed.isEmpty()) {	1✔
211	return;	1✔
212	}
213
214	writer.write("<hr/>");	1✔
215	writer.write("<center><h3>Suppressed warnings</h3></center>");	1✔
216	writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");	1✔
217	writer.println("<th>File</th><th>Line</th><th>Rule</th><th>NOPMD or Annotation</th><th>Reason</th></tr>");	1✔
218
219	StringBuilder buf = new StringBuilder(500);	1✔
220	boolean colorize = true;	1✔
221	for (Report.SuppressedViolation sv : suppressed) {	1✔
222	buf.setLength(0);	1✔
223	buf.append("<tr");	1✔
224	if (colorize) {	1!
225	buf.append(" bgcolor=\"lightgrey\"");	1✔
226	}
227	colorize = !colorize;	1!
228	buf.append("> ").append(System.lineSeparator());	1✔
229	RuleViolation rv = sv.getRuleViolation();	1✔
230	String userMessage = Optional.ofNullable(sv.getUserMessage()).orElse("");	1✔
231	buf.append("<td align=\"left\">").append(renderFileNameEscaped(rv.getFileId(), rv.getBeginLine())).append("</td>").append(System.lineSeparator());	1✔
232	buf.append("<td align=\"center\">").append(rv.getBeginLine()).append("</td>").append(System.lineSeparator());	1✔
233	buf.append("<td align=\"center\">").append(renderRuleNameEscaped(rv.getRule())).append("</td>").append(System.lineSeparator());	1✔
234	buf.append("<td align=\"center\">").append(escape(sv.getSuppressor().getId())).append("</td>").append(System.lineSeparator());	1✔
235	buf.append("<td align=\"center\">").append(escape(userMessage)).append("</td>").append(System.lineSeparator());	1✔
236	buf.append("</tr>").append(System.lineSeparator());	1✔
237	writer.write(buf.toString());	1✔
238	}	1✔
239	writer.write("</table>");	1✔
240	}	1✔
241
242	private void glomConfigurationErrors(final PrintWriter writer, final List<ConfigurationError> configErrors) throws IOException {
243	if (configErrors.isEmpty()) {	1✔
244	return;	1✔
245	}
246
247	writer.write("<hr/>");	1✔
248	writer.write("<center><h3>Configuration errors</h3></center>");	1✔
249	writer.println("<table align=\"center\" cellspacing=\"0\" cellpadding=\"3\"><tr>");	1✔
250	writer.println("<th>Rule</th><th>Problem</th></tr>");	1✔
251
252	StringBuilder buf = new StringBuilder(500);	1✔
253	boolean colorize = true;	1✔
254	for (Report.ConfigurationError ce : configErrors) {	1✔
255	buf.setLength(0);	1✔
256	buf.append("<tr");	1✔
257	if (colorize) {	1!
258	buf.append(" bgcolor=\"lightgrey\"");	1✔
259	}
260	colorize = !colorize;	1!
261	buf.append("> ").append(System.lineSeparator());	1✔
262	buf.append("<td>").append(renderRuleNameEscaped(ce.rule())).append("</td>").append(System.lineSeparator());	1✔
263	buf.append("<td>").append(escape(ce.issue())).append("</td>").append(System.lineSeparator());	1✔
264	buf.append("</tr>").append(System.lineSeparator());	1✔
265	writer.write(buf.toString());	1✔
266	}	1✔
267	writer.write("</table>");	1✔
268	}	1✔
269
270	private String maybeWrap(String filenameEscaped, String line) {
271	if (StringUtils.isBlank(linkPrefix)) {	1✔
272	return filenameEscaped;	1✔
273	}
274	String newFileName = filenameEscaped.replace('\\', '/');	1✔
275
276	if (replaceHtmlExtension) {	1✔
277	int index = filenameEscaped.lastIndexOf('.');	1✔
278	if (index >= 0) {	1!
279	newFileName = filenameEscaped.substring(0, index);	×
280	}
281	}
282
283	return "<a href=\"" + linkPrefix + newFileName + (replaceHtmlExtension ? ".html#" : "#") + line + "\">"	1✔
284	+ newFileName + "</a>";
285	}
286	}

pmd / pmd / 440

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous