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 ClassTotal number of occurrencesTag strings used by tag class
@todo0@todo
TODO119TODO

Each tag is detailed below:

TODO

Number of occurrences found in the code: 119

org.itracker.core.resources.ITrackerResourceBundleLine
should dataArray be re-factored out?33
org.itracker.core.resources.ItrackerResourcesITLine
set languageDAO of ConfigurationService63
org.itracker.model.ConfigurationLine
every configuration item should have a name, similar to a Java property in a properties file. A description would be nice to have too. name + version should be the natural key. (note: we shouldn't allow 2 configuration items with the same name and version, but with different types). </p> <p/> <p> But since <code>name</code> is nullable, only the type and value can be used as natural key at the moment. This should be a temporary situation, because the value is allowed to change. </p>110
org.itracker.model.CustomFieldLine
use type-safe enum CustomField.DateFormat90
org.itracker.model.CustomFieldITLine
makit unit-test?12
org.itracker.model.CustomFieldValueITLine
makit unit-test?12
org.itracker.model.IssueActivityTypeTestLine
Remove call to depreciated code13
org.itracker.model.IssueAttachmentTestLine
does it still fail sometimes?90
org.itracker.model.IssueFieldLine
throw IllegalArgumentException instead of IssueException ?259
throw IllegalArgumentException instead of IssueException ?336
org.itracker.model.IssueFieldITLine
makit unit-test?13
org.itracker.model.NotificationTestLine
remove method from Notification36
org.itracker.model.ProjectLine
it would be a Set, not list86
org.itracker.model.WorkflowScriptLine
what's the expected type here? private Collection projectFields;62
org.itracker.model.util.IssueUtilitiesITLine
make tests using the database where appropriat (setup testdata from datasets)16
org.itracker.model.util.UserUtilitiesLine
Could use an enumeration41
must we really use this BASE64Encoder()? it seems to be not support by jrockit rt.jar hash = Base64.encodeBytes(raw);217
org.itracker.persistence.dao.BaseHibernateDaoImplITLine
fails currently try { dao.save(l); fail("entity is already persisted, save must throw DataAccessException"); } catch (DataAccessException dae) { // OK }54
org.itracker.persistence.dao.IssueActivityDAOLine
rename to findByIssue()20
org.itracker.persistence.dao.PermissionDAOImplITLine
fix implementation or test30
org.itracker.persistence.dao.UserDAOImplITLine
fails, please fix89
org.itracker.selenium.CreateIssueSeleniumITLine
not working from env assertTrue("System URL not contained in Message body, " + systemURL + ", " + smtpMessageBody, StringUtils.containsIgnoreCase(smtpMessageBody, systemURL));84
--No comment--115
org.itracker.selenium.SeleniumManagerLine
FIXME not working yet due to javascript issues126
org.itracker.services.IssueServiceLine
Add Javadocs here: document this whole class.describe what is the use for this method.132
shall we deprecate this one? why do we need to give it a userId?172
org.itracker.services.NotificationServiceLine
whats its use? <p/> Moved from {@link IssueService}, could not find out the purpose of this method. What will happen with this added {@link Notification}?103
org.itracker.services.authentication.AbstractPluggableAuthenticatorLine
Rewrite Javadocs here: we don't have session beans or EJBs anymore28
org.itracker.services.authentication.adsson.GetUserModelFromADPrivilegedActionLine
Add Javadocs here18
org.itracker.services.authentication.adsson.WindowsSSONAuthenticatorLine
get user from jcifs54
org.itracker.services.implementations.ConfigurationServiceImplLine
when current version is outdated?1099
org.itracker.services.implementations.IssueServiceITLine
test function for unavailable projects163
test function for unavailable projects190
test getUnassignedIssues(true)232
please somebody do tests on populate (multiple?) custom fields on an issue Test method for {@link org.itracker.services.IssueService#setIssueFields(java.lang.Integer, java.util.List)} .517
org.itracker.services.implementations.IssueServiceImplLine
Cleanup this file, go through all issues, todos, etc.40
move to {@link NotificationService}159
maybe it has no use at all. is it obsolete? when I'd set the issue-fields on an issue and then save/update issue, would it be good enough?545
There is no relation saved to database yet?797
configurationService should be set from context1023
translate activity for adding attachments IssueActivity activityAdd = new IssueActivity(issue, attachment.getUser(), IssueActivityType.ATTACHMENT_ADDED); activityAdd.setDescription(attachment.getOriginalFileName()); issue.getActivities().add(activityAdd);1189
move to {@link NotificationService} ?1347
org.itracker.services.implementations.NotificationServiceITLine
should be different owner and creator, projectowners109
org.itracker.services.implementations.NotificationServiceImplLine
Cleanup this file, go through all issues, todos, etc.43
final debugging/integration/implementation402
Decide if this code is really needed and document for what403
localize..624
check these 3 lines - do we need them?: notifications.addAll(issue.getNotifications()); notifications.add(notification); issue.setNotifications(notifications);783
is it needed to update issue too?788
url should be automatically generated by configuration (baseurl) and notification (issue-id).993
org.itracker.services.implementations.ProjectServiceImplLine
Cleanup this file, go through all issues, todos, etc.31
Add Javadocs here: document this whole class.32
implement Locale-aware ProjectFields.210
Decide if this code is really needed and document for what314
org.itracker.services.implementations.ProjectServiceImplITLine
--No comment--362
--No comment--369
org.itracker.services.implementations.UserServiceImplLine
use a factory to hide this.660
don't swallow exceptions!! MUST be propagated to the caller!!680
org.itracker.services.util.NamingUtilitiesITLine
Add Javadocs here: what is the purpose of this class?14
org.itracker.web.actions.PortalHomeActionLine
Action Cleanup23
since repeating code, set a common Comparator variable to contain the Comparator to use and execute the sort pre issue type only once.117
set the next value based on the request attribute!232
org.itracker.web.actions.admin.AdminHomeActionLine
performance issue when attachments size needs to be calculated over many issues ! select sum(size) from IssueAttachment151
org.itracker.web.actions.admin.configuration.EditConfigurationActionLine
Action Cleanup54
, set correct activity? IssueActivity activity = new IssueActivity(); activity.setActivityType(IssueActivityType.SYSTEM_UPDATE); activity.setDescription(ITrackerResources.getString("itracker.activity.system.status")); issues.get(i).getActivities().add(activity);219
need to fix this RJST activity.setIssue(issues.get(i));228
org.itracker.web.actions.admin.configuration.EditCustomFieldFormActionLine
Action Cleanup44
org.itracker.web.actions.admin.configuration.EditCustomFieldValueFormActionLine
Action Cleanup56
org.itracker.web.actions.admin.configuration.ImportDataProcessActionLine
, removed name attribute, so it's defaulted to the key155
, removed name attribute, so it's defaulted to the key159
?302
org.itracker.web.actions.admin.configuration.RemoveConfigurationItemActionLine
Action Cleanup44
org.itracker.web.actions.admin.project.EditProjectScriptFormActionLine
maybe intended to be an 'action' for the EditProjectScriptAction?50
org.itracker.web.actions.admin.report.EditReportActionLine
cleanup the service.81
org.itracker.web.actions.admin.report.EditReportFormActionLine
Action Cleanup35
org.itracker.web.actions.preferences.EditPreferencesActionLine
Action Cleanup60
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.EditPreferencesFormActionLine
Action Cleanup56
remove this hack, this should be handled central, there are other instances of this hack89
org.itracker.web.actions.project.CreateIssueActionLine
Action Cleanup55
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 ;) )195
org.itracker.web.actions.project.CreateIssueFormActionLine
Action Cleanup53
org.itracker.web.actions.project.EditIssueActionLine
can we make this token optional (configurable) and probably by form, not over the whole app..57
org.itracker.web.actions.project.EditIssueFormActionLine
Sort attachments Collections.sort(attachments, IssueAttachment.CREATE_DATE_COMPARATOR);121
org.itracker.web.actions.project.MoveIssueActionLine
is seems first condition is not necessary139
return detailed messages on the missing authorization140
org.itracker.web.actions.project.ViewIssueActionLine
verify this code.59
org.itracker.web.actions.user.LoginActionLine
Deprecated code. Can we get rid of it? Why is it so complicated?? We only want to iknow if success or no success160
no information about login-credentials should be shown235
org.itracker.web.actions.user.SelfRegisterActionLine
remove this hack, this should be handled central, there are other instances of this hack92
org.itracker.web.filters.ExecuteAlwaysFilterLine
itracker.web.generic.unknown for unknown user?172
this should be configured per-instance. Request server-name should only be used for exception and logged (configuration not found!)300
remove deprecated currLocale attribute318
org.itracker.web.forms.CustomFieldValueFormLine
setup request env for validation output110
org.itracker.web.forms.IssueFormLine
should issue, project, user, services be available too?200
why do we need to updateIssue here, and can not later? issueService.updateIssue(issue, user.getId());534
simplify this code, it's not readable, unsave yet.690
What happens here, validation?855
org.itracker.web.ptos.ProjectPTOLine
Code Cleanup: this method is not used and I don't like the name (what does setup stand for?)161
Decide if this code is really needed and document for what162
org.itracker.web.scheduler.tasks.MailNotificationLine
never used, commented, task added: int totalMessages = src_folder.getMessageCount();98
process message120
org.itracker.web.servlets.GenericControllerLine
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, ReportDownloadController47
org.itracker.web.servlets.ReportDownloadControllerLine
the 2-3 lines are most propably not used; commented, task added Commented more not used lines, but added a task for testing. HttpSession session = request.getSession(); UserModel user = (UserModel) session.getAttribute("user"); User user = (session == null ? null : (User) session.getAttribute(Constants.USER_KEY));53
org.itracker.web.servlets.RssFeedControllerLine
semantic style statusLocalizedString = IssueUtilities.getStatusName(issue.getStatus(), locale); severityLocalizedString = IssueUtilities.getSeverityName(issue.getSeverity(), locale); if (issue.getComponents().size() == 0) { componentsSize = ITrackerResources.getString( ListIssuesActionUtil.RES_KEY_UNKNOWN, locale); } else { componentsSize = issue.getComponents().get(0).getName() + (issue.getComponents().size() > 1 ? " (+)" : ""); }275
org.itracker.web.util.ImportExportTagsLine
--No comment--43
org.itracker.web.util.ImportExportUtilitiesLine
instead to have a string returned, it should directly serialize the export to the response-writer.120
unmarshal from is145
marshal to System.out153
org.itracker.web.util.ImportExportUtilitiesITLine
Not a valid XML..449
org.itracker.web.util.ImportHandlerLine
handle configuration for setting the BASE label265
Make status Enum386
org.itracker.web.util.ListIssuesActionUtilLine
check for other occurences for this constants and maybe place somewhere else?22
check from here...174
check to here...187
org.itracker.web.util.LoginUtilitiesLine
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>146
this is a hack, remove when possible344
org.itracker.web.util.ServletContextUtilsLine
In fact this shouldn't be needed / used, ITracker's JSP have way too much java code13

R.Ø.S.A.