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 |
|---|---|---|
| FIXME's | 1 | FIXME |
| TODO's | 128 | TODO |
Each tag is detailed below:
Number of occurrences found in the code: 1
| org.kuali.student.common.dto.DtoConstants | Line |
|---|---|
| Need to split out proposal states (ie. workflow states) versus dto states | 20 |
Number of occurrences found in the code: 128
| org.kuali.student.common.dto.DtoConstants | Line |
|---|---|
| Ideally this method should not be hardcoded here. Also determining next state may be a more complicated and not just be a simple sequence. | 35 |
| org.kuali.student.common.infc.Comparison | Line |
| Decide for complex Lists can they be checked to see how many occurences they have? | 53 |
| Decide on other operators | 54 |
| Deicde on operators to search collections inside such as any | 55 |
| Decide how to search on dynamic attributes | 56 |
| org.kuali.student.common.infc.Criteria | Line |
| Decide if null or empty list is ok? I.e. can you get ALL? | 29 |
| org.kuali.student.common.infc.Status | Line |
| switch this to hold a count or something, a boolean that always is true is confusing. | 26 |
| Figure out where the message came from because it is not in the R1 wiki https://wiki.kuali.org/display/KULSTU/statusInfo+Structure | 27 |
| org.kuali.student.core.atp.mock.AtpServiceMockPersistenceImpl | Line |
| Put the check once methods to access milestons/atp using relations are added to the service | 376 |
| Put the check once methods to access milestons/atp using relations are added to the service | 473 |
| org.kuali.student.datadictionary.RiceDataDictionaryValidatorImplDoesNotWorkButShould | Line |
| figure why this does not validate anything | 33 |
| org.kuali.student.datadictionary.infc.AttributeDefinitionInfc | Line |
| figure out how to handle sub-objects. I.e. using dot notation? | 34 |
| get Rice to rename this to getMinOccurs to it matches the setter. | 69 |
| Get RICE to rename this getMaxOccurs so it matches the setter | 78 |
| Get RICE to set a value that means UNBOUNDED, perhaps 999999999 | 79 |
| ask if this should be isUnique instead of getUnique? | 179 |
| Ask RICE to create an interface for ValidCharactersConstraint so we can return that instead of the generic constraint | 206 |
| org.kuali.student.datadictionary.infc.DictionaryEntryInfc | Line |
| determine how/if this works with the than the "unique" flag on attribute definition | 79 |
| determine how/if this works with the id | 80 |
| org.kuali.student.datadictionary.util.Bean2DictionaryConverter | Line |
| fix this to use a CollectionDefinition ad.setMaxOccurs(DictionaryConstants.UNBOUNDED); | 75 |
| org.kuali.student.datadictionary.util.CriteriaValidatorParser | Line |
| Consider getting all the errors and throw them as a group instead of as we find them | 106 |
| org.kuali.student.datadictionary.util.Dictionary2BeanComparer | Line |
| deal with collections compareAddDiscrepancy(discrepancies, adDict.getName() + " maxOccurs", adDict.getMaximumNumberOfElements(), adBean.getMaximumNumberOfElements()); | 66 |
| org.kuali.student.datadictionary.util.DictionaryFormatter | Line |
| Deal with collections if (ad.getMaximumNumberOfElements() != null) { if (ad.getMaximumNumberOfElements().intValue() == 0) { return "Not allowed"; } } | 421 |
| deal with collections | 609 |
| other more complex constraints | 760 |
| org.kuali.student.datadictionary.util.DictionaryValidator | Line |
| Cross compare to make sure min is not greater than max and that default value is valid itself | 124 |
| make the date parser configurable like the validator is | 219 |
| more validation | 270 |
| more validation | 327 |
| org.kuali.student.datadictionary.util.TestBaseValidCharsDictionary | Line |
| fix reg ex so it allows 4.0 but excludes 4.1, 4.2 etc... assertNotNull (v.processValidCharConstraint ("test", vc, null, "4.1")); | 212 |
| maybe allow these special characters | 308 |
| maybe allow these special characters | 353 |
| org.kuali.student.enrollment.lpr.mock.CriteriaMatcherInMemory | Line |
| Deal with lists if (ad.getMaximumNumberOfElements() == null) { return false; } if (ad.getMaximumNumberOfElements() <= 1) { return false; } | 91 |
| make sure the key is the same as the name | 170 |
| org.kuali.student.enrollment.lpr.mock.LuiPersonRelationServiceMockPersistenceImpl | Line |
| Add this method to the service interface | 359 |
| reevaluate if this method is needed -- I can see no use case for it Map<String, LuiPersonRelationTypeInfo> types = new HashMap(); for (LuiPersonRelationInfo lpr : this.lprCache.values()) { if (!lpr.getPersonId().equals(personId)) { continue; } if (!lpr.getLuiId().equals(luiId)) { continue; } if (!lpr.getState().equals(relationState)) { continue; } LuiPersonRelationTypeInfo type = this.getLuiPersonRelationType(lpr.getType()); types.put(type.getKey(), type); } return new ArrayList(types.values()); } | 379 |
| org.kuali.student.enrollment.lpr.model.LuiPersonRelationEntity | Line |
| - need to retrieve the LuiPersonRelationState based on the return of dto.getState()? this.setPersonRelationState(new LuiPersonRelationStateEntity(dto.getStateKey())); | 65 |
| - need to retrieve the LuiPersonRelationType based on the return of dto.getType()? this.setPersonRelationType(new LuiPersonRelationTypeEntity(dto.getTypeKey())); | 67 |
| org.kuali.student.enrollment.lpr.service.conformance.tests.TestLuiPersonRelationServicePersistenceConformance | Line |
| when AttributeInfo gets it's own ID do the find by ID instead of values | 288 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 313 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 331 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 347 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 365 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 382 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 398 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 414 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 429 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 444 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 462 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 479 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 495 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 511 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 527 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 543 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 561 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 579 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 598 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 617 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 633 |
| review the generated test code and remove the default call to fail. fail("The test case is a prototype."); } | 651 |
| org.kuali.student.enrollment.lpr.service.decorators.LuiPersonRelationServiceAuthorizationDecorator | Line |
| Auto-generated method stub | 54 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 80 |
| org.kuali.student.enrollment.lpr.service.impl.LuiPersonRelationServiceImpl | Line |
| stub mock impl | 81 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 100 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 106 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 112 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 118 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 124 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 130 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 136 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 142 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 149 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 155 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 161 |
| - Attributes? | 181 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 187 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 193 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 199 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 205 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 211 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 217 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 223 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 229 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 235 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 253 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 264 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 270 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 276 |
| org.kuali.student.enrollment.lui.mock.LuiServiceMockPersistenceImpl | Line |
| consider speading up the list search by converting to a hashmap | 204 |
| consider speading up the list search by converting to a hashmap | 217 |
| consider speading up the list search by converting to a hashmap | 238 |
| org.kuali.student.enrollment.lui.service.impl.LuiServiceImpl | Line |
| Auto-generated method stub | 31 |
| Auto-generated method stub | 40 |
| Auto-generated method stub | 48 |
| Auto-generated method stub | 57 |
| Auto-generated method stub | 66 |
| Auto-generated method stub | 75 |
| Auto-generated method stub | 83 |
| Auto-generated method stub | 92 |
| Auto-generated method stub | 101 |
| Auto-generated method stub | 109 |
| Auto-generated method stub | 118 |
| Auto-generated method stub | 127 |
| Auto-generated method stub | 136 |
| Auto-generated method stub | 145 |
| Auto-generated method stub | 154 |
| Auto-generated method stub | 163 |
| Auto-generated method stub | 172 |
| Auto-generated method stub | 181 |
| Auto-generated method stub | 191 |
| Auto-generated method stub | 201 |
| Auto-generated method stub | 210 |
| Auto-generated method stub | 220 |
| Auto-generated method stub | 230 |
| Auto-generated method stub | 242 |
| Auto-generated method stub | 253 |
| Auto-generated method stub | 262 |
| org.kuali.student.kim.identity.mock.IdentityServiceMockImpl | Line |
| deal with Campus Codes | 128 |
| deal with multiple affiliations | 129 |
| org.kuali.student.kim.permission.mock.PermissionServiceMockImpl | Line |
| implement this check | 79 |
| implement this check | 84 |
| implement this check | 89 |
| Implement this | 145 |
| figure out what this is | 174 |
| org.kuali.student.kim.permission.mock.RoleServiceMockImpl | Line |
| implement check | 193 |
| org.kuali.student.r2.common.entity.MetaEntity | Line |
| what about versionInd? | 120 |
| org.kuali.student.r2.common.entity.TypeEntity | Line |
| what about RefObjId? | 63 |
| - refactor this into a central place; probably Igor's Converter | 68 |
