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 | 52 | FIXME |
| TODO's | 622 | TODO |
Each tag is detailed below:
Number of occurrences found in the code: 52
| org.apache.ojb.broker.platforms.KualiMySQLSequenceManagerImpl | Line |
|---|---|
| should we be closing this connection in a finally block? | 46 |
| should we be closing this result set in a finally block? | 53 |
| org.kuali.rice.core.api.criteria.LookupCustomizer | Line |
| add wilcards to make predicate transform more flexible ie. EqualsPredicate to AndPredicate | 35 |
| org.kuali.rice.core.api.resourceloader.ResourceLoaderContainer | Line |
| RICE MODULARITY | 72 |
| RICE MODULARITY | 78 |
| RICE MODULARITY | 91 |
| org.kuali.rice.core.framework.persistence.jdbc.sql.SqlBuilder | Line |
| wtf - weird! | 187 |
| org.kuali.rice.core.impl.config.module.ModuleConfigurer | Line |
| this class must be put in an API module somehow | 53 |
| attempting to see if this fixes the ksb tests... | 105 |
| org.kuali.rice.core.impl.criteria.CriteriaLookupServiceOjbImpl | Line |
| this method exists to make javac happy - to bad I didn't know how to fix this.... | 223 |
| org.kuali.rice.core.impl.resourceloader.RiceResourceLoaderFactory | Line |
| RICE MODULARITY hack to not break the hack in ResourceLoaderContainer.moveKSBLoadersDownHack(); | 46 |
| org.kuali.rice.kew.actionlist.dao.impl.ActionListDAOJpaImpl | Line |
| this causes null pointer - actionItemExt.setGroup(actionItem.getGroup()); actionItemExt.setLastApprovedDate(actionItem.getLastApprovedDate()); actionItemExt.setRowStyleClass(); | 122 |
| org.kuali.rice.kew.actionrequest.dao.impl.ActionRequestDAOJpaImpl | Line |
| should be jpa bulk update? | 65 |
| org.kuali.rice.kew.bo.lookup.DocSearchCriteriaDTOLookupableHelperServiceImpl | Line |
| ctk - make sure and check that it's ok to do this here. I may move this out to the doc search processor | 448 |
| any reason this is inside this "if" instead of the outer one, like above - this seems inconsistent | 1027 |
| org.kuali.rice.kew.bo.lookup.DocumentRouteHeaderValueLookupableHelperServiceImpl | Line |
| ctk - make sure and check that it's ok to do this here. I may move this out to the doc search processor | 333 |
| org.kuali.rice.kew.config.ThinClientKEWConfigurer | Line |
| RICE MODULARITY need to fix this class at some point | 26 |
| org.kuali.rice.kew.docsearch.DocumentLookupCriteriaBuilder | Line |
| don't force this when dd changes are in, instead delete line 1 row below and uncomment one two lines below | 113 |
| org.kuali.rice.kew.docsearch.DocumentLookupCriteriaProcessorKEWAdapter | Line |
| definitely not the best place for this | 219 |
| org.kuali.rice.kew.docsearch.StandardDocumentSearchResultProcessor | Line |
| Chris get rid of all sortable references! | 57 |
| org.kuali.rice.kew.doctype.DocumentTypePolicyEnum | Line |
| needs docs | 31 |
| org.kuali.rice.kew.dto.DTOConverter | Line |
| we assume there is only one for now | 184 |
| org.kuali.rice.kew.engine.node.dao.impl.RouteNodeDAOJpaImpl | Line |
| Can we do this better using just the JPQL query? | 109 |
| Not sure this query is returning what it needs to | 121 |
| org.kuali.rice.kew.engine.node.var.SetVarNode | Line |
| validation should be checked in documenttypexmlparser to start with | 65 |
| org.kuali.rice.kew.routeheader.dao.impl.DocumentRouteHeaderDAOJpaImpl | Line |
| Should this check for hibernate LockAcquisitionException | 137 |
| org.kuali.rice.kew.rule.GenericRoleAttribute | Line |
| ok, this MUST return true, as it IS evaluated (don't know why) maybe investigate only calling isMatch on rule attributes as it doesn't seem that matching is relevant for role attributes...is it? throw new UnsupportedOperationException("Role attributes do not implement isMatch"); | 68 |
| org.kuali.rice.kew.rule.dao.impl.RuleAttributeDAOJpaImpl | Line |
| This query is returning multiple rows, which one should it return | 97 |
| org.kuali.rice.kew.rule.dao.impl.RuleDAOJpaImpl | Line |
| nothing uses this, it's not in ruleDAO interface public List findRuleBaseValuesByObjectGraph(RuleBaseValues ruleBaseValues) { ruleBaseValues.setCurrentInd(Boolean.TRUE); ruleBaseValues.setTemplateRuleInd(Boolean.FALSE); return (List) new QueryByObject(entityManager,ruleBaseValues).toQuery().getResultList(); } | 209 |
| org.kuali.rice.kew.rule.service.impl.RuleServiceTest | Line |
| When the operation below throws the expected exception (when JPA/Hibernate is used), it appears to do so while returning from the method call (at which time the expected-to-fail saving operation gets committed by Hibernate). Unfortunately, the exception gets thrown at the wrong time when returning, because any attempts to run subsequent unit tests or unit test methods during the same JUnit test run will fail, due to NOWAIT exceptions during test case startup. | 344 |
| org.kuali.rice.kim.document.rule.AttributeValidationHelper | Line |
| This does not use the correct error path yet - may need to be moved up so that the error path is known Also, the above code would overwrite messages on the same attributes (namespaceCode) but on different rows | 122 |
| org.kuali.rice.kim.impl.responsibility.ReviewResponsibilityMaintainable | Line |
| this seems really dubious | 49 |
| org.kuali.rice.kim.service.impl.KimModuleService | Line |
| needs to send unbounded flag | 123 |
| org.kuali.rice.kim.service.impl.UiDocumentServiceImpl | Line |
| Huh!?!?, control is a Map in the above code but a ControlDefinition here?!?!? Maybe this should use the AttributesMapBuilder code to create this | 301 |
| This should be pulling by the PK, not using another method which pulls multiple records and then finds the right one here! | 2129 |
| org.kuali.rice.kim.service.support.impl.KimTypeServiceBase | Line |
| I don't like this - if two attributes have the same sort code, they will overwrite each other | 774 |
| org.kuali.rice.kns.authorization.PermissionDerivedRoleTypeServiceImpl | Line |
| dangerous - data changes could cause an infinite loop - should add thread-local to trap state and abort | 95 |
| org.kuali.rice.kns.datadictionary.AttributeDefinition | Line |
| JLR - need to recreate this functionality using the ValidCharsConstraint logic | 220 |
| org.kuali.rice.kns.datadictionary.validation.capability.LookupConstrainable | Line |
| rename to getLookupConstraint() | 27 |
| org.kuali.rice.kns.datadictionary.validation.processor.CaseConstraintProcessor | Line |
| not sure if the definition and attribute value reader should change under this case | 71 |
| org.kuali.rice.kns.datadictionary.validation.processor.ValidCharactersConstraintProcessor | Line |
| This shouldn't surface label key itself to the user - it should look up the label key, but this needs to be implemented in Rice | 188 |
| org.kuali.rice.kns.service.impl.DictionaryValidationServiceImpl | Line |
| JLR - this is what the code was doing effectively already, but seems weird not to throw an exception here if you try to validate something that doesn't have an attribute definition | 182 |
| JLR : Replacing this logic with KS-style validation is trickier, since KS validation requires a DataProvider object that can look back and find other attribute values aside from the one we're working on. Also - the date stuff below is implemented very differently. validator.validateAttributeField(businessObject, fieldName); | 463 |
| JLR - this is now redundant and should be using the same code as the required processing elsewhere, but the control definition stuff doesn't really fit it doesn't seem to be used anywhere | 604 |
| There's got to be a cleaner way of doing this. | 1110 |
| WJG - Do we want to make entry be constrainable? | 1138 |
| I think we may want to use a new CollectionConstrainable interface instead to obtain from DictionaryObjectAttributeValueReader | 1181 |
| It's inefficient to be creating new attribute reader for each item in collection | 1195 |
| org.kuali.rice.kns.service.impl.SessionDocumentServiceImpl | Line |
| Currently using formKey for sessionId | 262 |
| org.kuali.rice.kns.util.ObjectUtils | Line |
| (laran) This dependence should be inverted. Instead of having a core class depend on PojoPropertyUtilsBean, which is in the web layer, the web layer should depend downward to the core. | 212 |
| org.kuali.rice.kns.web.struts.action.KualiLookupAction | Line |
| any reason this is inside this "if" instead of the outer one, like above - this seems inconsistent | 307 |
| org.kuali.rice.kns.web.ui.Field | Line |
| this one definitely seems iffy, could be confused with regular fieldType, is there another better name or can this go away? | 109 |
Number of occurrences found in the code: 622
| edu.sampleu.bookstore.maintenance.AuthorMaintainable | Line |
|---|---|
| Auto-generated method stub | 24 |
| edu.sampleu.bookstore.maintenance.BookMaintainable | Line |
| Auto-generated method stub | 25 |
| edu.sampleu.bookstore.rule.AuthorMaintenanceDocumentRule | Line |
| Auto-generated method stub | 31 |
| mocks.MockJavaMailSender | Line |
| g - THIS METHOD NEEDS JAVADOCS | 52 |
| g - THIS METHOD NEEDS JAVADOCS | 62 |
| g - THIS METHOD NEEDS JAVADOCS | 72 |
| g - THIS METHOD NEEDS JAVADOCS | 82 |
| g - THIS METHOD NEEDS JAVADOCS | 92 |
| g - THIS METHOD NEEDS JAVADOCS | 102 |
| g - THIS METHOD NEEDS JAVADOCS | 112 |
| mocks.MockRuleServiceImpl | Line |
| ewestfal - THIS METHOD NEEDS JAVADOCS | 74 |
| org.apache.ojb.broker.platforms.PlatformMckoiImpl | Line |
| Shold we throw some kind of exception here if the escape character is different ? | 175 |
| org.kuali.rice.core.api.resourceloader.ParentChildResourceLoader | Line |
| ewestfal - THIS METHOD NEEDS JAVADOCS | 135 |
| org.kuali.rice.core.framework.persistence.jdbc.sql.Criteria | Line |
| Rewrite this class with a better criteria building algorithm. | 45 |
| org.kuali.rice.core.framework.persistence.jdbc.sql.SqlBuilder | Line |
| What to do here now that the JPA version does not extend platform aware? | 116 |
| What to do here now that the JPA version does not extend platform aware? | 315 |
| org.kuali.rice.core.framework.persistence.jpa.RiceNumericStringSequenceStyleGenerator | Line |
| Make sure this generator works with MySQL, and verify that the correct numeric Hibernate Type is being used in the configure() method override. | 31 |
| Should we add a configuration parameter for allowing users to specify a different supported Hibernate numeric Type if desired? | 40 |
| org.kuali.rice.core.framework.persistence.jpa.metadata.EntityDescriptor | Line |
| Cache by name and probably by class | 142 |
| Cache by name and probably by class | 155 |
| org.kuali.rice.core.framework.persistence.jpa.metadata.MetadataManager | Line |
| Should we add inverse join columns? | 591 |
| Implement default name See: http://www.oracle.com/technology/products/ias/toplink/jpa/resources/toplink-jpa-annotations.html#JoinColumn | 645 |
| org.kuali.rice.core.framework.persistence.ojb.TransactionManagerFactory | Line |
| what to do here return KSBServiceLocator.getTransactionManager(); | 61 |
| org.kuali.rice.core.framework.persistence.platform.DatabasePlatform | Line |
| Had to move this down into embedded source because of the OJB dependencies. This probably will go away once we get rid of the embedded plugin. | 27 |
| refactor to use a parsed Date object or milliseconds instead of date String | 67 |
| This method should be replaced by the "prepared statement" functionality in the future. | 122 |
| org.kuali.rice.core.impl.config.property.AbstractBaseConfig | Line |
| why was this using ConfigContext.getCurrentConfig in the rice BaseConfig | 123 |
| can this be moved to default config file | 172 |
| why going to currentContextConfig | 173 |
| org.kuali.rice.core.impl.config.property.HierarchicalConfigParser | Line |
| consider implementing with iteration instead of recursion, and using the jakarta commons support for parameter expansion | 212 |
| org.kuali.rice.core.impl.impex.xml.ClassLoaderEntityResolver | Line |
| maybe prefix should be changed from "resource:" to "internal:" or just "workflow:" given that it can be resolved in arbitrary ways other than ClassLoader "resources" | 31 |
| revisit making this more sophisticated than just the classloader of this class (thread context classloader? plugin classloader?) | 72 |
| org.kuali.rice.core.impl.impex.xml.XmlExporterServiceImpl | Line |
| KULRICE-4420 - this needs cleanup | 67 |
| org.kuali.rice.core.impl.impex.xml.XmlImpexRegistryImpl | Line |
| --No comment-- | 27 |
| org.kuali.rice.core.impl.persistence.dao.GenericDaoOjb | Line |
| g1zhang - THIS METHOD NEEDS JAVADOCS | 305 |
| g1zhang - THIS METHOD NEEDS JAVADOCS | 318 |
| org.kuali.rice.core.impl.style.StyleServiceImpl | Line |
| should really be caching here! | 66 |
| org.kuali.rice.core.mail.EmailContent | Line |
| supercede this with Spring framework and/or JavaMail mail classes | 22 |
| org.kuali.rice.core.util.ClassLoaderUtils | Line |
| why not make this so that it throws ClassNoteFoundException and doesn't return null? this is more standard behavior... | 135 |
| org.kuali.rice.core.web.format.Formatter | Line |
| add registry for non-navigable classes so there's a way to disable formatting selectively for given types contained in arrays or Collections. | 274 |
| org.kuali.rice.core.web.format.PhoneNumberFormatter | Line |
| foreign phone numbers can be different lengths end Kuali Foundation modification | 35 |
| org.kuali.rice.core.web.format.TypeFormatter | Line |
| Auto-generated method stub | 39 |
| Auto-generated method stub | 44 |
| org.kuali.rice.devtools.generators.ImmutableJaxbGenerator | Line |
| modify this constructor as needed to pass any required values and invoke the appropriate 'setter' methods"); | 332 |
| modify as needed to pass any required values and add them to the signature of the 'create' method"); | 344 |
| if create() is modified to accept required parameters, this will need to be modified"); | 354 |
| add validation of input value if required and throw IllegalArgumentException if needed"); | 373 |
| org.kuali.rice.edl.impl.EDLServlet | Line |
| Fix this in a better way (reworking the command structure maybe?) fix for KULRICE-4057 to make sure we don't destory docContent on empty command params | 95 |
| Do we not want to set the content type for the response? | 107 |
| org.kuali.rice.edl.impl.OfficeOfAffirmativeActionTest | Line |
| --No comment-- | 107 |
| --No comment-- | 111 |
| --No comment-- | 115 |
| --No comment-- | 119 |
| org.kuali.rice.edl.impl.components.ValidationComponent | Line |
| in the future, allow this to be pluggable, hardcode for now | 106 |
| remove - handled this in the widgets Element edlElement = EDLXmlUtils.getEDLContent(dom, false); Element edlSubElement = EDLXmlUtils.getOrCreateChildElement(edlElement, "data", true); NodeList versionNodes = edlSubElement.getChildNodes(); for (int i = 0; i < versionNodes.getLength(); i++) { Element version = (Element) versionNodes.item(i); String current = version.getAttribute("current"); if (current == "true") { NodeList fieldNodes = version.getChildNodes(); for (int j = 0; j < fieldNodes.getLength(); j++) { Element field = (Element) fieldNodes.item(j); String fieldName = field.getAttribute("name"); if(fieldName.equals(key)) { field.setAttribute("invalid", "true"); break; } } } } | 117 |
| org.kuali.rice.edl.impl.components.WorkflowDocumentActions | Line |
| the problem here is that the XML is still updated on a cancel so we end up without any attribute content in the document content | 112 |
| org.kuali.rice.edl.impl.components.WorkgroupWorkflowEDLConfigComponent | Line |
| add support for namespace here! | 70 |
| org.kuali.rice.kcb.exception.KCBCheckedException | Line |
| arh14 - THIS CONSTRUCTOR NEEDS A JAVADOC | 35 |
| Auto-generated constructor stub | 37 |
| org.kuali.rice.kcb.service.impl.MessageDelivererRegistryServiceImpl | Line |
| we'll need to implement a plugin registry discovery mechanism long term. | 49 |
| org.kuali.rice.kcb.web.spring.UserPreferencesController | Line |
| does not traverse bus yet | 84 |
| org.kuali.rice.ken.deliverer.impl.KEWActionListMessageDeliverer | Line |
| move hardcoded web controller actions here... | 119 |
| org.kuali.rice.ken.service.impl.NotificationMessageContentServiceImpl | Line |
| modify clone recipient list so that: | 405 |
| org.kuali.rice.ken.service.impl.NotificationServiceImpl | Line |
| implement pessimistic locking on the message delivery | 193 |
| implement pessimistic locking on all these message deliveries now, do dispatch in reverse...dismiss each message delivery via the appropriate deliverer | 218 |
| locking mark as unlocked messageDelivery.setLockedDate(null); | 244 |
| org.kuali.rice.ken.services.impl.NotificationRecipientServiceKimImplTest | Line |
| --No comment-- | 57 |
| org.kuali.rice.ken.util.Util | Line |
| optimize this | 197 |
| org.kuali.rice.ken.util.XslSourceResolver | Line |
| Auto-generated constructor stub | 37 |
| org.kuali.rice.ken.xpath.XPathTest | Line |
| optimize this | 90 |
| org.kuali.rice.kew.actionlist.OutboxTest | Line |
| this can no longer be set on the fly and grabbed through the preference service (default values are set at startup) | 234 |
| this is a prime candidate for a mocking tool | 235 |
| org.kuali.rice.kew.actionlist.dao.impl.ActionListDAOJpaImpl | Line |
| , merge will not update the outboxitem pointer to the merged entity | 538 |
| org.kuali.rice.kew.actionlist.dao.impl.ActionListDAOOjbImpl | Line |
| delyea: add all workgroups here? orCrit.addOrCriteria(userCrit); orCrit.addOrCriteria(groupCrit); crit.addAndCriteria(orCrit); crit.addEqualTo("delegationType", KEWConstants.DELEGATION_PRIMARY); filter.setDelegationType(KEWConstants.DELEGATION_PRIMARY); filter.setExcludeDelegationType(false); filteredByItems += filteredByItems.length() > 0 ? ", " : ""; filteredByItems += "Primary Delegator Id"; filterOn = true; } else if (filter.getDelegatorId().trim().equals(KEWConstants.ALL_CODE)) { // user wishes to see all secondary delegations crit.addEqualTo("workflowId", user.getWorkflowUserId().getWorkflowId()); crit.addEqualTo("delegationType", KEWConstants.DELEGATION_SECONDARY); filter.setDelegationType(KEWConstants.DELEGATION_SECONDARY); filter.setExcludeDelegationType(false); filteredByItems += filteredByItems.length() > 0 ? ", " : ""; filteredByItems += "Secondary Delegator Id"; filterOn = true; } else if (filter.getPrimaryDelegateId() != null && !"".equals(filter.getPrimaryDelegateId().trim())) { // user wishes to see primary delegation for a single user Criteria userCrit = new Criteria(); Criteria groupCrit = new Criteria(); Criteria orCrit = new Criteria(); userCrit.addEqualTo("delegatorWorkflowId", user.getWorkflowUserId().getWorkflowId()); | 380 |
| delyea: add all workgroups here? orCrit.addOrCriteria(userCrit); orCrit.addOrCriteria(groupCrit); crit.addAndCriteria(orCrit); crit.addEqualTo("delegationType", KEWConstants.DELEGATION_PRIMARY); filter.setDelegationType(KEWConstants.DELEGATION_PRIMARY); filter.setExcludeDelegationType(false); filteredByItems += filteredByItems.length() > 0 ? ", " : ""; filteredByItems += "Primary Delegator Id"; filterOn = true; } else if (filter.getDelegatorId() != null && !"".equals(filter.getDelegatorId().trim())) { // user wishes to see secondary delegation for a single user crit.addEqualTo("workflowId", user.getWorkflowUserId().getWorkflowId()); crit.addEqualTo("delegationType", KEWConstants.DELEGATION_SECONDARY); filter.setDelegationType(KEWConstants.DELEGATION_SECONDARY); filter.setExcludeDelegationType(false); Criteria userCrit = new Criteria(); Criteria groupCrit = new Criteria(); if (filter.isExcludeDelegatorId()) { Criteria userNull = new Criteria(); userCrit.addNotEqualTo("delegatorWorkflowId", filter.getDelegatorId()); userNull.addIsNull("delegatorWorkflowId"); userCrit.addOrCriteria(userNull); Criteria groupNull = new Criteria(); groupCrit.addNotEqualTo("delegatorWorkgroupId", filter.getDelegatorId()); groupNull.addIsNull("delegatorWorkgroupId"); groupCrit.addOrCriteria(groupNull); crit.addAndCriteria(userCrit); crit.addAndCriteria(groupCrit); } else { Criteria orCrit = new Criteria(); userCrit.addEqualTo("delegatorWorkflowId", filter.getDelegatorId()); groupCrit.addEqualTo("delegatorWorkgroupId", filter.getDelegatorId()); orCrit.addOrCriteria(userCrit); orCrit.addOrCriteria(groupCrit); crit.addAndCriteria(orCrit); } filteredByItems += filteredByItems.length() > 0 ? ", " : ""; filteredByItems += "SeDelegator Id"; filterOn = true; } else if (StringUtils.isBlank(filter.getPrimaryDelegateId()) && StringUtils.isBlank(filter.getDelegatorId())) { crit.addEqualTo("workflowId", user.getWorkflowUserId().getWorkflowId()); if (filter.getDelegationType() != null && !"".equals(filter.getDelegationType().trim())) { if (filter.isExcludeDelegationType()) { Criteria critNotEqual = new Criteria(); Criteria critNull = new Criteria(); critNotEqual.addNotEqualTo("delegationType", filter.getDelegationType()); critNull.addIsNull("delegationType"); critNotEqual.addOrCriteria(critNull); crit.addAndCriteria(critNotEqual); } else { crit.addEqualTo("delegationType", filter.getDelegationType()); } } } | 405 |
| org.kuali.rice.kew.actionrequest.dao.impl.ActionRequestDAOJpaImpl | Line |
| Runtime might not be the right thing to do here... | 255 |
| org.kuali.rice.kew.actionrequest.dao.impl.ActionRequestDAOOjbImpl | Line |
| Runtime might not be the right thing to do here... | 113 |
| org.kuali.rice.kew.actionrequest.service.impl.ActionRequestServiceImpl | Line |
| this still won't work in certain cases when checking from the root | 794 |
| since we only store the workgroup id for workgroup requests, if the user is in a workgroup that has a request than we need get all the requests with workgroup ids and see if our user is in that group | 990 |
| org.kuali.rice.kew.actions.ActionRegistryImpl | Line |
| ActionTakenEvent is not an interface so we can't fetch them through the GlobalResourceLoader, for now, just use the ObjectDefinitionResolver | 114 |
| the exception handling here is a bit wonky | 118 |
| org.kuali.rice.kew.actions.AdHocRouteTest | Line |
| test adhocing of approve requests | 282 |
| org.kuali.rice.kew.actions.MoveDocumentAction | Line |
| this whole bit is a bit hacky at the moment | 121 |
| org.kuali.rice.kew.actions.ReturnToPreviousNodeAction | Line |
| will this work properly in the case of an ALL APPROVE role requests with some of the requests already completed? | 100 |
| might not need this, if so, do role check | 184 |
| what if the activeNodeInstance already has next nodes? | 401 |
| really we need to call transitionTo on this node, how can we do that? this isn't an issue yet because we only allow simple nodes and split nodes at the moment which do no real work on transitionTo but we may need to enhance that in the future | 411 |
| org.kuali.rice.kew.actions.RouteDocumentAction | Line |
| this will get all pending AR's even if they haven't been in an action list... This seems bad | 100 |
| org.kuali.rice.kew.actions.SuperUserActionRequestApproveEvent | Line |
| this should be checked | 89 |
| org.kuali.rice.kew.actions.SuperUserApproveEvent | Line |
| this is used because calling this code from SuperUserAction without it causes an optimistic lock | 64 |
| org.kuali.rice.kew.actiontaken.ActionTakenValue | Line |
| this stinks to have to have a dependency on UserSession here | 109 |
| this stinks to have to have a dependency on UserSession here | 143 |
| org.kuali.rice.kew.actiontaken.dao.impl.ActionTakenDAOJpaImpl | Line |
| perhaps runtime isn't the best here, maybe a dao runtime exception | 121 |
| org.kuali.rice.kew.actiontaken.dao.impl.ActionTakenDAOOjbImpl | Line |
| perhaps runtime isn't the best here, maybe a dao runtime exception | 111 |
| org.kuali.rice.kew.batch.XmlIngestionTest | Line |
| beef this up need a reliable way to test if the file arrived in the right date-stamped subdirectory (maybe just pick the last, or first directory?) | 161 |
| org.kuali.rice.kew.batch.XmlPollerServiceImpl | Line |
| try reading the pending file or stop? | 216 |
| org.kuali.rice.kew.batch.XmlSchemaTest | Line |
| finish this test when we figure out how to conveniently use namespaces with XPath | 103 |
| org.kuali.rice.kew.bo.lookup.DocSearchCriteriaDTOLookupableHelperServiceImpl | Line |
| ideally implement KNS updates to make this not require code from the parent | 116 |
| set the criteria on this from below method instead of this (so we're not calling out twice for the same object) | 122 |
| can we add something to this to get it to highlight the right field too? Maybe in arg1 | 157 |
| additional BORestrictions through generator or component to lock down per document? | 222 |
| where to get these from? HtmlData returnUrl = new AnchorHtmlData(); | 234 |
| find a better place for this | 269 |
| change this to an enum (or another dd property) | 272 |
| check exisiting formatter here, ideally we should be getting this formatter from col.getFormatter in most cases formatters | 311 |
| can we call into a method in the result processor to get this (or set something on the criteria) propValue = maskValueIfNecessary(element.getClass(), col.getPropertyName(), propValue, businessObjectRestrictions); | 352 |
| change to grab URL from config variable | 361 |
| chris - this method should call the criteria processor adapter which will call the criteria processor (either standard or custom) and massage the data into the proper format this is called by setbo in super(which is called by form) so should be called when the page needs refreshing | 483 |
| move over code that checks for doctype (actually should that be in the refresh, since that's where the doc type will be coming back to?) | 487 |
| This should probably be moved into spring injection since it's a constant | 525 |
| chris - this method should call the criteria processor adapter which will call the criteria processor (either standard or custom) and massage the data into the proper format this is called by setbo in super(which is called by form) so should be called when the page needs refreshing | 568 |
| move over code that checks for doctype (actually should that be in the refresh, since that's where the doc type will be coming back to?) | 572 |
| This should probably be moved into spring injection since it's a constant | 606 |
| also check if standard here (maybe from object if use criteria) | 680 |
| chris - THIS METHOD NEEDS JAVADOCS return super.checkForAdditionalFields(fieldValues); | 857 |
| can we add something to this to get it to highlight the right field too? Maybe in arg1 | 912 |
| is there a better way to override a single field value? | 1059 |
| Chris - (2 stage clear) set the isOnlyDocTypeFilled, to true if only doc type coming in (besides hidden) and false otherwise) | 1240 |
| org.kuali.rice.kew.bo.lookup.DocumentRouteHeaderValueLookupableHelperServiceImpl | Line |
| chris - THIS METHOD NEEDS JAVADOCS | 93 |
| KNS updates to make this not require code from the parent | 112 |
| move this into actual adapter as well | 118 |
| where to get these from? | 165 |
| convert columns either here or in the getColumns method ADDED (3) | 169 |
| change to grab URL from config variable | 250 |
| move this into adapter | 295 |
| temporary work around until is a criteria | 309 |
| chris - this method should call the criteria processor adapter which will call the criteria processor (either standard or custom) and massage the data into the proper format this is called by setbo in super(which is called by form) so should be called when the page needs refreshing | 361 |
| move over code that checks for doctype (actually should that be in the refresh, since that's where the doc type will be coming back to?) | 365 |
| get this from parameters (or bo?) | 384 |
| same Doc type if(docType==oldDocType) { | 387 |
| same Doc type | 394 |
| ctk Add this code back in when KNS impacting changes are worked back | 407 |
| also check if standard here (maybe from object if use criteria) | 430 |
| move these strings to constants (probably after moving this to criteria | 436 |
| org.kuali.rice.kew.clientapp.ThinClientTest | Line |
| remove when http://jira.codehaus.org/browse/SUREFIRE-535 is fixed | 163 |
| org.kuali.rice.kew.config.KEWConfigurer | Line |
| Because a lot of our lifecycles live behind the embedded plugin and the KEWConfigurer does not, this is a simple measure to load these without having to deal with the removal of the embedded plugin right away. | 114 |
| org.kuali.rice.kew.config.ThinClientKEWConfigurer | Line |
| this seems like a total hack the way this is happening, see the addModulesResourceLoaders // method RiceConfigurer as well kewConfigurer.getResourceLoadersToRegister(); } | 45 |
| org.kuali.rice.kew.config.ThinClientResourceLoader | Line |
| we need to add support for other invocation protocols and implementations, but for now... | 186 |
| org.kuali.rice.kew.docsearch.CustomDocumentSearchGeneratorTest | Line |
| delyea - adding underscore below invalidates via REGEX but doesn't blow up on route or addSearchable? | 193 |
| org.kuali.rice.kew.docsearch.DocSearchCriteriaDTO | Line |
| this is here for historic reasons and has been replaced by a managed state in the CriteriaDTO... eliminate | 595 |
| chris - THIS METHOD NEEDS JAVADOCS | 805 |
| org.kuali.rice.kew.docsearch.DocumentLookupCriteriaBuilder | Line |
| Chris, Should be reevaluated in whole after released for KFS This method ... | 88 |
| org.kuali.rice.kew.docsearch.DocumentLookupCriteriaProcessorKEWAdapter | Line |
| remove this and use service locator or try helper in WorkflowUtils if sufficient | 45 |
| instead of hardcoding these let's see about getting them from spring | 158 |
| special processing for some field types | 187 |
| can these be used directly in values finder also there is an option key and value property that could probably be used by all of these | 205 |
| fix this in criteria this field shouldn't be blank values otherwise have to reset this for some reason | 212 |
| move to values finder class | 222 |
| can these be used directly in values finder also there is an option key and value property that could probably be used by all of these | 232 |
| fix this in criteria this field shouldn't be blank values otherwise have to reset this for some reason | 239 |
| chris - How should this handle advanced? I thought we were only hiding main | 369 |
| org.kuali.rice.kew.docsearch.DocumentSearchGenerator | Line |
| delyea - documentation | 29 |
| org.kuali.rice.kew.docsearch.DocumentSearchTest | Line |
| Add some wildcard testing for the document type attribute once wildcards are usable with it. | 339 |
| org.kuali.rice.kew.docsearch.StandardDocumentSearchGenerator | Line |
| do we really want to allow the document search if there is no User Session? This is mainly to allow for the unit tests to run but I wonder if we need to push the concept of the "executing user" into the doc search api in some way... | 1010 |
| this is currently concatenating strings together with HTML elements, this seems bad in this location, perhaps we should move this to the web layer (and perhaps enhance the searchable attributes portion of the DocSearchDTO data structure?) | 1054 |
| we should probably clean this up some, but we are going to exclude those KeyValues that have a null label. This will happen in the case of Quickfinders which don't really represent criteria anyway. Note however, that it is legal for the label to be the empty string. At some point we will probably need to do some more work to untangle this mess | 1733 |
| org.kuali.rice.kew.docsearch.dao.impl.DocumentSearchDAOJdbcImpl | Line |
| delyea - look at refactoring | 95 |
| delyea: use searchable attribute count here? | 157 |
| org.kuali.rice.kew.docsearch.dao.impl.DocumentSearchDAOOjbImpl | Line |
| delyea - look at refactoring | 95 |
| delyea: use searchable attribute count here? | 166 |
| org.kuali.rice.kew.docsearch.xml.StandardGenericXMLSearchableAttribute | Line |
| after face-to-face work in december 2008, this was throwing a nullpointerexception for lookups with date pickers assuming this code will go away after the document search conversion | 524 |
| org.kuali.rice.kew.docsearch.xml.StandardGenericXMLSearchableAttributeTest | Line |
| put document content here? | 1019 |
| this is currently commented out because we can't test this properly through the unit test since the filtering of the column actually happens in the web-tier. Shoudl this be the case? Maybe we need to re-examine when we refactor document search. | 1214 |
| org.kuali.rice.kew.doctype.ApplicationDocumentStatusId | Line |
| Finish implementing this method. Compare o to pk and return true or false. | 94 |
| Implement this method | 99 |
| org.kuali.rice.kew.doctype.DocumentTypePermissionServiceTest | Line |
| set up actual KIM permissions in DB and verify this permission works | 54 |
| org.kuali.rice.kew.doctype.DocumentTypePolicyId | Line |
| Implement the equals() and hashCode() methods. | 27 |
| Finish implementing this method. Compare o to pk and return true or false. | 98 |
| Implement this method | 103 |
| org.kuali.rice.kew.doctype.service.impl.DocumentTypePermissionServiceImpl | Line |
| See KULRICE-3490, make assembly of permission details dynamic based on db config | 371 |
| org.kuali.rice.kew.document.DocumentTypeMaintainable | Line |
| delyea - do we need to call this multiple times as in "DocumentTypeServiceImpl.save(DocumentType, boolean)" | 149 |
| org.kuali.rice.kew.document.RoutingRuleMaintainable | Line |
| can we just let the locking key be the primary key? (ruleBaseValuesId) | 140 |
| org.kuali.rice.kew.document.RoutingRuleMaintainableBusRule | Line |
| hook validation of rule data into PreRules | 266 |
| org.kuali.rice.kew.dto.DTOConverter | Line |
| delyea - should below be using actionRequest.getRoleName()? | 510 |
| should this be private? If so, rename to convertActionRequestDTO for consistency. | 658 |
| org.kuali.rice.kew.engine.BlanketApproveEngine | Line |
| document magical join node workage (ask Eric) | 111 |
| this has been set arbitrarily high because the implemented processing model here will probably not work for large parallel object graphs. This needs to be re-evaluated, see KULWF-459. | 112 |
| org.kuali.rice.kew.engine.CompatUtils | Line |
| potential for infinite recursion here if their document type has loops in it. Should this be a concern? If their routing version is really "route level" then there should be no cycles. | 55 |
| org.kuali.rice.kew.engine.SequentialRoutingTest | Line |
| when we are able to, we should also verify the RouteNodeInstances are correct | 123 |
| org.kuali.rice.kew.engine.StandardWorkflowEngine | Line |
| throw a new 'RoutingException' which holds the RoutingState | 148 |
| update document content on context? | 227 |
| more info in message | 265 |
| extract this into some sort of component which handles transitioning document state | 386 |
| is the logic for going processed still going to be valid? | 404 |
| perhaps the policies could also be factored out? | 409 |
| factor out this magical post processing | 412 |
| what route method name to pass to this? | 486 |
| in some cases, someone may modify the route header in the post processor, if we don't save before and reload after we will get an optimistic lock exception, we need to work on a better solution for this! | 571 |
| get the routeContext in this method - it should be a better object than the nodeInstance | 574 |
| get the routeContext in this method - it should be a better object than the nodeInstance | 602 |
| get the routeContext in this method - it should be a better object than the nodeInstance | 610 |
| this shows up in a few places and could totally be extracted to a method | 638 |
| get the routeContext in this method - it should be a better object than the nodeInstance | 661 |
| get the routeContext in this method - it should be a better object than the nodeInstance | 673 |
| org.kuali.rice.kew.engine.simulation.SimulationEngine | Line |
| set document content | 133 |
| delyea - deep copy below | 490 |
| org.kuali.rice.kew.engine.transition.DynamicTransitionEngine | Line |
| interate the all the nodes and attach the dynamic node as the 'process' don't include the dynamic node instance in the routing structure - require a correctly built graph change dynamic node signiture to next node because of above reconcile branching if necessary | 39 |
| implement this feature throw new UnsupportedOperationException("Need to implement support for branch generation!"); | 108 |
| org.kuali.rice.kew.help.web.HelpAction | Line |
| hook up KIM permissions to this | 76 |
| hook up KIM permissions to this | 97 |
| hook up KIM permissions to this | 109 |
| implement the help search as a lookupable, rendering this code redundant | 153 |
| org.kuali.rice.kew.messaging.exceptionhandling.ExceptionRoutingServiceImpl | Line |
| is there a reason we reload the document here? | 197 |
| org.kuali.rice.kew.notes.web.AttachmentServlet | Line |
| This should probably be put into KEWConstants when contributed back to Rice 1.0.3 | 54 |
| org.kuali.rice.kew.plugin.HotDeployer | Line |
| somehow the code which checks for new plugins and which loads plugins initially needs to be consolidated, maybe with some sort of set of PluginLocators? or something along those lines? | 101 |
| org.kuali.rice.kew.plugin.PluginRegistryFactory | Line |
| maybe ensure that if these directories are the same, that only one gets through | 39 |
| org.kuali.rice.kew.plugin.ServerPluginRegistry | Line |
| make the delay configurable | 71 |
| org.kuali.rice.kew.plugin.ZipFilePluginLoader | Line |
| how to handle locked files under windows?!? This will throw an IOException in this case. | 119 |
| org.kuali.rice.kew.postprocessor.DocumentRouteLevelChange | Line |
| for now we will include the new node-based routing fields onto this object to avoid an interface change to the PostProcessor interface. | 29 |
| org.kuali.rice.kew.quicklinks.dao.impl.QuickLinksDAOOjbImpl | Line |
| Should this happen??? | 137 |
| org.kuali.rice.kew.responsibility.dao.ResponsibilityIdDAO | Line |
| To change the template for this generated type comment go to Window - Preferences - Java - Code Style - Code Templates | 23 |
| org.kuali.rice.kew.responsibility.service.impl.ResponsibilityIdServiceImpl | Line |
| To change the template for this generated type comment go to Window - Preferences - Java - Code Style - Code Templates | 26 |
| org.kuali.rice.kew.routeheader.AppDocStatusTest | Line |
| when we are able to, we should also verify the RouteNodeInstances are correct | 130 |
| when we are able to, we should also verify the RouteNodeInstances are correct | 211 |
| org.kuali.rice.kew.routeheader.DocumentRouteHeaderValue | Line |
| the isRouteLevelDocument junk can be ripped out | 267 |
| org.kuali.rice.kew.routeheader.StandardDocumentContent | Line |
| can we have application content without a root node? | 117 |
| org.kuali.rice.kew.routeheader.dao.impl.DocumentRouteHeaderDAOJpaImpl | Line |
| What cache do we clear when using JPA | 150 |
| What cache do we clear when using JPA | 171 |
| org.kuali.rice.kew.routeheader.service.impl.WorkflowDocumentServiceImpl | Line |
| As with superUserReturnDocumentToPreviousNode, we allow for the passing in of a document ID here to allow for the document load inside the current running transaction. Otherwise we get an optimistic lock exception when attempting to save the branch after the transition to the 'A' status. | 348 |
| As with superUserReturnDocumentToPreviousNode, we allow for the passing in of a document ID here to allow for the document load inside the current running transaction. Otherwise we get an optimistic lock exception when attempting to save the branch after the transition to the 'A' status. | 393 |
| remove this implementation in favor of having the SuperUserAction call through the WorkflowDocument object. This method is here to resolve KULWF-727 where we were getting an optimistic lock exception from the super user screen on return to previous node. This allows us to load the DocumentRouteHeaderValue inside of the transaction interceptor so that we can stay within the same PersistenceBroker cache. | 402 |
| org.kuali.rice.kew.routemodule.RouteModuleRemoteAdapter | Line |
| add new findActionRequests to RouteModuleRemote | 54 |
| this should be moved to a validate somewhere's... | 70 |
| org.kuali.rice.kew.routing.web.ClientAppDocHandlerRedirectAction | Line |
| what should happen here if parms are missing; no proper ActionForward from here | 84 |
| org.kuali.rice.kew.rule.AbstractWorkflowAttribute | Line |
| document these two methods ... what are these for? | 66 |
| org.kuali.rice.kew.rule.FlexRM | Line |
| really the route context just needs to be able to support nested create and clears (i.e. a Stack model similar to transaction intercepting in Spring) and we wouldn't have to do this | 172 |
| org.kuali.rice.kew.rule.GenericWorkflowAttribute | Line |
| implement me! | 139 |
| this is not fully implemented! e.g. generic property map like StandardGenericXmlAttribute | 155 |
| org.kuali.rice.kew.rule.TemplateRuleSelector | Line |
| really the route context just needs to be able to support nested create and clears (i.e. a Stack model similar to transaction intercepting in Spring) and we wouldn't have to do this | 84 |
| org.kuali.rice.kew.rule.bo.RuleBaseValuesLookupableHelperServiceImpl | Line |
| jjhanso - THIS METHOD NEEDS JAVADOCS return super.performLookup(lookupForm, resultTable, bounded); | 453 |
| org.kuali.rice.kew.rule.bo.RuleDelegationLookupableHelperServiceImpl | Line |
| jjhanso - THIS METHOD NEEDS JAVADOCS return super.performLookup(lookupForm, resultTable, bounded); | 423 |
| org.kuali.rice.kew.rule.dao.impl.RuleDAOJpaImpl | Line |
| clear the cache | 450 |
| org.kuali.rice.kew.rule.dao.impl.RuleDelegationDAOJpaImpl | Line |
| jjhanso - THIS METHOD NEEDS JAVADOCS | 91 |
| jjhanso - THIS METHOD NEEDS JAVADOCS | 104 |
| org.kuali.rice.kew.rule.service.impl.RuleServiceImpl | Line |
| if more than one delegate is edited from the create delegation screen (which currently can not happen), then this logic will not work. | 192 |
| consolidate this method with makeCurrent. The reason there's a seperate implementation is because the original makeCurrent(...) could not properly handle versioning a List of multiple rules (including multiple delegates rules for a single parent. ALso, this work is being done for a patch so we want to mitigate the impact on the existing rule code. | 246 |
| this method ended up being a mess, we should get rid of this as soon as we can | 1124 |
| FIXME: need to copy the rule expression here too? | 1325 |
| this method is extremely slow, if we could implement a more optimized query here, that would help tremendously | 1422 |
| org.kuali.rice.kew.rule.xmlrouting.StandardGenericXMLRuleAttribute | Line |
| possibly simplify this even further by making WorkflowAttributeValidationError a WorkflowServiceError, and dispense with custom error generators... | 616 |
| org.kuali.rice.kew.server.BeanConverterTester | Line |
| there is some crossover between this test and the DocumentContentTest, do we really need both of them??? | 106 |
| org.kuali.rice.kew.server.DTOConverterTest | Line |
| there is some crossover between this test and the DocumentContentTest, do we really need both of them??? | 104 |
| org.kuali.rice.kew.service.WorkflowDocument | Line |
| delyea - refactor this to take into account non-initiator owned documents | 722 |
| org.kuali.rice.kew.service.impl.WorkflowUtilityWebServiceImpl | Line |
| confirm that the initiator is not already there in the actionstaken | 516 |
| org.kuali.rice.kew.support.xstream.XStreamSafeEvaluatorTest | Line |
| we currently can't support xpath expressions that start with .// When we can, uncomment this test code. nodeList = (NodeList)xpath.evaluate(XPATH_GLOBAL_VALUE, node, XPathConstants.NODESET); assertEquals("There should be two value elements.", 2, nodeList.getLength()); // evaluate as xstreamsafe nodeList = (NodeList)xpath.evaluate(wrapXStreamSafe(XPATH_GLOBAL_VALUE), node, XPathConstants.NODESET); assertEquals("There should be three value elements now.", 3, nodeList.getLength()); | 174 |
| org.kuali.rice.kew.test.FakeService | Line |
| consider merging with {@link TestServiceInterface}/{@link GenericTestService} in KSB module | 24 |
| org.kuali.rice.kew.test.FakeServiceImpl | Line |
| consider merging with {@link TestServiceInterface}/{@link GenericTestService} in KSB module | 26 |
| org.kuali.rice.kew.useroptions.UserOptionsId | Line |
| Implement the equals() and hashCode() methods. | 26 |
| org.kuali.rice.kew.util.ConfigStringLookup | Line |
| temporarily disabling configuration parameter resolution against the racms because it's been causing some issues | 51 |
| org.kuali.rice.kew.webservice.impl.SimpleDocumentActionsWebServiceImpl | Line |
| how do we want to handle a "Complete" request? | 243 |
| how do we want to handle a "Complete" request? | 361 |
| is this necessary? | 719 |
| do we need to return the standard result set? results = createResults(workflowDocument); | 723 |
| is this necessary? | 776 |
| do we need to return the standard result set? results = createResults(workflowDocument); | 780 |
| is this necessary? | 836 |
| org.kuali.rice.kew.xml.ClassLoaderEntityResolver | Line |
| maybe prefix should be changed from "resource:" to "internal:" or just "workflow:" given that it can be resolved in arbitrary ways other than ClassLoader "resources" | 31 |
| revisit making this more sophisticated than just the classloader of this class (thread context classloader? plugin classloader?) | 72 |
| org.kuali.rice.kew.xml.DocumentTypeXmlParser | Line |
| have a validation function that takes an xpath statement and blows chunks if that statement returns false | 346 |
| org.kuali.rice.kew.xml.GroupXmlParser | Line |
| HACK!!!!!!! Use IDMService.addPrincipalToGroup | 309 |
| HACK!!!!!!! Use IDMService.addPrincipalToGroup | 326 |
| HACK!!!!!!! Use IDMService.addPrincipalToGroup | 342 |
| org.kuali.rice.kew.xml.RuleExtensionXmlParser | Line |
| need test case for this | 97 |
| org.kuali.rice.kew.xml.export.GroupXmlExporter | Line |
| jjhanso - THIS METHOD NEEDS JAVADOCS | 64 |
| org.kuali.rice.kew.xml.xstream.XStreamSafeEvaluator | Line |
| I noticed that the original implementation of this method was not doing the following work so I'm just tacking it on the end, there's probably a more elegent way to integrate it with the algorithm above... | 97 |
| we currently can't support expressions that start with .// | 151 |
| org.kuali.rice.kim.api.common.assignee.Assignee | Line |
| Need modelized DelegateInfo | 62 |
| org.kuali.rice.kim.api.common.assignee.AssigneeContract | Line |
| Get a better description | 22 |
| org.kuali.rice.kim.api.entity.Type | Line |
| if create() is modified to accept required parameters, this will need to be modified | 146 |
| org.kuali.rice.kim.api.entity.personal.EntityBioDemographics | Line |
| modify as needed to pass any required values and add them to the signature of the 'create' method | 322 |
| org.kuali.rice.kim.api.entity.phone.EntityPhone | Line |
| get extension from country code table | 365 |
| append "+xxx" if country is not the default country | 366 |
| org.kuali.rice.kim.api.entity.principal.Principal | Line |
| modify as needed to pass any required values and add them to the signature of the 'create' method | 151 |
| if create() is modified to accept required parameters, this will need to be modified | 159 |
| org.kuali.rice.kim.api.entity.privacy.EntityPrivacyPreferences | Line |
| add validation of input value if required and throw IllegalArgumentException if needed | 231 |
| add validation of input value if required and throw IllegalArgumentException if needed | 236 |
| add validation of input value if required and throw IllegalArgumentException if needed | 241 |
| add validation of input value if required and throw IllegalArgumentException if needed | 246 |
| add validation of input value if required and throw IllegalArgumentException if needed | 251 |
| add validation of input value if required and throw IllegalArgumentException if needed | 256 |
| add validation of input value if required and throw IllegalArgumentException if needed | 261 |
| org.kuali.rice.kim.api.test.JAXBAssert | Line |
| - copied this from the core api test module, find a better way to share this code! | 33 |
| org.kuali.rice.kim.bo.role.dto.KimPermissionInfo | Line |
| remove this and find a better way to do this. Should be done by next week with role doc task | 129 |
| org.kuali.rice.kim.bo.ui.GroupDocumentMember | Line |
| remove the default | 56 |
| org.kuali.rice.kim.bo.ui.KimDocumentRoleMember | Line |
| remove the default | 69 |
| org.kuali.rice.kim.bo.ui.KimDocumentRoleResponsibility | Line |
| this needs to be changed to use the KimResponsibilityInfo object but the changes are involved in the UiDocumentService based on the copyProperties method used to move the data to/from the document/real objects | 84 |
| org.kuali.rice.kim.bo.ui.KimDocumentRoleResponsibilityAction | Line |
| this needs to be changed to use the KimResponsibilityInfo object but the changes are involved in the UiDocumentService based on the copyProperties method used to move the data to/from the document/real objects | 85 |
| this needs to be changed to use the KimResponsibilityInfo object but the changes are involved in the UiDocumentService based on the copyProperties method used to move the data to/from the document/real objects | 161 |
| org.kuali.rice.kim.bo.ui.PersonDocumentPhone | Line |
| get extension from country code table | 170 |
| append "+xxx" if country is not the default country | 171 |
| org.kuali.rice.kim.client.acegi.KualiDistributedSessionExpiredAuthentication | Line |
| lsymms - THIS METHOD NEEDS JAVADOCS | 40 |
| lsymms - THIS METHOD NEEDS JAVADOCS | 50 |
| org.kuali.rice.kim.client.acegi.KualiDistributedSessionFilter | Line |
| Need to add check for missing DST (update {@link org.kuali.rice.kim.sesn.DistributedSession}) | 32 |
| org.kuali.rice.kim.dao.impl.KimRoleDaoOjb | Line |
| if no role id found that means principalname not matched, need to do something to force to return empty list | 421 |
| org.kuali.rice.kim.document.rule.IdentityManagementGroupDocumentRule | Line |
| do not have detail bus rule yet, so just check this for now. | 149 |
| org.kuali.rice.kim.document.rule.IdentityManagementPersonDocumentRule | Line |
| do not have detail bus rule yet, so just check this for now. | 555 |
| org.kuali.rice.kim.document.rule.IdentityManagementRoleDocumentRule | Line |
| do not have detail bus rule yet, so just check this for now. | 586 |
| org.kuali.rice.kim.impl.group.GroupServiceBase | Line |
| This is a prime candidate for the new criteria builder CriteriaBuilder<GroupBo> criteriaBuilder = CriteriaBuilder.newCriteriaBuilder(GroupBo.class); criteriaBuilder.in("id", new ArrayList(groupIds)); QueryByCriteria.Builder<GroupBo> qbcBuilder = QueryByCriteria.Builder.create(GroupBo.class); qbcBuilder.set | 77 |
| hopefully there is an efficient orm way to do this | 85 |
| org.kuali.rice.kim.impl.group.GroupServiceImpl | Line |
| PRIME example of something for new Criteria API | 286 |
| org.kuali.rice.kim.impl.responsibility.ResponsibilityInquirableImpl | Line |
| remove this | 168 |
| org.kuali.rice.kim.impl.responsibility.ResponsibilityLookupableHelperServiceImpl | Line |
| why is this not using the businessObject parmeter's class? | 91 |
| org.kuali.rice.kim.impl.responsibility.ResponsibilityServiceImpl | Line |
| filter with qualifiers | 295 |
| filter with qualifiers | 408 |
| org.kuali.rice.kim.inquiry.PermissionInquirableImpl | Line |
| remove this | 172 |
| org.kuali.rice.kim.lookup.KimLookupableHelperServiceImpl | Line |
| to let it rendering 'create new' and 'edit'/'copy' button | 43 |
| org.kuali.rice.kim.lookup.KimTypeLookupableHelperServiceImpl | Line |
| clean this up. | 84 |
| org.kuali.rice.kim.service.impl.IdentityManagementServiceImpl | Line |
| cache this? | 822 |
| org.kuali.rice.kim.service.impl.IdentityServiceImpl | Line |
| We need to try and remove this. Currently, without it, some integration tests fail because of some sort of OJB caching and not filling in the type values. We need to figure out why this is happening and fix it. Yes, this is a hack :P | 289 |
| - Use a DAO | 403 |
| - use a dao? | 423 |
| - what if principal not found, NullPointerException | 424 |
| org.kuali.rice.kim.service.impl.PermissionServiceImpl | Line |
| This could be made a little better by collecting the role IDs into a set and then processing the distinct list rather than a check for every permission | 221 |
| org.kuali.rice.kim.service.impl.PersonServiceImpl | Line |
| - now check if these people have the given role build a principal list | 333 |
| org.kuali.rice.kim.service.impl.RoleServiceBase | Line |
| pulling attribute IDs repeatedly is inefficient - consider caching the entire list as a map | 1234 |
| org.kuali.rice.kim.service.impl.RoleServiceImpl | Line |
| ? get groups for principal and get those as well? this implementation may be incomplete, as groups and sub-roles are not considered | 279 |
| org.kuali.rice.kim.service.impl.UiDocumentServiceImpl | Line |
| in order to resolve optimistic locking issue. has to get entity and set the version number if entity records matched Need to look into this. kimEntity = origEntity; | 174 |
| - remove this hack... This is here because currently jpa only seems to be going 2 levels deep on the eager fetching. | 662 |
| this should be replaced with the retrieval and storage of that value in the document tables and not re-retrieved here | 1266 |
| principalId is not ready here yet ? | 1316 |
| verify if you want to add && newRoleMember.isActive() condition to if... | 1388 |
| verify if you want to add && newRoleMember.isActive() condition to if... | 2208 |
| verify if you want to add && newRoleMember.isActive() condition to if... | 2384 |
| implement logic same as role members - do not insert qualifiers with blank values | 2440 |
| when a group is inactivated, inactivate the memberships of principals in that group and the memberships of that group in roles | 2592 |
| verify if you want to add && newGroupMember.isActive() condition to if... | 2669 |
| org.kuali.rice.kim.service.support.KimPermissionTypeService | Line |
| clarify this description | 42 |
| org.kuali.rice.kim.service.support.impl.KimTypeServiceBase | Line |
| will this work for user ID attributes? | 290 |
| Formatter for currency? | 484 |
| Formatter for currency? | 499 |
| this method looks like it could use some refactoring | 620 |
| use id for defnid ? definition.setId(typeAttribute.getKimAttributeId()); | 772 |
| org.kuali.rice.kim.sesn.DistributedSession | Line |
| add clear principals to clearSesn | 33 |
| org.kuali.rice.kim.sesn.timeouthandlers.AbstractTimeoutHandler | Line |
| change how TimoutHandlers choose fields from ndx to name. | 20 |
| org.kuali.rice.kim.sesn.timeouthandlers.DisabledTimeoutHandler | Line |
| lsymms - THIS METHOD NEEDS JAVADOCS | 33 |
| org.kuali.rice.kim.test.service.GroupServiceTest | Line |
| reenable this once update service updated | 89 |
| org.kuali.rice.kim.test.service.PermissionServiceTest | Line |
| - getting a SOAPFaultException on this call; fix and un-comment assertFalse(getPermissionService().hasPermissionByTemplateName("entity124pId", "KUALI", "Default", new AttributeSet())); | 68 |
| - jax-ws remoted service returns null; local return empty List. Fix webservice return | 86 |
| org.kuali.rice.kim.test.service.PersonServiceImplTest | Line |
| property is being appended in the wrong place - fix | 204 |
| org.kuali.rice.kim.web.struts.action.IdentityManagementPersonDocumentAction | Line |
| how should this forward be handled | 609 |
| org.kuali.rice.kim.web.struts.action.IdentityManagementRoleDocumentAction | Line |
| move this into the UI service - action should not be making ORM-layer calls | 123 |
| org.kuali.rice.kns.bo.AdHocRoutePersonId | Line |
| Implement the equals() and hashCode() methods. | 29 |
| org.kuali.rice.kns.bo.AdHocRouteRecipient | Line |
| we should not be referencing kew constants from this class and wedding ourselves to that workflow application Ad Hoc Route Recipient Business Object | 30 |
| org.kuali.rice.kns.bo.AdHocRouteWorkgroupId | Line |
| Implement the equals() and hashCode() methods. | 29 |
| org.kuali.rice.kns.dao.impl.LookupDaoJpa | Line |
| What to do here now that the JPA version does not extend platform aware? propertyName = getDbPlatform().getUpperCaseFunction() + "(__JPA_ALIAS[[0]]__." + propertyName + ")"; | 417 |
| org.kuali.rice.kns.dao.impl.NoteDaoJpa | Line |
| Notes - Chris move remoteObjectId string to constants | 72 |
| org.kuali.rice.kns.dao.impl.NoteDaoOjb | Line |
| Notes - Chris move remoteObjectId string to constants | 85 |
| Notes - Chris move remoteObjectId string to constants | 108 |
| org.kuali.rice.kns.datadictionary.AttributeDefinition | Line |
| rename to control once ControlDefinition is removed | 75 |
| This may not be required since we now use ComplexAttributeDefinition | 94 |
| currently requiring a control or controlField, but this should not be case (AttrField should probably do the check) | 315 |
| org.kuali.rice.kns.datadictionary.DataDictionary | Line |
| determine property type using generics type definition | 479 |
| org.kuali.rice.kns.datadictionary.DataDictionaryIndexMapper | Line |
| arh14 - THIS METHOD NEEDS JAVADOCS | 213 |
| org.kuali.rice.kns.datadictionary.DocumentCollectionPath | Line |
| mpham - THIS METHOD NEEDS JAVADOCS | 65 |
| org.kuali.rice.kns.datadictionary.DocumentValuePathGroup | Line |
| mpham - THIS METHOD NEEDS JAVADOCS | 65 |
| org.kuali.rice.kns.datadictionary.LookupDefinition | Line |
| catch exception if service locator call fails | 157 |
| org.kuali.rice.kns.datadictionary.RoutingTypeDefinition | Line |
| wliang - THIS METHOD NEEDS JAVADOCS | 65 |
| org.kuali.rice.kns.datadictionary.exporter.AttributesMapBuilder | Line |
| consider whether to export class names from the attribute security | 95 |
| org.kuali.rice.kns.datadictionary.mask.MaskFormatterSubString | Line |
| MOVE TO UNIT TEST: move this validation into the unit tests | 38 |
| org.kuali.rice.kns.datadictionary.validation.ValidationPattern | Line |
| UNIT TEST: compile all patterns to test | 91 |
| org.kuali.rice.kns.datadictionary.validation.constraint.CharsetPatternConstraint | Line |
| delyea: add in list of valid characters here? StringBuilder validCharsBuf = new StringBuilder(); for (int i = 0; i < getValidChars().length(); i++) { validCharsBuf.append(getValidChars().charAt(i)); if (i != getValidChars().length() - 1) { validCharsBuf.append(", "); } } | 85 |
| org.kuali.rice.kns.datadictionary.validation.constraint.ValidCharactersConstraint | Line |
| delyea: remove jsValue from docs here Note that if jsValue has any value it will always <b>override</b> the following...<b> If the labelKey matches one of the following values:<br> email<br> | 15 |
| org.kuali.rice.kns.datadictionary.validation.processor.BasePrerequisiteConstraintProcessor | Line |
| Does this code need to be able to look at more than just the other immediate members of the object? | 45 |
| org.kuali.rice.kns.datadictionary.validation.processor.CaseConstraintProcessor | Line |
| What happens when the field is not in the dataProvider? | 60 |
| org.kuali.rice.kns.datadictionary.validation.processor.RangeConstraintProcessor | Line |
| decide if there is any reason why the following would be insufficient - i.e. if something numeric could still be cast to String at this point | 82 |
| JLR - need a code review of the conversions below to make sure this is the best way to ensure accuracy across all numerics This will throw NumberFormatException if the value is 'NaN' or infinity... probably shouldn't be a NFE but something more intelligible at a higher level | 106 |
| org.kuali.rice.kns.document.DocumentAuthorizerBaseTest | Line |
| rewrite this test private static final Logger LOG = Logger.getLogger(DocumentAuthorizerBaseTest.class); | 24 |
| the test for preRouteEditAuthorization should be removed GlobalVariables.setUserSession(new UserSession("quickstart")); PessimisticLockTestDocumentAuthorizer lockDocumentAuthorizer = new PessimisticLockTestDocumentAuthorizer(); lockDocumentAuthorizer.USES_PESSIMISTIC_LOCKING = false; | 522 |
| sp20369 - THIS METHOD NEEDS JAVADOCS | 698 |
| org.kuali.rice.kns.document.DocumentBase | Line |
| Auto-generated method stub | 129 |
| Auto-generated method stub | 466 |
| better error handling at the lower level and a better error message are needed here | 493 |
| org.kuali.rice.kns.document.MaintenanceDocumentBase | Line |
| - build out with bo label once we get the data dictionary stuff in place build out the right classname | 181 |
| refactor this out into a method | 384 |
| better error handling at the lower level and a better error message are needed here | 743 |
| org.kuali.rice.kns.document.authorization.MaintenanceDocumentRestrictionsBase | Line |
| next block could probably be removed since the superclass would return null for a read-only field | 53 |
| org.kuali.rice.kns.inquiry.KualiInquirableImpl | Line |
| generics do not match between call to module service and call to lookup service | 109 |
| If this is to get a single BO, why using the lookup service? | 137 |
| replace with inquiry menu bar | 570 |
| org.kuali.rice.kns.lookup.AbstractLookupableHelperServiceImpl | Line |
| why is this not using the businessObject parmeter's class? | 493 |
| org.kuali.rice.kns.lookup.KualiLookupableHelperServiceImpl | Line |
| Fix? - this does not handle nested properties within the EBO. | 75 |
| org.kuali.rice.kns.lookup.LookupResultsServiceImpl | Line |
| use the other identifier | 164 |
| use the other identifier | 185 |
| org.kuali.rice.kns.lookup.LookupUtils | Line |
| chb: that should not be the case -- the relationship object the two rely upon should be established outside of the lookup/quickfinder code | 474 |
| org.kuali.rice.kns.maintenance.KualiGlobalMaintainableImpl | Line |
| this needs refactoring ... its kind of lame that we have this set of compound list statements, this should all be refactored. This could be moved into a method on all GBOs, like GBO.prepareForSave(), or even better, subclass KualiGlobalMaintainableImpl for each global, since this is all maintainable-related stuff. | 63 |
| remove this whole pseudo-assertion code block once this gets moved into a doc-specific maintainableImpl class. | 73 |
| Revisit this. Changing since getPrimaryKeys and listPrimaryKeyFieldNames are apparently same. May be we might want to replace listPrimaryKeyFieldNames with getPrimaryKeys... Not sure. | 79 |
| org.kuali.rice.kns.maintenance.KualiMaintainableImpl | Line |
| rename once 'newCollectionLines' is removed | 108 |
| should we be using ObjectUtils? also, this needs dictionary enhancement to indicate fields to/not to copy | 1623 |
| the logic in this method is very odd, but I don't know how clients are extending this impl. It seems like clients made non persistent objects extend PBO in order to not break the framework which requires PBO everywhere, KRAD is fixing that. | 1718 |
| make sure we are doing formatting here eventually | 1897 |
| should this write some sort of missing message on the old side instead? | 1905 |
| org.kuali.rice.kns.maintenance.Maintainable | Line |
| remove from interface once converted to KRAD | 409 |
| org.kuali.rice.kns.maintenance.rules.MaintenanceDocumentRuleBase | Line |
| Good suggestion from Aaron, dont bother checking the DB, if all of the objects PK fields dont have values. If any are null or empty, then we're done. The current way wont fail, but it will make a wasteful DB call that may not be necessary, and we want to minimize these. | 745 |
| should this be getting labels from the view dictionary use the DataDictionary service to translate field name into human-readable label | 781 |
| org.kuali.rice.kns.service.BusinessObjectAuthorizationService | Line |
| refactor for general objects | 31 |
| org.kuali.rice.kns.service.KualiRuleServiceTest | Line |
| commenting thse out for now, need to analyze and see if it still makes sense with new notes"copied from document " + // sourceDocumentHeaderId); | 391 |
| org.kuali.rice.kns.service.impl.BusinessObjectAuthorizationServiceImpl | Line |
| what is the equivalent of control.isButton in KRAD | 315 |
| this requires intimate knowledge of DocumentHelperServiceImpl | 537 |
| org.kuali.rice.kns.service.impl.BusinessObjectDictionaryServiceImpl | Line |
| stupid, change to return int | 323 |
| org.kuali.rice.kns.service.impl.BusinessObjectMetaDataServiceImpl | Line |
| four different exit points?! | 190 |
| move out to a separate method so that the logic for finding the relationships is similar to primitiveReference | 278 |
| org.kuali.rice.kns.service.impl.DictionaryValidationServiceImpl | Line |
| reenable when we come up with a strategy to handle fields that are not editable else if (itemDefinition instanceof MaintainableCollectionDefinition) { MaintainableCollectionDefinition collectionDefinition = (MaintainableCollectionDefinition) itemDefinition; Collection<BusinessObject> c = (Collection<BusinessObject>) ObjectUtils.getPropertyValue(businessObject, itemDefinition.getName()); if (c != null) { int i = 0; for (BusinessObject o : c) { String newErrorPrefix = errorPrefix + itemDefinition.getName() + "[" + i + "]."; validateBusinessObjectOnMaintenanceDocumentHelper(o, collectionDefinition.getMaintainableCollections(), newErrorPrefix); validateBusinessObjectOnMaintenanceDocumentHelper(o, collectionDefinition.getMaintainableFields(), newErrorPrefix); i++; } } }*/ } } | 356 |
| Formatter for currency? | 549 |
| Formatter for currency? | 565 |
| Implement custom validators | 1031 |
| Do we need to be able to handle simple collections (ie. String, etc) | 1186 |
| org.kuali.rice.kns.service.impl.DocumentServiceImpl | Line |
| gah! this is awful | 820 |
| org.kuali.rice.kns.service.impl.KualiModuleServiceImpl | Line |
| Warning! Kludge! Hack! Will be replaced! KULRICE-2921 | 218 |
| org.kuali.rice.kns.service.impl.NoteServiceImpl | Line |
| this method seems awfully out of place in this service | 104 |
| Why is a deep copy being done? Nowhere that this is called uses the given note argument again after calling this method. | 116 |
| org.kuali.rice.kns.service.impl.PersistenceStructureServiceJpaImpl | Line |
| Write JPA Version | 109 |
| Write JPA Version | 140 |
| org.kuali.rice.kns.service.impl.RiceApplicationConfigurationMediationServiceImpl | Line |
| I think the code that's below here will actually pull in more than one reference to a particular application's config service if it is deployed in a cluster, it needs to only pull a single RiceApplicationConfigurationService implementation per application id. Also, may want to consider load balancing and failover in those cases? It might be best to try and utilize the client-side KSB proxies that handle a lot of this stuff for us | 100 |
| Need a better way to catch if service is not active (404 error) | 126 |
| org.kuali.rice.kns.uif.container.CollectionGroup | Line |
| is this necessary? | 260 |
| org.kuali.rice.kns.uif.container.DocumentView | Line |
| figure out what this is used for | 41 |
| org.kuali.rice.kns.uif.container.LookupView | Line |
| delyea: Fill in javadocs here | 34 |
| org.kuali.rice.kns.uif.container.TreeGroup | Line |
| set object path for prototypes equal to the tree group object path? | 68 |
| org.kuali.rice.kns.uif.container.View | Line |
| fix this generic issue | 827 |
| org.kuali.rice.kns.uif.core.Component | Line |
| javadoc | 409 |
| this should not be on component interface since it only applies if the layout manager supports it, need some sort of layoutOptions map for field level options that depend on the manager | 426 |
| this should not be on component interface since it only applies if the layout manager supports it, need some sort of layoutOptions map for field level options that depend on the manager | 451 |
| org.kuali.rice.kns.uif.core.ComponentBase | Line |
| handle all components not only AttributeField | 236 |
| org.kuali.rice.kns.uif.core.MaintenanceActiveCollectionFilter | Line |
| if newly added line, show record | 70 |
| org.kuali.rice.kns.uif.field.ActionField | Line |
| possibly fix some other way - this is a workaround, prevents showing history and showing home again on actions which submit the form | 155 |
| org.kuali.rice.kns.uif.field.AttributeField | Line |
| remove later, this should be done within the service lifecycle | 194 |
| org.kuali.rice.kns.uif.field.ErrorsField | Line |
| this may be changed to: if this is set on a field it will attempt show client side validation errors in the closest parent container error container | 395 |
| org.kuali.rice.kns.uif.field.LookupCriteriaAttributeField | Line |
| delyea: this used to take into account if the class was Inactivateable: Inactivateable.class.isAssignableFrom(businessObjectClass) | 78 |
| delyea: check to see if the propertyName needs to be checked for instances where getPropertyName() returns "bo.active" | 80 |
| delyea: FieldUtils.createAndPopulateFieldsForLookup used to allow for a set of property names to be passed in via the URL parameters of the lookup url to set fields as 'read only' | 87 |
| org.kuali.rice.kns.uif.history.HistoryEntry | Line |
| a variety of these settings are not used in the current implementation of breadcrumbs and history, they may be removed later if they prove unuseful in future changes | 23 |
| unsure of use | 136 |
| org.kuali.rice.kns.uif.layout.TableLayoutManager | Line |
| implement repeat header | 164 |
| remove once we have modifier | 181 |
| org.kuali.rice.kns.uif.modifier.CompareFieldCreateModifier | Line |
| add script for value changed? | 198 |
| org.kuali.rice.kns.uif.service.LookupViewHelperService | Line |
| delyea - should be on custom Controller? public boolean performCustomAction(boolean ignoreErrors); | 172 |
| org.kuali.rice.kns.uif.service.ViewService | Line |
| remove once can get beans by type | 122 |
| org.kuali.rice.kns.uif.service.impl.AttributeQueryServiceImpl | Line |
| need to apply formatter for field or have method in object property utils | 84 |
| org.kuali.rice.kns.uif.service.impl.LookupViewHelperServiceImpl | Line |
| delyea - investigate if encryptionService is even needed due to new spring binding | 91 |
| delyea: where to take into account the sort ascending value (old KNS appeared to ignore?) | 112 |
| delyea: switch the bounded flag to be unbounded to match underlying method calls in getSearchResultsWithBounding() | 297 |
| delyea - is this the best way to set that the entire set has a returnable row? | 300 |
| delyea - DOCUMENT THE FOLLOWING CHANGES (next 5 lines) props.put(KNSConstants.DISPATCH_REQUEST_PARAMETER, KNSConstants.MAINTENANCE_NEW_METHOD_TO_CALL); props.put(KNSConstants.BUSINESS_OBJECT_CLASS_ATTRIBUTE, getDataObjectClass().getName()); | 685 |
| delyea - DOCUMENT THE FOLLOWING CHANGES (next 2 lines) url = UrlFactory.parameterizeUrl(KNSConstants.MAINTENANCE_ACTION, props); | 692 |
| delyea - DOCUMENT THE FOLLOWING CHANGES FROM KNSConstants to UifConstants | 763 |
| delyea - sgibson just hacked in the link text, probably should come from somewhere | 764 |
| delyea - DOCUMENT THE FOLLOWING CHANGES (next 3 lines) props.put(KNSConstants.BUSINESS_OBJECT_CLASS_ATTRIBUTE, dataObject.getClass().getName()); | 828 |
| delyea - DOCUMENT THE FOLLOWING CHANGES (next 2 lines) return UrlFactory.parameterizeUrl(KNSConstants.MAINTENANCE_ACTION, props); | 832 |
| delyea - Multiple value returns are not yet implemented if(StringUtils.isEmpty(lookupForm.getHtmlDataType()) || HtmlData.ANCHOR_HTML_DATA_TYPE.equals(lookupForm.getHtmlDataType())) { | 939 |
| delyea - Multiple value returns are not yet implemented | 948 |
| delyea - Figure out how to impelement Anchors in new UIF String lookupAnchor = ""; if (StringUtils.isNotEmpty(anchor)) { lookupAnchor = anchor; } buffer.append("&anchor=" + lookupAnchor + "&docNum=" + (StringUtils.isEmpty(getDocNum()) ? "" : getDocNum())); | 1015 |
| org.kuali.rice.kns.uif.service.impl.ViewHelperServiceImpl | Line |
| initialize from dictionary | 224 |
| should check to see if there is an add line method on the collection parent and if so call that instead of just adding to the collection (so that sequence can be set) | 738 |
| this should invoke rules, sublclasses like the document view should create the document add line event | 769 |
| look into other ways of identifying a line so we can deal with unordered collections | 792 |
| this should invoke rules, sublclasses like the document view should create the document delete line event | 823 |
| this should go through our formatters | 922 |
| org.kuali.rice.kns.uif.service.impl.ViewServiceImpl | Line |
| remove once we can get beans by type from spring | 47 |
| org.kuali.rice.kns.uif.util.ClientValidationUtils | Line |
| make this show the fields its requiring | 569 |
| org.kuali.rice.kns.uif.util.ComponentBeanPostProcessor | Line |
| removed until ids are fixed to not have underscore if (StringUtils.contains(component.getId(), "_")) { throw new RuntimeException(("Component id is not allowed to have an underscore")); } | 71 |
| org.kuali.rice.kns.uif.widget.Inquiry | Line |
| need general format util that uses spring | 158 |
| should we really put the link label here or just wrap the written value? | 195 |
| org.kuali.rice.kns.uif.widget.LightBox | Line |
| move to component base | 41 |
| org.kuali.rice.kns.uif.widget.QuickFinder | Line |
| --No comment-- | 129 |
| displayedFieldnames in org.kuali.rice.kns.lookup.LookupUtils.generateFieldConversions(BusinessObject, String, BusinessObjectRelationship, String, List, String) | 163 |
| displayedFieldnames and displayedQFFieldNames in generateLookupParameters(BusinessObject, String, BusinessObjectRelationship, String, List, String) | 188 |
| org.kuali.rice.kns.util.FieldUtils | Line |
| do we need to prefix the universalIdAttributeName in Field as well? | 886 |
| assign the correct propertyName to the container in the first place. For now, I'm wary of the weird usages of constructContainerField(). | 1373 |
| org.kuali.rice.kns.util.KNSUtils | Line |
| this method will probably need to be exposed in a public KNSUtils class as it is used by several different modules. That will have to wait until ModuleService and KualiModuleService are moved to core though. | 135 |
| org.kuali.rice.kns.util.MaintenanceUtils | Line |
| implement real authorization for Maintenance Document Save/Route - KULNRVSYS-948 | 406 |
| org.kuali.rice.kns.util.ObjectUtils | Line |
| The rules about this dot notation needs to be explained in Confluence using examples. | 228 |
| genericize collectionContainsObjectWithIdentitcalKey() to leverage this method? | 711 |
| org.kuali.rice.kns.web.comparator.CellComparatorHelper | Line |
| , do we really need to create so many comparators (1 per each cell)? | 95 |
| org.kuali.rice.kns.web.spring.UifConfigurableWebBindingInitializer | Line |
| do we need this since we are switching to spring tags | 48 |
| org.kuali.rice.kns.web.spring.UifControllerHandlerInterceptor | Line |
| do we need to clear the messages before this so that formatting and validation errors done during the binding are not cleared out? | 70 |
| org.kuali.rice.kns.web.spring.UifServletRequestDataBinder | Line |
| this should get from form not previous view, test | 121 |
| revisit and see if we can have a general pattern if ((previousView != null) && !parameterMap.containsKey(UifParameters.RENDER_FULL_VIEW)) { form.setRenderFullView(false); } | 136 |
| org.kuali.rice.kns.web.spring.UifViewBeanWrapper | Line |
| clear cache? | 180 |
| clear cache? | 187 |
| org.kuali.rice.kns.web.spring.controller.DocumentControllerBase | Line |
| authorization on document actions if (KEWConstants.SUPERUSER_COMMAND.equalsIgnoreCase(command)) { form.setSuppressAllButtons(true); } | 113 |
| turn this back on refreshAdHocRoutingWorkgroupLookups(request, form); | 186 |
| if the happens we may need to save form to session or account for it | 193 |
| prerules kualiDocumentFormBase.setDerivedValuesOnForm(request); ActionForward preRulesForward = promptBeforeValidation(mapping, form, request, response); if (preRulesForward != null) { return preRulesForward; } | 221 |
| adhoc recipients getDocumentService().routeDocument(document, form.getAnnotation(), combineAdHocRecipients(kualiDocumentFormBase)); | 237 |
| this needs more analysis before porting can finish | 260 |
| does this really belong in the loop if | 267 |
| need to look at screen, will most of this just be bound to the | 288 |
| not ready for question framework yet | 360 |
| figure out what to return | 377 |
| org.kuali.rice.kns.web.spring.controller.MaintenanceDocumentController | Line |
| getting double view if we call base, not sure how to handle so pasting in superclass code super.docHandler(formBase, request, response); * begin copy/paste from the base | 75 |
| set state in view form.setReadOnly(true); | 98 |
| should be in the view as permission | 223 |
| remove once error path is fixed in business rules // update error messages Map<String, AutoPopulatingList> adjustedPathErrors = new HashMap<String, AutoPopulatingList>(); MessageMap messageMap = GlobalVariables.getMessageMap(); for (Entry<String, AutoPopulatingList> pathErrors : messageMap.getErrorMessages().entrySet()) { String path = pathErrors.getKey(); if (path.startsWith("document.newMaintainableObject") && !path.startsWith("document.newMaintainableObject.dataObject")) { String adjustedPath = StringUtils.replace(path, "document.newMaintainableObject", "document.newMaintainableObject.dataObject"); adjustedPathErrors.put(adjustedPath, pathErrors.getValue()); } } | 259 |
| org.kuali.rice.kns.web.spring.controller.UifControllerBase | Line |
| this needs setup for lightbox and possible home location property | 279 |
| this code still needs ported with whatever we are supposed to do on refresh | 306 |
| lookup anchors and doc number? | 361 |
| multi-value lookup requests | 363 |
| org.kuali.rice.kns.web.spring.form.LookupForm | Line |
| delyea - is this the best way to decide whether to display the maintenance actions? | 153 |
| delyea - how to handle whether to show maintenance links or return links if (!isShowMaintenanceLinks()) { if (StringUtils.contains(getBackLocation(), "/" + KNSConstants.PORTAL_ACTION) || StringUtils.contains(getBackLocation(), "/index.html")) { showMaintenanceLinks = true; } } | 267 |
| org.kuali.rice.kns.web.struts.action.KualiAction | Line |
| create a method for this to be used by both lookup & inquiry ? | 654 |
| org.kuali.rice.kns.web.struts.action.KualiDocumentActionBase | Line |
| this should be pulled into a properties file | 302 |
| remove this when further testing passed if (form instanceof KualiDocumentFormBase) { UserSession userSession = (UserSession) request.getSession().getAttribute(RiceConstants.USER_SESSION_KEY); // force to recreate formkey in execute method if (document instanceof SessionDocument && userSession.retrieveObject(kualiDocumentFormBase.getFormKey()) != null) { userSession.removeObject(kualiDocumentFormBase.getFormKey());; } } | 548 |
| remove this when further testing passed if (form instanceof KualiDocumentFormBase) { UserSession userSession = (UserSession) request.getSession().getAttribute(RiceConstants.USER_SESSION_KEY); // force to recreate formkey in execute method if (document instanceof SessionDocument && userSession.retrieveObject(kualiDocumentFormBase.getFormKey()) != null) { userSession.removeObject(kualiDocumentFormBase.getFormKey());; } } | 588 |
| gah! this is awful | 1381 |
| org.kuali.rice.kns.web.struts.action.KualiMaintenanceDocumentAction | Line |
| this code should be abstracted out into a helper also is it a problem that we're not calling setGenerateDefaultValues? it blanked out the below values when I did maybe we need a new generateDefaultValues that doesn't overwrite? | 315 |
| Cathy remember to delete this block of comments after I've tested. PersistableBusinessObject bo = document.getNewMaintainableObject().getBusinessObject(); Collection maintCollection = this.extractCollection(bo, collectionName); String docTypeName = ((MaintenanceDocument) maintenanceForm.getDocument()).getDocumentHeader().getWorkflowDocument().getDocumentType(); Class collectionClass = extractCollectionClass(docTypeName, collectionName); | 534 |
| sort of collection, new instance should be first | 677 |
| Should we keep this logic and continue using currentTabIndex as the key in the tabStates HashMap ? | 738 |
| Should we keep this logic and continue using currentTabIndex as the key in the tabStates HashMap ? | 865 |
| org.kuali.rice.kns.web.struts.form.KualiDocumentFormBase | Line |
| we should not be referencing kew constants from this class and wedding ourselves to that workflow application This class is the base action form for all documents. | 62 |
| is this still needed? I think it's obsolete now | 84 |
| Chris - Notes: remove the above and change the below from boNotes when notes are finished overriding note formatter to make sure they post back the full timestamp | 155 |
| rk implemented to account for caps coming from kuali user service from workflow | 571 |
| org.kuali.rice.kns.web.struts.form.KualiMaintenanceForm | Line |
| to be persisted in the lookup results service instead? | 89 |
| org.kuali.rice.kns.web.struts.form.LookupForm | Line |
| remove these once DD changes have been made | 467 |
| remove these once DD changes have been made | 472 |
| remove these once DD changes have been made | 477 |
| remove these once DD changes have been made | 482 |
| org.kuali.rice.kns.web.struts.pojo.PojoFormBase | Line |
| see KULOWF-194 testForPojoHack(this, keypath); | 181 |
| org.kuali.rice.kns.web.ui.FieldBridge | Line |
| St. Ailish says review this. A question was raised on 11-16-2006 Tuscon meeting as to why this is done here and not in the formatter. | 310 |
| org.kuali.rice.kns.web.ui.SectionBridge | Line |
| Revisit this. Changing since getPrimaryKeys and listPrimaryKeyFieldNames are apparently same. May be we might want to replace listPrimaryKeyFieldNames with getPrimaryKeys... Not sure. | 753 |
| org.kuali.rice.kns.workflow.DataDictionarySearchableAttributeTest | Line |
| Add the commented-out boolean search expressions back in once KULRICE-3698 is complete. | 186 |
| org.kuali.rice.kns.workflow.attribute.DataDictionarySearchableAttribute | Line |
| this is a workaround to hide the Field from the search criteria. This should be removed once hiding the entire Row is working | 256 |
| org.kuali.rice.kns.workflow.attribute.KualiXmlAttributeHelper | Line |
| - enter JIRA below removes wf:xstreamsafe( and ) below separates each wf:xstreamsafe() section into separate 'finds' | 55 |
| - Fix the document content element generation | 141 |
| org.kuali.rice.kns.workflow.service.impl.WorkflowDocumentServiceImpl | Line |
| make the 1 a constant | 358 |
| org.kuali.rice.krad.app.persistence.jpa.RiceLocalContainerEntityManagerFactoryBean | Line |
| Add more vendor specific defaults... | 113 |
| org.kuali.rice.krms.api.engine.EngineResults | Line |
| - need to determine what goes here... | 7 |
| org.kuali.rice.krms.api.engine.ExecutionEnvironment | Line |
| we don't support updating facts, refactor this method | 36 |
| org.kuali.rice.krms.api.engine.Term | Line |
| make this pretty | 95 |
| org.kuali.rice.krms.api.repository.agenda.AgendaTreeEntryDefinition | Line |
| ... | 6 |
| org.kuali.rice.krms.api.repository.context.ContextDefinitionContract | Line |
| this java doc is confusing | 60 |
| org.kuali.rice.krms.api.repository.proposition.PropositionDefinition | Line |
| change this to field name to id | 66 |
| org.kuali.rice.krms.api.repository.proposition.PropositionParameter | Line |
| isBlank or is null ??? | 183 |
| check against valid values | 197 |
| org.kuali.rice.krms.api.repository.proposition.PropositionParameterType | Line |
| ... | 22 |
| org.kuali.rice.krms.api.repository.proposition.PropositionType | Line |
| ... | 25 |
| org.kuali.rice.krms.api.repository.term.TermResolverDefinition | Line |
| proper validation & javadocs | 201 |
| */); | 208 |
| */); | 218 |
| */); | 228 |
| */); | 245 |
| */); | 255 |
| org.kuali.rice.krms.api.repository.type.KrmsTypeAttribute | Line |
| verify that the attributeDefinitionID field matches the id field in the builder | 207 |
| org.kuali.rice.krms.api.test.JAXBAssert | Line |
| - copied this from the core api test module, find a better way to share this code! | 33 |
| org.kuali.rice.krms.framework.TermResolutionEngineTest | Line |
| test exception variants: - TermResolver throws TermResolutionException - TermResolutionEngine is passed a null term | 293 |
| what should the TermResolutionEngine do if a resolver throws a RuntimeException? | 298 |
| org.kuali.rice.krms.framework.engine.AgendaTreeEntry | Line |
| ... | 21 |
| org.kuali.rice.krms.framework.engine.BasicExecutionEnvironment | Line |
| inject this (will have to make it non-final) | 36 |
| org.kuali.rice.krms.framework.engine.CollectionOfComparablesTermBasedProposition | Line |
| log this appropriately | 42 |
| org.kuali.rice.krms.framework.engine.EngineResultsImpl | Line |
| Auto-generated method stub | 24 |
| Auto-generated method stub | 30 |
| org.kuali.rice.krms.framework.engine.StringTermBasedProposition | Line |
| remove? ComparableTermBasedProposition does this generically | 5 |
| org.kuali.rice.krms.framework.engine.TermResolutionEngineImpl | Line |
| could cache these plans somewhere, since future agendas will likely require the same plans | 64 |
| org.kuali.rice.krms.framework.engine.expression.BinaryOperatorExpression | Line |
| ... | 21 |
| org.kuali.rice.krms.framework.engine.expression.BooleanValidatingExpression | Line |
| ... | 22 |
| org.kuali.rice.krms.framework.engine.expression.ComparisonOperator | Line |
| this implementation seems largely incomplete, it seems we are need to have some kind of engine or utility which can coerce types to possible forms for comparision purposes? For now, let's verify they are of the same type | 39 |
| not sure what to do about this cast and whether or not it will safe, be sure to hit this in unit testing! | 57 |
| org.kuali.rice.krms.framework.engine.expression.ConstantExpression | Line |
| ... | 21 |
| org.kuali.rice.krms.framework.engine.expression.Expression | Line |
| ... | 21 |
| org.kuali.rice.krms.framework.engine.expression.ExpressionBasedProposition | Line |
| ... | 25 |
| org.kuali.rice.krms.framework.engine.expression.FunctionExpression | Line |
| ... | 25 |
| org.kuali.rice.krms.framework.engine.expression.TermExpression | Line |
| ... | 23 |
| org.kuali.rice.krms.framework.engine.result.Log4jResultListener | Line |
| Auto-generated method stub | 12 |
| org.kuali.rice.krms.impl.provider.repository.LazyAction | Line |
| ... | 26 |
| org.kuali.rice.krms.impl.provider.repository.LazyAgendaTree | Line |
| ... | 23 |
| org.kuali.rice.krms.impl.provider.repository.LazyProposition | Line |
| ... | 29 |
| org.kuali.rice.krms.impl.provider.repository.RepositoryToEngineTranslatorImpl | Line |
| ... | 58 |
| log warning when termResolver comes back null? or throw exception? | 110 |
| this limits an agenda to a single event. Is that good enough? | 118 |
| org.kuali.rice.krms.impl.provider.repository.RuleRepositoryContextProvider | Line |
| should have an execution option that throws an error here if a context does not exist? | 29 |
| org.kuali.rice.krms.impl.provider.repository.SimplePropositionTypeService | Line |
| - need some way to define data type on the prop parameter as well? Not all constants will actually be String values!!! | 90 |
| throw an exception if function is null? | 100 |
| need to check types here? expression object probably needs a getType on it so that we can confirm that the types will be compatible? | 110 |
| org.kuali.rice.krms.impl.repository.AgendaBoServiceImpl | Line |
| deal with active flag | 37 |
| org.kuali.rice.krms.impl.repository.FunctionBoServiceImpl | Line |
| ... | 9 |
| --No comment-- | 18 |
| --No comment-- | 24 |
| org.kuali.rice.krms.impl.repository.KrmsTypeBoServiceImpl | Line |
| dseibert - THIS METHOD NEEDS JAVADOCS | 164 |
| dseibert - THIS METHOD NEEDS JAVADOCS | 175 |
| org.kuali.rice.krms.impl.repository.RuleRepositoryServiceImpl | Line |
| use new criteria API so we can match multiple qualifiers at once. | 47 |
| prevent circular, endless looping | 127 |
| validate that only either rule or subagenda, not both | 136 |
| org.kuali.rice.krms.impl.repository.TermBoService | Line |
| javadocs | 32 |
| org.kuali.rice.krms.impl.ui.EditorController | Line |
| throw out this hacky junk | 59 |
| org.kuali.rice.krms.impl.ui.EditorDocumentMaintainable | Line |
| change this, it makes more sense for MAINTENANCE_EDIT_ACTION | 61 |
| here's where we can handle persisting our agenda and context | 86 |
| gilesp - THIS METHOD NEEDS JAVADOCS super.processAfterNew(document, parameters); | 96 |
| gilesp - THIS METHOD NEEDS JAVADOCS | 111 |
| gilesp - THIS METHOD NEEDS JAVADOCS | 122 |
| org.kuali.rice.ksb.config.xfire.XFireWSS4JInHandler | Line |
| Replace this class with cxf wss4j in interceptor | 31 |
| org.kuali.rice.ksb.config.xfire.XFireWSS4JOutHandler | Line |
| Replace this class with a cxf wss4j out interceptor | 30 |
| org.kuali.rice.ksb.impl.bus.ServiceBusImpl | Line |
| - would be better to return an Endpoint that contained an internal proxy to all the services so fail-over would be easier to implement! | 241 |
| - this should also probably check the current status of the service? | 244 |
| org.kuali.rice.ksb.impl.registry.LazyRemoteServiceRegistryConnector | Line |
| ... | 40 |
| org.kuali.rice.ksb.impl.registry.diff.LocalServicesDiff | Line |
| ... | 26 |
| org.kuali.rice.ksb.impl.registry.diff.RemoteServicesDiff | Line |
| ... | 25 |
| org.kuali.rice.ksb.impl.registry.diff.ServiceRegistryDiffCalculator | Line |
| ... | 25 |
| org.kuali.rice.ksb.messaging.DelayedAsynchronousServiceTest | Line |
| this isn't the best test ever because it's relying on waits and timing which is most likely doomed to occasional failure in the CI environment. If this occurs than I may need to yank this. A better long term solution would be to allow for the use of callbacks for delayed asynchronous services but that's not something I wanted to try to tackle at the moment | 74 |
| org.kuali.rice.ksb.messaging.JmsMessagingNoRun | Line |
| renable this test | 26 |
| org.kuali.rice.ksb.messaging.config.KSBConfigurer | Line |
| redo this once we're back to JPA springFileLocations.add("classpath:org/kuali/rice/ksb/config/KSBJPASpringBeans.xml"); | 118 |
| hack 'cause KSB used KNS - this needs to be fixed!!! | 136 |
| org.kuali.rice.ksb.messaging.serviceconnectors.HttpInvokerConnector | Line |
| we need to add support for other invocation protocols and implementations, but for now... | 102 |
| org.kuali.rice.ksb.messaging.serviceconnectors.ServiceConnectorFactory | Line |
| switch this to use serviceConfiguration.getType() at some point in the future and allow for this to be easily "pluggable" with new connector types | 83 |
| org.kuali.rice.ksb.messaging.servlet.KSBDispatcherServlet | Line |
| this just seems weird as this controller is initially null when it's created, does there need to be some synchronization here? | 119 |
| org.kuali.rice.ksb.messaging.web.MessageQueueAction | Line |
| better error handling here | 393 |
| org.kuali.rice.ksb.messaging.web.ServiceRegistryAction | Line |
| is this what really constitutes a "refresh" of the service registry? | 58 |
| org.kuali.rice.ksb.security.soap.CXFWSS4JInInterceptor | Line |
| Replace this class with cxf wss4j in interceptor | 41 |
| org.kuali.rice.ksb.testclient1.JaxWsEchoServiceImpl | Line |
| Will Gomes - THIS METHOD NEEDS JAVADOCS | 40 |
| org.kuali.rice.test.data.UnitTestData | Line |
| Come back and add the JPQL equivalents | 39 |
| org.kuali.rice.test.lifecycles.SQLDataLoaderLifecycle | Line |
| may way to do something with the dataLoader | 59 |
| org.rice.krms.test.AbstractBoTest | Line |
| get rid of this hack that is needed to set up the OJB properties location at the right time. BEGIN hack | 18 |
| this is extra annoying, we have to have our own copy of RiceOJB.properties >:-[ | 26 |
| org.rice.krms.test.RepositoryCreateAndExecuteIntegrationTest | Line |
| custom functions too. | 148 |
| do this fur eel | 199 |
| I'm concerned about how this will deal w/ concurrency. | 347 |
| org.rice.krms.test.TestBoService | Line |
| Auto-generated method stub | 69 |
| Auto-generated method stub | 79 |
| Auto-generated method stub | 114 |
| Auto-generated method stub | 123 |
| Auto-generated method stub | 133 |