Tag List Report
The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
Tag Class | Total number of occurrences | Tag strings used by tag class |
---|---|---|
@todo | 0 | @todo |
TODO | 54 | TODO |
Each tag is detailed below:
TODO
Number of occurrences found in the code: 54
org.itracker.selenium.CreateIssueSeleniumIT | Line |
---|---|
--No comment-- | 126 |
org.itracker.selenium.LogoutSeleniumIT | Line |
not use username as password! | 40 |
org.itracker.web.actions.PortalHomeAction | Line |
Action Cleanup | 25 |
since repeating code, set a common Comparator variable to contain the Comparator to use and execute the sort pre issue type only once. | 126 |
set the next value based on the request attribute! | 237 |
org.itracker.web.actions.admin.AdminHomeAction | Line |
performance issue when attachments size needs to be calculated over many issues ! select sum(size) from IssueAttachment | 136 |
org.itracker.web.actions.admin.configuration.EditConfigurationAction | Line |
Action Cleanup | 55 |
org.itracker.web.actions.admin.configuration.EditCustomFieldFormAction | Line |
Action Cleanup | 45 |
org.itracker.web.actions.admin.configuration.EditCustomFieldValueFormAction | Line |
Action Cleanup | 56 |
org.itracker.web.actions.admin.configuration.ImportDataProcessAction | Line |
, removed name attribute, so it's defaulted to the key | 155 |
, removed name attribute, so it's defaulted to the key | 159 |
? | 302 |
org.itracker.web.actions.admin.configuration.RemoveConfigurationItemAction | Line |
Action Cleanup | 43 |
org.itracker.web.actions.admin.project.EditProjectScriptFormAction | Line |
maybe intended to be an 'action' for the EditProjectScriptAction? | 50 |
org.itracker.web.actions.admin.report.EditReportAction | Line |
cleanup the service. | 76 |
org.itracker.web.actions.admin.report.EditReportFormAction | Line |
Action Cleanup | 35 |
org.itracker.web.actions.preferences.EditPreferencesAction | Line |
Action Cleanup | 60 |
the following checks make no sense from my perspective. This check should happen in the ExecuteAlways filter maybe Shall we remove it? | 75 |
should this check happen earlier? | 142 |
org.itracker.web.actions.preferences.EditPreferencesFormAction | Line |
Action Cleanup | 56 |
remove this hack, this should be handled central, there are other instances of this hack | 89 |
org.itracker.web.actions.project.CreateIssueAction | Line |
Action Cleanup | 55 |
this is absolutely complex, unreadable code. why do it, what does it do, can we keep it simple? it seems to set issueCustomField (issueFields), you might be able to refactor this into its own method (hiding in a method ;) ) | 192 |
org.itracker.web.actions.project.CreateIssueFormAction | Line |
Action Cleanup | 53 |
org.itracker.web.actions.project.EditIssueFormAction | Line |
Sort attachments Collections.sort(attachments, IssueAttachment.CREATE_DATE_COMPARATOR); | 125 |
org.itracker.web.actions.project.ListProjectsAction | Line |
add a secure token to feed-URL | 109 |
org.itracker.web.actions.project.MoveIssueAction | Line |
is seems first condition is not necessary | 139 |
return detailed messages on the missing authorization | 140 |
org.itracker.web.actions.user.SelfRegisterAction | Line |
remove this hack, this should be handled central, there are other instances of this hack | 92 |
org.itracker.web.filters.ExecuteAlwaysFilter | Line |
this should be configured per-instance. Request server-name should only be used for exception and logged (configuration not found!) | 184 |
remove deprecated currLocale attribute | 200 |
org.itracker.web.forms.CustomFieldValueForm | Line |
setup request env for validation output | 109 |
org.itracker.web.forms.IssueForm | Line |
should issue, project, user, services be available too? | 196 |
why do we need to updateIssue here, and can not later? issueService.updateIssue(issue, user.getId()); | 586 |
simplify this code, it's not readable, unsave yet. | 742 |
What happens here, validation? | 907 |
org.itracker.web.scheduler.tasks.MailNotification | Line |
never used, commented, task added: int totalMessages = src_folder.getMessageCount(); | 98 |
process message | 120 |
org.itracker.web.servlets.GenericController | Line |
Rewrite Javadocs here This needs documentation. Is it still used? What is it used for? How? It is referenced by AttachementDownloadController, (@deprecated Use org.itracker.web.actions.admin.attachment.DownloadAttachmentAction instead.) ReportChartController, ReportDownloadController | 48 |
org.itracker.web.util.ImportExportTags | Line |
--No comment-- | 43 |
org.itracker.web.util.ImportExportUtilities | Line |
instead to have a string returned, it should directly serialize the export to the response-writer. | 129 |
unmarshal from is | 148 |
marshal to System.out | 156 |
org.itracker.web.util.ImportExportUtilitiesIT | Line |
not stable | 220 |
Not a valid XML.. | 467 |
org.itracker.web.util.ImportHandler | Line |
handle configuration for setting the BASE label | 265 |
Make status Enum | 386 |
org.itracker.web.util.ListIssuesActionUtil | Line |
check for other occurences for this constants and maybe place somewhere else? | 21 |
temporary pagination helper | 232 |
org.itracker.web.util.LoginUtilities | Line |
the order of retrieving locale from request should be: <ol> <li>request-attribute Constants.LOCALE_KEY</li> <li>request-param 'loc'</li> <li>session attribute <code>Constants.LOCALE_KEY</code></li> <li>cookie 'loc'</li> <li>request.getLocale()/request.getLocales()</li> <li>ITrackerResources.DEFAULT_LOCALE</li> </ol> </p> | 149 |
this is a hack, remove when possible | 379 |
cookie could be removed | 393 |
option on project that all users can view own issues | 546 |
org.itracker.web.util.ServletContextUtils | Line |
In fact this shouldn't be needed / used, ITracker's JSP have way too much java code | 13 |