You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by lo...@apache.org on 2022/03/30 11:17:00 UTC
svn commit: r1899384 [11/14] - /myfaces/site/publish/tobago/
Modified: myfaces/site/publish/tobago/pmd.html
URL: http://svn.apache.org/viewvc/myfaces/site/publish/tobago/pmd.html?rev=1899384&r1=1899383&r2=1899384&view=diff
==============================================================================
--- myfaces/site/publish/tobago/pmd.html (original)
+++ myfaces/site/publish/tobago/pmd.html Wed Mar 30 11:17:00 2022
@@ -1,13464 +1,23 @@
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
- <title>Tobago – PMD Results</title>
+ <title>Tobago – Apache Tobago</title>
<link rel="stylesheet" type="text/css" media="all" href="./css/maven-base.css"/>
<link rel="stylesheet" type="text/css" media="all" href="./css/maven-theme.css"/>
<link rel="stylesheet" type="text/css" media="print" href="./css/print.css"/>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
+ <meta http-equiv="refresh" content="5; URL=https://myfaces.apache.org/#tobago">
</head>
<body>
-<div id="header">
- <div id="top_menu">
- <div id="top_menu_container">
-
-
- <div class="date">
- Last Published: 03 Nov 2020
- </div>
- <ul>
- <li>
- <a href="http://www.apache.org/">Apache</a>
- </li>
- <li>
- <a href="../">MyFaces</a>
- </li>
- <li>
- <a href="./">Tobago</a>
- </li>
- <li>
- <a href="mail-lists.html">Mailing Lists</a>
- </li>
- </ul>
- <div class="clear"></div>
- </div> <!-- close top_menu_container -->
- </div> <!-- close top_menu -->
- <div id="header_bottom">
- <div id="logo_banner_container">
- <div id="logo">
- <a href="./">
-
- <img src="images/tobagoLogo.png" alt=""/>
-
- </a>
- </div>
- <div id="banner">
- <a href="../">
-
- <img src="img/banners/MyFaces_logo.jpg" alt=""/>
-
- </a>
- </div>
- <div class="clear"></div>
- </div> <!-- close logo_banner_container -->
- </div> <!-- close header_bottom -->
-</div> <!-- close header -->
-<div id="main_container">
- <div id="left_column">
- <div class="left_menu_top">
- <div class="left_menu_bottom">
- <h5>Tobago</h5>
- <div class="content">
- <ul class="left_menu">
-
- <li class="none">
- <a href="index.html">Introduction</a>
- </li>
-
- <li class="none">
- <a href="http://tobago-vm.apache.org/download.html" class="externalLink">Download<span/></a>
- </li>
-
- <li class="none">
- <a href="http://tobago-vm.apache.org/" class="externalLink">Demo & Docs<span/></a>
- </li>
-
- <li class="none">
- <a href="guide.html">Guide to Tobago</a>
- </li>
- </ul>
- </div><!-- close content -->
- </div><!-- close left_menu_bottom -->
- </div><!-- close left_menu_top -->
- <div class="left_menu_top">
- <div class="left_menu_bottom">
- <h5>Project Documentation</h5>
- <div class="content">
- <ul class="left_menu">
-
-
- <li class="collapsed">
- <a href="project-info.html">Project Information</a>
- </li>
-
-
- <li class="expanded">
- <a href="project-reports.html">Project Reports</a>
- <ul>
-
- <li class="none">
- <a href="cpd.html">CPD</a>
- </li>
-
- <li class="none">
- <a href="pmd.html" class="selected">PMD</a>
- </li>
-
- <li class="none">
- <a href="xref/index.html">Source Xref</a>
- </li>
-
- <li class="none">
- <a href="xref-test/index.html">Test Source Xref</a>
- </li>
-
- <li class="none">
- <a href="checkstyle-aggregate.html">Checkstyle</a>
- </li>
-
- <li class="none">
- <a href="apidocs/index.html">JavaDocs</a>
- </li>
-
- <li class="none">
- <a href="testapidocs/index.html">Test JavaDocs</a>
- </li>
-
- <li class="none">
- <a href="surefire-report.html">Surefire Report</a>
- </li>
-
- <li class="none">
- <a href="changelog.html">Change Log</a>
- </li>
-
- <li class="none">
- <a href="file-activity.html">File Activity</a>
- </li>
-
- <li class="none">
- <a href="dev-activity.html">Developer Activity</a>
- </li>
-
- <li class="none">
- <a href="rat-report.html">Rat Report</a>
- </li>
-
- <li class="none">
- <a href="dependency-analysis.html">Dependency Analysis</a>
- </li>
- </ul>
- </li>
- </ul>
- </div><!-- close content -->
- </div><!-- close left_menu_bottom -->
- </div><!-- close left_menu_top -->
- <div class="left_menu_top">
- <div class="left_menu_bottom">
- <h5>Foundation</h5>
- <div class="content">
- <ul class="left_menu">
-
- <li class="none">
- <a href="http://apache.org/" class="externalLink">ASF<span/></a>
- </li>
-
- <li class="none">
- <a href="http://www.apache.org/foundation/sponsorship.html" class="externalLink">Sponsorship<span/></a>
- </li>
-
- <li class="none">
- <a href="http://www.apache.org/foundation/thanks.html" class="externalLink">Thanks<span/></a>
- </li>
-
- <li class="none">
- <a href="http://www.apache.org/security/" class="externalLink">Security<span/></a>
- </li>
-
- <li class="none">
- <a href="http://www.apache.org/licenses/" class="externalLink">License<span/></a>
- </li>
- </ul>
- </div><!-- close content -->
- </div><!-- close left_menu_bottom -->
- </div><!-- close left_menu_top -->
- <div id="powered_by">
- <a href="http://maven.apache.org/" title="Built by Maven" class="poweredBy">
- <img alt="Built by Maven" src="./img/banners/maven-feather.png"></img>
- </a>
- </div> <!-- close powered_by -->
- <div id="powered_by">
- <a href="http://www.apache.org/events/current-event.html" title="ApacheCon">
- <img alt="ApacheCon" src="http://www.apache.org/events/current-event-125x125.png"/>
- </a>
- </div>
- <div class="clear"></div>
- </div>
- <div id="body_column">
- <div>
- <div class="section">
- <h2><a name="PMD_Results"></a>PMD Results</h2>
- <p>The following document contains the results of <a class="externalLink"
- href="http://pmd.sourceforge.net/">PMD</a> 5.6.1.</p></div>
- <div class="section">
- <h2><a name="Files"></a>Files</h2>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocomponentAttributes.java"></a>Tobago Core -
- org/apache/myfaces/tobago/component/Attributes.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/component/Attributes.html#L447">447</a>–<a
- href="./xref/org/apache/myfaces/tobago/component/Attributes.html#L451">451</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoconfigTobagoConfig.java"></a>Tobago Core -
- org/apache/myfaces/tobago/config/TobagoConfig.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>getInstance method always creates a new object and hence does not comply to Singleton Design Pattern
- behaviour. Please review
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/config/TobagoConfig.html#L49">49</a>–<a
- href="./xref/org/apache/myfaces/tobago/config/TobagoConfig.html#L62">62</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/config/TobagoConfig.html#L56">56</a>–<a
- href="./xref/org/apache/myfaces/tobago/config/TobagoConfig.html#L61">61</a></td>
- </tr>
- <tr class="b">
- <td>Class contains multiple getInstance methods. Please review.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/config/TobagoConfig.html#L68">68</a>–<a
- href="./xref/org/apache/myfaces/tobago/config/TobagoConfig.html#L70">70</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextMarkup.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/Markup.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>The class 'Markup' has a Standard Cyclomatic Complexity of 4 (Highest = 10).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L59">59</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L497">497</a></td>
- </tr>
- <tr class="a">
- <td>This class has too many methods, consider refactoring it.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L59">59</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L497">497</a></td>
- </tr>
- <tr class="b">
- <td>The class 'Markup' has a Cyclomatic Complexity of 5 (Highest = 11).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L59">59</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L497">497</a></td>
- </tr>
- <tr class="a">
- <td>The class 'Markup' has a Modified Cyclomatic Complexity of 4 (Highest = 10).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L59">59</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L497">497</a></td>
- </tr>
- <tr class="b">
- <td>Consider using varargs for methods or constructors which take an array the last parameter.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L204">204</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L205">205</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L211">211</a></td>
- </tr>
- <tr class="a">
- <td>Consider using varargs for methods or constructors which take an array the last parameter.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L214">214</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'string' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L221">221</a></td>
- </tr>
- <tr class="a">
- <td>Local variable 'stringBuilder' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L222">222</a></td>
- </tr>
- <tr class="b">
- <td>Consider using varargs for methods or constructors which take an array the last parameter.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L242">242</a></td>
- </tr>
- <tr class="a">
- <td>Prefer StringBuffer over += for concatenating strings</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L250">250</a></td>
- </tr>
- <tr class="b">
- <td>This call to Collection.toArray() may be optimizable</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L286">286</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L310">310</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L311">311</a></td>
- </tr>
- <tr class="a">
- <td>Use equals() to compare object references.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L334">334</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L337">337</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L351">351</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L381">381</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L395">395</a></td>
- </tr>
- <tr class="b">
- <td>The method 'contains' has a Standard Cyclomatic Complexity of 10.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L447">447</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L477">477</a></td>
- </tr>
- <tr class="a">
- <td>The method 'contains' has a Cyclomatic Complexity of 11.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L447">447</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L477">477</a></td>
- </tr>
- <tr class="b">
- <td>The method 'contains' has a Modified Cyclomatic Complexity of 10.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L447">447</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L477">477</a></td>
- </tr>
- <tr class="a">
- <td>Use equals() to compare object references.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L448">448</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L454">454</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L476">476</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L455">455</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L464">464</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/Markup.html#L466">466</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/Markup.html#L475">475</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextMarkupConfig.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/MarkupConfig.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Private field 'markups' could be made final; it is only initialized in the declaration or
- constructor.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/MarkupConfig.html#L30">30</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextThemeImpl.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/ThemeImpl.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Avoid using redundant field initializer for 'locked'</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L51">51</a></td>
- </tr>
- <tr class="a">
- <td>Parameter 'add' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L58">58</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'base' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L58">58</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L64">64</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L65">65</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L66">66</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'list' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L183">183</a></td>
- </tr>
- <tr class="a">
- <td>This call to Collection.toArray() may be optimizable</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L186">186</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'list' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L189">189</a></td>
- </tr>
- <tr class="a">
- <td>This call to Collection.toArray() may be optimizable</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L192">192</a></td>
- </tr>
- <tr class="b">
- <td>StringBuffer constructor is initialized with size 16, but has at least 121 characters appended.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L224">224</a></td>
- </tr>
- <tr class="a">
- <td>StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L225">225</a></td>
- </tr>
- <tr class="b">
- <td>StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L226">226</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L228">228</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L234">234</a></td>
- </tr>
- <tr class="b">
- <td>Avoid appending characters as strings in StringBuffer.append.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L229">229</a></td>
- </tr>
- <tr class="a">
- <td>Avoid appending characters as strings in StringBuffer.append.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L231">231</a></td>
- </tr>
- <tr class="b">
- <td>StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L235">235</a></td>
- </tr>
- <tr class="a">
- <td>StringBuffer (or StringBuilder).append is called consecutively without reusing the target variable.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L236">236</a></td>
- </tr>
- <tr class="b">
- <td>Avoid appending characters as strings in StringBuffer.append.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L239">239</a></td>
- </tr>
- <tr class="a">
- <td>Avoid appending characters as strings in StringBuffer.append.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L244">244</a></td>
- </tr>
- <tr class="b">
- <td>Avoid appending characters as strings in StringBuffer.append.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L249">249</a></td>
- </tr>
- <tr class="a">
- <td>Avoid appending characters as strings in StringBuffer.append.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeImpl.html#L254">254</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextThemeResource.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/ThemeResource.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Parameter 'priority' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResource.html#L45">45</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResource.html#L65">65</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextThemeResources.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/ThemeResources.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L51">51</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L52">52</a></td>
- </tr>
- <tr class="a">
- <td>Local variable 'includeScript' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L95">95</a></td>
- </tr>
- <tr class="b">
- <td>Local variable 'includeScript' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L100">100</a></td>
- </tr>
- <tr class="a">
- <td>Local variable 'includeStyle' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L104">104</a></td>
- </tr>
- <tr class="b">
- <td>Local variable 'includeStyle' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L109">109</a></td>
- </tr>
- <tr class="a">
- <td>Local variable 'resource' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L128">128</a></td>
- </tr>
- <tr class="b">
- <td>Local variable 'resource' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L142">142</a></td>
- </tr>
- <tr class="a">
- <td>Local variable 'resource' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L164">164</a></td>
- </tr>
- <tr class="b">
- <td>Local variable 'resource' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeResources.html#L178">178</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextThemeScript.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/ThemeScript.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Parameter 'type' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeScript.html#L35">35</a></td>
- </tr>
- <tr class="a">
- <td>Parameter 'o' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeScript.html#L40">40</a></td>
- </tr>
- <tr class="b">
- <td>Local variable 'that' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/ThemeScript.html#L50">50</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextTobagoContext.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/TobagoContext.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/TobagoContext.html#L62">62</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/TobagoContext.html#L63">63</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/TobagoContext.html#L73">73</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/TobagoContext.html#L74">74</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextTobagoExceptionHandler.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/TobagoExceptionHandler.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Private field 'wrapped' could be made final; it is only initialized in the declaration or
- constructor.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/TobagoExceptionHandler.html#L48">48</a></td>
- </tr>
- <tr class="b">
- <td>Avoid instantiating new objects inside loops</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/TobagoExceptionHandler.html#L76">76</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/TobagoExceptionHandler.html#L78">78</a></td>
- </tr>
- <tr class="a">
- <td>Avoid instantiating new objects inside loops</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/TobagoExceptionHandler.html#L91">91</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/TobagoExceptionHandler.html#L93">93</a></td>
- </tr>
- <tr class="b">
- <td>Avoid instantiating new objects inside loops</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/TobagoExceptionHandler.html#L97">97</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/TobagoExceptionHandler.html#L99">99</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextTransientStateHolder.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/TransientStateHolder.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/TransientStateHolder.html#L33">33</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/TransientStateHolder.html#L34">34</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagocontextUserAgent.java"></a>Tobago Core -
- org/apache/myfaces/tobago/context/UserAgent.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>The class 'UserAgent' has a Modified Cyclomatic Complexity of 3 (Highest = 19).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L34">34</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L392">392</a></td>
- </tr>
- <tr class="a">
- <td>The class 'UserAgent' has a Standard Cyclomatic Complexity of 3 (Highest = 19).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L34">34</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L392">392</a></td>
- </tr>
- <tr class="b">
- <td>The class 'UserAgent' has a Cyclomatic Complexity of 3 (Highest = 20).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L34">34</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L392">392</a></td>
- </tr>
- <tr class="a">
- <td>The String literal "msie" appears 12 times in this file; the first occurrence is on line
- 40
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L40">40</a></td>
- </tr>
- <tr class="b">
- <td>The String literal "opera" appears 4 times in this file; the first occurrence is on line
- 105
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L105">105</a></td>
- </tr>
- <tr class="a">
- <td>The String literal "mozilla" appears 4 times in this file; the first occurrence is on line
- 129
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L129">129</a></td>
- </tr>
- <tr class="b">
- <td>The String literal "gecko" appears 5 times in this file; the first occurrence is on line
- 152
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L152">152</a></td>
- </tr>
- <tr class="a">
- <td>The method 'getInstance' has a Standard Cyclomatic Complexity of 19.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L281">281</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L336">336</a></td>
- </tr>
- <tr class="b">
- <td>The method 'getInstance' has a Modified Cyclomatic Complexity of 19.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L281">281</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L336">336</a></td>
- </tr>
- <tr class="a">
- <td>The method 'getInstance' has a Cyclomatic Complexity of 20.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L281">281</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L336">336</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L352">352</a>–<a
- href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L354">354</a></td>
- </tr>
- <tr class="a">
- <td>Consider using varargs for methods or constructors which take an array the last parameter.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L366">366</a></td>
- </tr>
- <tr class="b">
- <td>Consider using varargs for methods or constructors which take an array the last parameter.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/context/UserAgent.html#L384">384</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoconvertDateTimeConverter.java"></a>Tobago Core -
- org/apache/myfaces/tobago/convert/DateTimeConverter.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>The class 'DateTimeConverter' has a Cyclomatic Complexity of 8 (Highest = 20).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L48">48</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L176">176</a></td>
- </tr>
- <tr class="a">
- <td>The class 'DateTimeConverter' has a Modified Cyclomatic Complexity of 6 (Highest = 16).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L48">48</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L176">176</a></td>
- </tr>
- <tr class="b">
- <td>The class 'DateTimeConverter' has a Standard Cyclomatic Complexity of 6 (Highest = 16).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L48">48</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L176">176</a></td>
- </tr>
- <tr class="a">
- <td>The method 'getAsObject' has a Cyclomatic Complexity of 20.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L66">66</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L122">122</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'facesContext' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L66">66</a></td>
- </tr>
- <tr class="a">
- <td>The method 'getAsObject' has a Standard Cyclomatic Complexity of 16.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L66">66</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L122">122</a></td>
- </tr>
- <tr class="b">
- <td>The method 'getAsObject' has a Modified Cyclomatic Complexity of 16.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L66">66</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L122">122</a></td>
- </tr>
- <tr class="a">
- <td>Parameter 'string' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L66">66</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'component' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L66">66</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L81">81</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L89">89</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L83">83</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L89">89</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L85">85</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L89">89</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L92">92</a></td>
- </tr>
- <tr class="a">
- <td>Local variable 'sdf' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L92">92</a></td>
- </tr>
- <tr class="b">
- <td>When instantiating a SimpleDateFormat object, specify a Locale</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L92">92</a></td>
- </tr>
- <tr class="a">
- <td>Parameter 'facesContext' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L125">125</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'object' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L125">125</a></td>
- </tr>
- <tr class="a">
- <td>The method 'getAsString' has a Cyclomatic Complexity of 12.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L125">125</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L150">150</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'component' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L125">125</a></td>
- </tr>
- <tr class="a">
- <td>Local variable 'calendar' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L133">133</a></td>
- </tr>
- <tr class="b">
- <td>Local variable 'sdf' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L137">137</a></td>
- </tr>
- <tr class="a">
- <td>Parameter 'mandatoryChars' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L152">152</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'mandatoryChars' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L160">160</a></td>
- </tr>
- <tr class="a">
- <td>Parameter 'name' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L160">160</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'c' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L160">160</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L169">169</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L174">174</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DateTimeConverter.html#L171">171</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoconvertDurationConverter.java"></a>Tobago Core -
- org/apache/myfaces/tobago/convert/DurationConverter.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>The class 'DurationConverter' has a Standard Cyclomatic Complexity of 8 (Highest = 10).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L57">57</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L175">175</a></td>
- </tr>
- <tr class="a">
- <td>The class 'DurationConverter' has a Cyclomatic Complexity of 8 (Highest = 10).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L57">57</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L175">175</a></td>
- </tr>
- <tr class="b">
- <td>Unnecessary wrapper object creation</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L84">84</a></td>
- </tr>
- <tr class="a">
- <td>Avoid declaring a variable if it is unreferenced before a possible exit point.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L106">106</a></td>
- </tr>
- <tr class="b">
- <td>The method 'getUnitFactor' has a Standard Cyclomatic Complexity of 10.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L143">143</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L173">173</a></td>
- </tr>
- <tr class="a">
- <td>The method 'getUnitFactor' has a Cyclomatic Complexity of 10.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L143">143</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L173">173</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L146">146</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L155">155</a></td>
- </tr>
- <tr class="a">
- <td>The default label should be the last label in a switch statement</td>
- <td><a href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L156">156</a>–<a
- href="./xref/org/apache/myfaces/tobago/convert/DurationConverter.html#L172">172</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventFacesEventWrapper.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/FacesEventWrapper.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Private field 'wrappedFacesEvent' could be made final; it is only initialized in the declaration or
- constructor.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/FacesEventWrapper.html#L31">31</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventMethodExpressionStateChangeListener.java"></a>Tobago
- Core - org/apache/myfaces/tobago/event/MethodExpressionStateChangeListener.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Avoid using redundant field initializer for 'isTransient'</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionStateChangeListener.html#L33">33</a>
- </td>
- </tr>
- <tr class="b">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionStateChangeListener.html#L35">35</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/MethodExpressionStateChangeListener.html#L36">36</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventMethodExpressionTabChangeListener.java"></a>Tobago Core
- - org/apache/myfaces/tobago/event/MethodExpressionTabChangeListener.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Avoid using redundant field initializer for 'isTransient'</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTabChangeListener.html#L33">33</a>
- </td>
- </tr>
- <tr class="a">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTabChangeListener.html#L35">35</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTabChangeListener.html#L36">36</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventMethodExpressionTreeExpansionListener.java"></a>Tobago
- Core - org/apache/myfaces/tobago/event/MethodExpressionTreeExpansionListener.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Avoid using redundant field initializer for 'isTransient'</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTreeExpansionListener.html#L34">34</a>
- </td>
- </tr>
- <tr class="b">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTreeExpansionListener.html#L36">36</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTreeExpansionListener.html#L37">37</a>
- </td>
- </tr>
- <tr class="a">
- <td>Avoid unused private methods such as 'elContext()'.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTreeExpansionListener.html#L48">48</a>
- </td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventMethodExpressionTreeSelectedListener.java"></a>Tobago
- Core - org/apache/myfaces/tobago/event/MethodExpressionTreeSelectedListener.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Avoid using redundant field initializer for 'isTransient'</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTreeSelectedListener.html#L33">33</a>
- </td>
- </tr>
- <tr class="b">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTreeSelectedListener.html#L35">35</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTreeSelectedListener.html#L36">36</a>
- </td>
- </tr>
- <tr class="a">
- <td>Avoid unused private methods such as 'elContext()'.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/MethodExpressionTreeSelectedListener.html#L47">47</a>
- </td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventPageActionEvent.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/PageActionEvent.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Private field 'action' could be made final; it is only initialized in the declaration or
- constructor.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/PageActionEvent.html#L31">31</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventResetInputActionListener.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/ResetInputActionListener.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/ResetInputActionListener.html#L34">34</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/ResetInputActionListener.html#L35">35</a></td>
- </tr>
- <tr class="b">
- <td>Consider using varargs for methods or constructors which take an array the last parameter.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/ResetInputActionListener.html#L37">37</a></td>
- </tr>
- <tr class="a">
- <td>This call to Collection.toArray() may be optimizable</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/ResetInputActionListener.html#L42">42</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventSheetAction.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/SheetAction.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/SheetAction.html#L66">66</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/SheetAction.html#L67">67</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventSortActionEvent.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/SortActionEvent.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Private field 'column' could be made final; it is only initialized in the declaration or
- constructor.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/SortActionEvent.html#L31">31</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventTabChangeEvent.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/TabChangeEvent.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Overridable method 'setOldState' called during object construction</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/TabChangeEvent.html#L45">45</a></td>
- </tr>
- <tr class="b">
- <td>Overridable method 'setNewState' called during object construction</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/TabChangeEvent.html#L46">46</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventTobagoActionSource.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/TobagoActionSource.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>The String literal "JSF 1.1 style is no longer supported!" appears 4 times in this file; the
- first occurrence is on line 39
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/TobagoActionSource.html#L39">39</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventTreeExpansionEvent.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/TreeExpansionEvent.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Private field 'oldExpanded' could be made final; it is only initialized in the declaration or
- constructor.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/TreeExpansionEvent.html#L30">30</a></td>
- </tr>
- <tr class="a">
- <td>Private field 'newExpanded' could be made final; it is only initialized in the declaration or
- constructor.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/TreeExpansionEvent.html#L31">31</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventTreeSelectedEvent.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/TreeSelectedEvent.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Private field 'oldSelected' could be made final; it is only initialized in the declaration or
- constructor.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/TreeSelectedEvent.html#L30">30</a></td>
- </tr>
- <tr class="b">
- <td>Private field 'newSelected' could be made final; it is only initialized in the declaration or
- constructor.
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/TreeSelectedEvent.html#L31">31</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventValueExpressionResetInputActionListener.java"></a>Tobago
- Core - org/apache/myfaces/tobago/event/ValueExpressionResetInputActionListener.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Document empty constructor</td>
- <td><a
- href="./xref/org/apache/myfaces/tobago/event/ValueExpressionResetInputActionListener.html#L45">45</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/ValueExpressionResetInputActionListener.html#L46">46</a>
- </td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventValueExpressionSheetStateChangeListener.java"></a>Tobago
- Core - org/apache/myfaces/tobago/event/ValueExpressionSheetStateChangeListener.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Document empty constructor</td>
- <td><a
- href="./xref/org/apache/myfaces/tobago/event/ValueExpressionSheetStateChangeListener.html#L38">38</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/ValueExpressionSheetStateChangeListener.html#L39">39</a>
- </td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a
- href="./xref/org/apache/myfaces/tobago/event/ValueExpressionSheetStateChangeListener.html#L49">49</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/ValueExpressionSheetStateChangeListener.html#L53">53</a>
- </td>
- </tr>
- <tr class="b">
- <td>No need to check for null before an instanceof</td>
- <td><a
- href="./xref/org/apache/myfaces/tobago/event/ValueExpressionSheetStateChangeListener.html#L86">86</a>
- </td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoeventValueExpressionTabChangeListener.java"></a>Tobago Core -
- org/apache/myfaces/tobago/event/ValueExpressionTabChangeListener.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/ValueExpressionTabChangeListener.html#L38">38</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/ValueExpressionTabChangeListener.html#L39">39</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/ValueExpressionTabChangeListener.html#L49">49</a>–<a
- href="./xref/org/apache/myfaces/tobago/event/ValueExpressionTabChangeListener.html#L53">53</a></td>
- </tr>
- <tr class="b">
- <td>No need to check for null before an instanceof</td>
- <td><a href="./xref/org/apache/myfaces/tobago/event/ValueExpressionTabChangeListener.html#L86">86</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoexceptionTobagoConfigurationException.java"></a>Tobago Core -
- org/apache/myfaces/tobago/exception/TobagoConfigurationException.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>Parameter 'message' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/exception/TobagoConfigurationException.html#L27">27</a></td>
- </tr>
- <tr class="a">
- <td>Parameter 'message' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/exception/TobagoConfigurationException.html#L31">31</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'cause' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/exception/TobagoConfigurationException.html#L31">31</a></td>
- </tr>
- <tr class="a">
- <td>Parameter 'cause' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/exception/TobagoConfigurationException.html#L35">35</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagoexceptionTobagoException.java"></a>Tobago Core -
- org/apache/myfaces/tobago/exception/TobagoException.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Parameter 'message' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/exception/TobagoException.html#L27">27</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'message' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/exception/TobagoException.html#L31">31</a></td>
- </tr>
- <tr class="a">
- <td>Parameter 'cause' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/exception/TobagoException.html#L31">31</a></td>
- </tr>
- <tr class="b">
- <td>Parameter 'cause' is not assigned and could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/exception/TobagoException.html#L35">35</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsAttributeHandler.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/AttributeHandler.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>A high number of imports can indicate a high degree of coupling within an object.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L20">20</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L433">433</a></td>
- </tr>
- <tr class="a">
- <td>Possible God class (WMC=90, ATFD=82, TCC=0.04678362573099415)</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L20">20</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L433">433</a></td>
- </tr>
- <tr class="b">
- <td>The class 'AttributeHandler' has a Modified Cyclomatic Complexity of 6 (Highest = 50).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L59">59</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L433">433</a></td>
- </tr>
- <tr class="a">
- <td>The class 'AttributeHandler' has a Cyclomatic Complexity of 7 (Highest = 57).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L59">59</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L433">433</a></td>
- </tr>
- <tr class="b">
- <td>The class 'AttributeHandler' has a Standard Cyclomatic Complexity of 6 (Highest = 50).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L59">59</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L433">433</a></td>
- </tr>
- <tr class="a">
- <td>The method 'apply' has a Standard Cyclomatic Complexity of 50.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L77">77</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L276">276</a></td>
- </tr>
- <tr class="b">
- <td>Avoid really long methods.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L77">77</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L276">276</a></td>
- </tr>
- <tr class="a">
- <td>The method 'apply' has a Cyclomatic Complexity of 57.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L77">77</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L276">276</a></td>
- </tr>
- <tr class="b">
- <td>The method 'apply' has a Modified Cyclomatic Complexity of 50.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L77">77</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L276">276</a></td>
- </tr>
- <tr class="a">
- <td>The method apply() has an NCSS line count of 158</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L77">77</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L276">276</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L84">84</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L274">274</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L88">88</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L106">106</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L111">111</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L129">129</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L197">197</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L204">204</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L265">265</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L273">273</a></td>
- </tr>
- <tr class="a">
- <td>Consider using varargs for methods or constructors which take an array the last parameter.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L300">300</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L306">306</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L312">312</a></td>
- </tr>
- <tr class="a">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L365">365</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/AttributeHandler.html#L366">366</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsConvertDateTimeHandler.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>The class 'ConvertDateTimeHandler' has a Cyclomatic Complexity of 8 (Highest = 15).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L42">42</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L123">123</a></td>
- </tr>
- <tr class="b">
- <td>The class 'ConvertDateTimeHandler' has a Standard Cyclomatic Complexity of 8 (Highest = 15).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L42">42</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L123">123</a></td>
- </tr>
- <tr class="a">
- <td>The class 'ConvertDateTimeHandler' has a Modified Cyclomatic Complexity of 8 (Highest = 15).</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L42">42</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L123">123</a></td>
- </tr>
- <tr class="b">
- <td>The method 'apply' has a Modified Cyclomatic Complexity of 15.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L62">62</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L122">122</a></td>
- </tr>
- <tr class="a">
- <td>The method 'apply' has a Cyclomatic Complexity of 15.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L62">62</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L122">122</a></td>
- </tr>
- <tr class="b">
- <td>The method 'apply' has a Standard Cyclomatic Complexity of 15.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L62">62</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L122">122</a></td>
- </tr>
- <tr class="a">
- <td>The method apply() has an NPath complexity of 654</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L62">62</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L122">122</a></td>
- </tr>
- <tr class="b">
- <td>Local variable 'dtConverter' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L75">75</a></td>
- </tr>
- <tr class="a">
- <td>Deeply nested if..then statements are hard to read</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L79">79</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/ConvertDateTimeHandler.html#L83">83</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsConverterHandler.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/ConverterHandler.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Avoid declaring a variable if it is unreferenced before a possible exit point.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConverterHandler.html#L54">54</a></td>
- </tr>
- <tr class="b">
- <td>New exception is thrown in catch block, original stack trace may be lost</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConverterHandler.html#L66">66</a></td>
- </tr>
- <tr class="a">
- <td>Deeply nested if..then statements are hard to read</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/ConverterHandler.html#L68">68</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/ConverterHandler.html#L70">70</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsEventHandler.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/EventHandler.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Avoid declaring a variable if it is unreferenced before a possible exit point.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/EventHandler.html#L137">137</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/EventHandler.html#L138">138</a></td>
- </tr>
- <tr class="b">
- <td>Avoid unused local variables such as 'faceletContext'.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/EventHandler.html#L137">137</a></td>
- </tr>
- <tr class="a">
- <td>Document empty constructor</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/EventHandler.html#L198">198</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/EventHandler.html#L199">199</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsFixCharacterEncodingFilter.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/FixCharacterEncodingFilter.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Document empty method body</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/FixCharacterEncodingFilter.html#L59">59</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/FixCharacterEncodingFilter.html#L60">60</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsGridLayoutConstraintHandler.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/GridLayoutConstraintHandler.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/GridLayoutConstraintHandler.html#L47">47</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/GridLayoutConstraintHandler.html#L51">51</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsHackResourceExtentionFilter.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/HackResourceExtentionFilter.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>The String literal ".js" appears 4 times in this file; the first occurrence is on line 64
- </td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/HackResourceExtentionFilter.html#L64">64</a></td>
- </tr>
- <tr class="a">
- <td>Local variable 'wrapper' could be declared final</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/HackResourceExtentionFilter.html#L65">65</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/HackResourceExtentionFilter.html#L81">81</a></td>
- </tr>
- <tr class="b">
- <td>Document empty method body</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/HackResourceExtentionFilter.html#L92">92</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/HackResourceExtentionFilter.html#L93">93</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsResource.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/Resource.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>These nested if statements could be combined</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/Resource.html#L73">73</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/Resource.html#L75">75</a></td>
- </tr>
- <tr class="a">
- <td>Document empty method body</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/Resource.html#L111">111</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/Resource.html#L112">112</a></td>
- </tr>
- <tr class="b">
- <td>Avoid autogenerated methods to access private fields and methods of inner / outer classes</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/Resource.html#L116">116</a></td>
- </tr>
- <tr class="a">
- <td>Avoid autogenerated methods to access private fields and methods of inner / outer classes</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/Resource.html#L117">117</a></td>
- </tr>
- <tr class="b">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/Resource.html#L133">133</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsSheetStateChangeSourceRule.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/SheetStateChangeSourceRule.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>These nested if statements could be combined</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/SheetStateChangeSourceRule.html#L40">40</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/SheetStateChangeSourceRule.html#L42">42</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsSortActionSourceRule.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/SortActionSourceRule.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>These nested if statements could be combined</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/SortActionSourceRule.html#L41">41</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/SortActionSourceRule.html#L43">43</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsSuggestMethodRule.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/SuggestMethodRule.java</h3>
- <table border="0" class="bodyTable">
- <tr class="a">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="b">
- <td>These nested if statements could be combined</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/SuggestMethodRule.html#L42">42</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/SuggestMethodRule.html#L44">44</a></td>
- </tr>
- <tr class="a">
- <td>These nested if statements could be combined</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/SuggestMethodRule.html#L47">47</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/SuggestMethodRule.html#L51">51</a></td>
- </tr>
- <tr class="b">
- <td>These nested if statements could be combined</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/SuggestMethodRule.html#L48">48</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/SuggestMethodRule.html#L50">50</a></td>
- </tr>
- <tr class="a">
- <td>Deeply nested if..then statements are hard to read</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/SuggestMethodRule.html#L48">48</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/SuggestMethodRule.html#L50">50</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsTabChangeListenerHandler.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/TabChangeListenerHandler.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>Avoid declaring a variable if it is unreferenced before a possible exit point.</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/TabChangeListenerHandler.html#L66">66</a></td>
- </tr>
- <tr class="b">
- <td>Deeply nested if..then statements are hard to read</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/TabChangeListenerHandler.html#L79">79</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/TabChangeListenerHandler.html#L81">81</a></td>
- </tr>
- <tr class="a">
- <td>Avoid if (x != y) ..; else ..;</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/TabChangeListenerHandler.html#L83">83</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/TabChangeListenerHandler.html#L88">88</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsTabChangeSourceRule.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/TabChangeSourceRule.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>These nested if statements could be combined</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/TabChangeSourceRule.html#L38">38</a>–<a
- href="./xref/org/apache/myfaces/tobago/facelets/TabChangeSourceRule.html#L40">40</a></td>
- </tr>
- </table>
- </div>
- <div class="section">
- <h3><a name="Tobago_Core_-_orgapachemyfacestobagofaceletsTagMethodExpression.java"></a>Tobago Core -
- org/apache/myfaces/tobago/facelets/TagMethodExpression.java</h3>
- <table border="0" class="bodyTable">
- <tr class="b">
- <th>Violation</th>
- <th>Line</th>
- </tr>
- <tr class="a">
- <td>New exception is thrown in catch block, original stack trace may be lost</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/TagMethodExpression.html#L64">64</a></td>
- </tr>
- <tr class="b">
- <td>New exception is thrown in catch block, original stack trace may be lost</td>
- <td><a href="./xref/org/apache/myfaces/tobago/facelets/TagMethodExpression.html#L66">66</a></td>
[... 11798 lines stripped ...]