Modules
Project Documentation
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 | 127 | FIXME |
| TODO's | 636 | TODO |
Each tag is detailed below:
Number of occurrences found in the code: 127
| org.kuali.student.common.assembly.transform.DefaultDataBeanMapper | Line |
|---|---|
| Probably a good idea to change "clazz" parameter to be "String objectType" to be consistent with convertFromBean method | 64 |
| Fix this so this warning never happens !! | 260 |
| org.kuali.student.common.assembly.transform.TransformationManager | Line |
| It might be more efficient to getMetadata outside of the for loop (unless metadata might be different) | 126 |
| org.kuali.student.common.assembly.util.IdTranslator | Line |
| workaround until orch dict can handle multi part keys on initiallookup defs | 61 |
| org.kuali.student.common.dao.impl.AbstractSearchableCrudDaoImpl | Line |
| SQL injection can occur here - or NOT if we need to assemble SQL to cover various ways one can compare criteria to a text | 126 |
| org.kuali.student.common.dto.DtoConstants | Line |
| Need to split out proposal states (ie. workflow states) versus dto states | 22 |
| org.kuali.student.common.rice.StudentWorkflowConstants | Line |
| The workflow doc type needs to be removed from ProposalWorkflowFilter | 6 |
| org.kuali.student.common.search.service.impl.CrossSearchManager | Line |
| this is pretty inefficient to loop through everything... a map structure for the cells might be better | 158 |
| breaks are bad... but there is no map in the cells | 167 |
| breaks are bad... but there is no map in the cells | 188 |
| breaks are bad... but there is no map in the cells | 216 |
| breaks are bad... but there is no map in the cells | 227 |
| needs a handle to the result params data types here | 305 |
| org.kuali.student.common.search.service.impl.SearchManagerImpl | Line |
| should we change these to Operation Failed Exceptions? also we need to handle invalid parameters. | 165 |
| org.kuali.student.common.ui.client.application.ViewContext | Line |
| change state to proper default or null | 40 |
| org.kuali.student.common.ui.client.configurable.mvc.Configurer | Line |
| WJG: I think state should be removed from the configurer | 23 |
| org.kuali.student.common.ui.client.configurable.mvc.DelayedToolView | Line |
| ? need to wire onReadyCallback into the model request, so that we aren't indicating that we're ready before the model is available? | 86 |
| org.kuali.student.common.ui.client.configurable.mvc.FieldDescriptor | Line |
| This could be problematic if minOccurs should be something other than 1 | 398 |
| org.kuali.student.common.ui.client.configurable.mvc.layouts.TabbedSectionLayout | Line |
| Better way to manage hierarchy, ordering, and handle to views | 51 |
| this is not safe for all sorts of reasons, do not call handlers directly like this. | 106 |
| keep or delete? | 132 |
| org.kuali.student.common.ui.client.security.SpringSecurityLoginRedirectHandler | Line |
| Need a way to get the proper redirect url, for now just reloading the page BrowserUtils.redirect(GWT.getHostPageBaseURL()); | 27 |
| org.kuali.student.common.ui.client.util.SearchUtils | Line |
| throw an exception? | 116 |
| org.kuali.student.common.ui.client.validator.DataModelValidator | Line |
| This is a temp solution for getting cross field min value | 680 |
| This is a temp solution for getting cross field max value | 699 |
| org.kuali.student.common.ui.client.widgets.containers.KSWrapper | Line |
| notify current controller of the page change so it can perform an action | 101 |
| before navigation event | 102 |
| does this do anything? | 160 |
| org.kuali.student.common.ui.client.widgets.list.KSSelectedList | Line |
| do we need to do a search? is this method ever going to be called? | 417 |
| org.kuali.student.common.ui.client.widgets.list.impl.KSLabelListImpl | Line |
| Is there a better way to display selected item here without redrawing | 103 |
| org.kuali.student.common.ui.client.widgets.pagetable.PagingScrollTableBuilder | Line |
| workaround for incubator bug [KSCOR-225] This table to be replaced in M6 | 176 |
| org.kuali.student.common.ui.client.widgets.search.KSPicker | Line |
| should we search on values to populate drop down here or later when user will access the screen? | 210 |
| This method of creating read is very inefficient, need better solution | 233 |
| is this configuration error? | 326 |
| what about the result id? | 368 |
| what about the result ids? | 372 |
| org.kuali.student.common.ui.client.widgets.search.SearchPanel | Line |
| hack - comparison to 'optional' - replace with check against 'optional' field and update related lookup metadata | 541 |
| throw an exception? | 552 |
| remove because required field '*' indication will be part of FieldElement class | 623 |
| Is an action complete callback really necessary here, couldn't this simply be a method to perform the seach action. | 835 |
| hack - comparison to 'optional' - replace with check against 'optional' field and update related lookup metadata | 870 |
| org.kuali.student.common.ui.client.widgets.search.SearchResultsTable | Line |
| do we really need to recreate the table for every refresh? | 99 |
| Do we still need this if condition? Added an else to the if(pagedResults) line to prevent searches being executed twice if the search name includes cross | 159 |
| more detail info here | 195 |
| org.kuali.student.common.ui.client.widgets.search.SwappablePanel | Line |
| eventually we will need 'Search For' label as well. can the use of 'Search By' or 'Search For' be automatically implied or do we need another lookup configuration parameter? | 40 |
| org.kuali.student.common.ui.client.widgets.search.TempSearchBackedTable | Line |
| more detail info here | 99 |
| auto adjusting width to fill table does not work with 1 column bug in incubator??? | 140 |
| do we really need to recreate the table for every refresh? | 151 |
| Undesirable solution to work with GWT 2.0 | 157 |
| org.kuali.student.common.ui.client.widgets.suggestbox.SearchSuggestOracle | Line |
| handle case when search for text is not appearing within search result - should not happen (misconfiguration) | 270 |
| org.kuali.student.common.ui.client.widgets.suggestbox.SuggestPicker | Line |
| Suggest picker marker interface, temporary fix? | 24 |
| org.kuali.student.common.ui.server.gwt.SearchDispatchRpcGwtServlet | Line |
| can this special case be handled after this call? | 69 |
| org.kuali.student.common.ui.server.gwt.SecurityRpcGwtServlet | Line |
| Is there a way to retrieve multiple permissions at once instead of calling isAuthorized multiple times? | 76 |
| org.kuali.student.common.ui.server.gwt.old.AbstractBaseDataOrchestrationRpcGwtServlet | Line |
| issues: -The Type/state config is hardcoded here which will cause troubles with different types and states -Workflow filter should be combined with this for save -The exception handling here needs standardization. Should RPC errors throw operation failed with just the message and log the message and exception? also should calls that return Boolean ever throw exceptions? | 48 |
| should not pass empty id. What to do here? | 79 |
| org.kuali.student.common.validator.DefaultValidatorImpl | Line |
| For clarity, might be better to handle this in the processRequireConstraint method instead. | 331 |
| org.kuali.student.core.comment.service.impl.TestCommentServiceImpl | Line |
| validation needs to be tested, this code doesn't work and I need to refactor CommentInfo commentInfo = new CommentInfo(); RichTextInfo commentText = new RichTextInfo(); commentText.setFormatted("<p>comment>!@#02h%$</p>"); commentText.setPlain("comment"); commentInfo.setCommentText(commentText); commentInfo.setReferenceTypeKey("referenceKey"); commentInfo.setReferenceId("referenceId"); commentInfo.setEffectiveDate(new Date()); commentInfo.setExpirationDate(new Date()); | 129 |
| org.kuali.student.core.document.service.impl.TestDocumentServiceImpl | Line |
| validation needs to be tested, this code doesn't work and I need to refactor DocumentInfo documentInfo = new DocumentInfo(); RichTextInfo desc = new RichTextInfo(); desc.setFormatted("<p>document>!%/#;&@$</p>"); desc.setPlain("document"); documentInfo.setDesc(desc); documentInfo.setFileName("sample.pdf"); documentInfo.setEffectiveDate(new Date()); documentInfo.setExpirationDate(new Date()); documentInfo.setType("kuali.org.Document"); documentInfo.setState("active"); | 244 |
| org.kuali.student.core.document.ui.server.upload.UploadServlet | Line |
| Probably temporary solution for type on document info | 152 |
| org.kuali.student.core.enumerationmanagement.service.impl.EnumerationManagementServiceImpl | Line |
| need real validation | 65 |
| need to get messages here | 115 |
| need to get messages here | 178 |
| org.kuali.student.core.organization.assembly.OrgProposalAssembler | Line |
| should this be something like org.proposal? | 313 |
| org.kuali.student.core.organization.service.impl.OrganizationServiceImpl | Line |
| redo validation here and for all calls to create/update return validator.validateTypeStateObject(orgInfo, getObjectStructure("orgInfo")); | 650 |
| org.kuali.student.core.organization.ui.client.mvc.controller.OrgProposalController | Line |
| needs to check validation results and display messages if validation failed This was reviewd as part of KSCOR-220, deferred to KSCOR-225, can't be fixed until server side validation completed | 454 |
| org.kuali.student.core.personsearch.service.impl.KsPerson | Line |
| Either find a cleaner fix or go back to using PersonSerice | 38 |
| org.kuali.student.core.statement.service.assembler.StatementTreeViewAssembler | Line |
| Unsure now if this is an exception or just return null or empty assemblyNode | 47 |
| org.kuali.student.core.statement.service.impl.TestStatementServiceImpl | Line |
| - Investigate why adding clu1, clu3, clu2 works but adding clu1, clu2, clu3 doesn't work | 157 |
| org.kuali.student.lum.common.client.lo.CategoryManagement | Line |
| [KSCOR-225] user needs to specify what LoRepository they want category to tagged with | 652 |
| org.kuali.student.lum.common.client.lo.LOCategoryBuilder | Line |
| [KSCOR-225] Class needs to be rewritten to use KSPicker instead of SuggestPicker and use lookup config through metadata | 465 |
| org.kuali.student.lum.common.client.lu.LUUIConstants | Line |
| Duplicated from lum-api LUConstants | 28 |
| org.kuali.student.lum.course.service.assembler.ActivityAssembler | Line |
| Unsure now if this is an exception or just return null or empty assemblyNode | 60 |
| org.kuali.student.lum.course.service.assembler.CourseAssembler | Line |
| Unsure now if this is an exception or just return null or empty assemblyNode | 286 |
| org.kuali.student.lum.course.service.assembler.CourseAssemblerConstants | Line |
| follow Norm's Type Keys (this requires impex changes) public final static String COURSE_RESULT_TYPE_GRADE = "kuali.resultType.grades"; public final static String COURSE_RESULT_TYPE_CREDITS = "kuali.resultType.credits"; | 56 |
| ask norm for these values | 74 |
| ask norm about this value | 90 |
| org.kuali.student.lum.course.service.assembler.CourseJointAssembler | Line |
| is this ever used? | 66 |
| is this ever used? | 93 |
| Unsure now if this is an exception or just return null or empty assemblyNode | 113 |
| org.kuali.student.lum.course.service.assembler.FormatAssembler | Line |
| Unsure now if this is an exception or just return null or empty assemblyNode | 102 |
| org.kuali.student.lum.course.service.impl.TestCourseServiceImpl | Line |
| --No comment-- | 804 |
| need a dictionary that defines StatamentTreeViewInfo | 925 |
| need a dictionary that defines StatamentTreeViewInfo | 937 |
| org.kuali.student.lum.lu.assembly.CluSetManagementAssembler | Line |
| should have it's own proposal types | 58 |
| --No comment-- | 524 |
| --No comment-- | 529 |
| --No comment-- | 533 |
| org.kuali.student.lum.lu.entity.Clu | Line |
| dates should be either set from the DB time as part of the insert statement, or set from the application. DB timestamp (CURRENT_TIMESTAMP) is preferred | 47 |
| org.kuali.student.lum.lu.service.impl.LuServiceImpl | Line |
| should be more checked service exceptions thrown | 3116 |
| should be more checked service exceptions thrown | 3646 |
| should be more checked service exceptions thrown | 3648 |
| , this should be DB time | 4017 |
| org.kuali.student.lum.lu.ui.course.client.configuration.CourseProposalConfigurer | Line |
| [KSCOR-225] Temporary fix til we have a real rich text editor addField(description, COURSE + "/" + DESCRIPTION, null); | 486 |
| [KSCOR-225] Commented out search code to display drop down with only current user, and disable select | 823 |
| org.kuali.student.lum.lu.ui.course.client.configuration.ViewCourseConfigurer | Line |
| [KSCOR-225] Temp paths waiting for DOL changes | 78 |
| [KSCOR-225] Initialize type and state from selected cluId | 84 |
| org.kuali.student.lum.lu.ui.course.client.controllers.CourseAdminController | Line |
| This static method should not live in WorkflowUtilities | 347 |
| org.kuali.student.lum.lu.ui.course.client.controllers.CourseAdminRetireController | Line |
| Need to add proper authorization checks for admin modify. | 136 |
| org.kuali.student.lum.lu.ui.course.client.controllers.CourseAdminWithoutVersionController | Line |
| This may not be what we want to do | 42 |
| Need to add proper authorization checks for admin modify. | 167 |
| org.kuali.student.lum.lu.ui.course.client.controllers.CourseProposalController | Line |
| Something looks odd in determining currentDocType | 307 |
| This method should not require a permissionType as a parameter | 986 |
| org.kuali.student.lum.lu.ui.course.client.controllers.VersionsController | Line |
| this is not updating the cluModel so state will not be updated in the model. May not be a problem. | 314 |
| org.kuali.student.lum.lu.ui.course.client.controllers.ViewCourseController | Line |
| this is not updating the cluModel so state will not be updated in the model. May not be a problem. | 162 |
| org.kuali.student.lum.lu.ui.course.client.widgets.CourseWorkflowActionList | Line |
| This needs to use a new workflow document type for admin modify with version | 322 |
| org.kuali.student.lum.lu.ui.main.client.widgets.ApplicationHeader | Line |
| notify current controller of the page change so it can perform an action | 98 |
| before navigation event | 99 |
| the following code gets overridden | 153 |
| org.kuali.student.lum.lu.ui.tools.client.configuration.CluSetsConfigurer | Line |
| do something else here, rework logic rootSection.redraw(); | 370 |
| org.kuali.student.lum.program.client.ProgramUtils | Line |
| Unregistering of handlers should be better handled | 145 |
| org.kuali.student.lum.program.client.major.edit.MajorEditController | Line |
| The proper way of doing this would be a single server side call to validate next state which would retrieve warnings & required for next state, instead of re-validating warnings for current state server side and validating required for next state client side. | 149 |
| This is ugly but gets us past a blocker issue. Theres something wrong with the way the models are handled in the major and variation controllers so they get out of sync. This is a temporary workaround | 214 |
| org.kuali.student.lum.program.client.major.proposal.MajorProposalController | Line |
| The proper way of doing this would be a single server side call to validate next state which would retrieve warnings & required for next state, instead of re-validating warnings for current state server side and validating required for next state client side. | 276 |
| This is ugly but gets us past a blocker issue. Theres something wrong with the way the models are handled in the major and variation controllers so they get out of sync. This is a temporary workaround | 336 |
| org.kuali.student.lum.program.client.major.view.MajorViewController | Line |
| Find a better way to do this. We shouldn't be maintaining two separate datamodels for progs and variations | 412 |
| org.kuali.student.lum.program.server.transform.VersionProgramFilter | Line |
| possible synchronization issue here with instance variable previousVersionMetadata | 109 |
| org.kuali.student.lum.program.service.assembler.CoreProgramAssembler | Line |
| Unsure now if this is an exception or just return null or empty assemblyNode | 74 |
| org.kuali.student.lum.program.service.assembler.CredentialProgramAssembler | Line |
| Unsure now if this is an exception or just return null or empty assemblyNode | 79 |
| org.kuali.student.lum.program.service.assembler.ProgramAssemblerConstants | Line |
| ask norm about this value | 109 |
| org.kuali.student.lum.program.service.assembler.ProgramVariationAssembler | Line |
| Unsure now if this is an exception or just return null or empty assemblyNode | 78 |
| org.kuali.student.lum.program.service.impl.ProgramRequirementAssembler | Line |
| Unsure now if this is an exception or just return null or empty assemblyNode | 103 |
| Derive from statement and rule types | 188 |
| org.kuali.student.lum.program.service.impl.ProgramServiceImpl | Line |
| , this validation should be moved into a custom validation class + configuration | 1523 |
| error should return using message service and not static text | 1547 |
| org.kuali.student.lum.program.service.impl.TestProgramServiceImpl | Line |
| --No comment-- | 132 |
| Updated version info isn't returned | 1077 |
| org.kuali.student.lum.statement.typekey.ReqComponentFieldTypes | Line |
| Why do we have test data in our code? | 54 |
Number of occurrences found in the code: 636
| org.kuali.student.common.assembly.AddOneAssembler | Line |
|---|---|
| Auto-generated method stub | 32 |
| Auto-generated method stub | 38 |
| Auto-generated method stub | 50 |
| Auto-generated method stub | 65 |
| Auto-generated method stub | 71 |
| org.kuali.student.common.assembly.data.Data | Line |
| probably need to add all of the other remove(type) methods | 607 |
| what should be done if we try to query cluset/clus/0/_runtimeData where cluset/0 returns a string instead of Data throw an exception here? | 864 |
| org.kuali.student.common.assembly.data.LookupMetadata | Line |
| is DEFAULT needed? it has 0 references | 53 |
| BUTTON has 0 references. Is it needed? | 63 |
| org.kuali.student.common.assembly.data.LookupParamMetadata | Line |
| none of these are used | 44 |
| org.kuali.student.common.assembly.data.Metadata | Line |
| When all dictionaries have been updated, this needs to be changed to a single value object. No need for it to be a list with new dictionary structure. | 55 |
| dump lookup/etc info as well | 141 |
| org.kuali.student.common.assembly.data.MetadataInterrogator | Line |
| worry aboutg special validators | 216 |
| worry about how this applies to non-strings? | 217 |
| worry aboutg special validators | 304 |
| worry about how this applies to non-strings? | 305 |
| worry aboutg special validators | 323 |
| worry about how this applies to non-strings? | 324 |
| worry about hard coding ids | 341 |
| Worry about hard coding the cut-off point | 345 |
| org.kuali.student.common.assembly.data.QueryPath | Line |
| revamp this method to use subList once GWT issue 1791 is fixed | 152 |
| org.kuali.student.common.assembly.data.UILookupData | Line |
| is DEFAULT needed? it has 0 references | 40 |
| BUTTON has 0 references. Is it needed? | 47 |
| org.kuali.student.common.assembly.dictionary.MetadataServiceImpl | Line |
| figure out why path=courseInfo.creditOptions.type matches any structure that has a type on it so that lookup gets returned for all types | 689 |
| org.kuali.student.common.assembly.dictionary.old.MetadataServiceImpl | Line |
| --No comment-- | 48 |
| org.kuali.student.common.assembly.helper.PropertyEnum | Line |
| merge this to common, when merging Norm's generated stuff | 18 |
| org.kuali.student.common.assembly.old.BaseAssembler | Line |
| Below must be changed to use constants from KualiStudentKimAttributes class (class is currently in LUM) | 65 |
| org.kuali.student.common.assembly.transform.AuthorizationFilter | Line |
| If fields were masked, need to prevent masked values from being persisted. | 82 |
| org.kuali.student.common.assembly.transform.DefaultDataBeanMapper | Line |
| Determine correct metadata to pass in getCollectionData() instead of null | 234 |
| Determine correct metadata to pass into convertFromBean() instead of null | 237 |
| Find correct metadata to pass in | 346 |
| Find correct metadata to pass in | 349 |
| org.kuali.student.common.assembly.util.AssemblerUtils | Line |
| need a "standard properties" enum for values that could be present on any object? | 94 |
| need a "standard properties" enum for values that could be present on any object? | 123 |
| org.kuali.student.common.dictionary.dto.ObjectStructureDefinition | Line |
| do we need this? | 15 |
| do we need this? | 21 |
| org.kuali.student.common.dictionary.dto.ValidCharsConstraint | Line |
| Should this be a list of values | 10 |
| org.kuali.student.common.dictionary.old.dto.Field | Line |
| there is no required here but there should be | 66 |
| org.kuali.student.common.dictionary.service.impl.DictionaryFormatter | Line |
| other more complex constraints | 755 |
| org.kuali.student.common.dictionary.service.impl.DictionaryValidator | Line |
| Cross compare to make sure min is not greater than max and that default value is valid itself | 133 |
| make the date parser configurable like the validator is | 267 |
| more validation | 330 |
| more validation | 402 |
| org.kuali.student.common.dto.DtoConstants | Line |
| Ideally this method should not be hardcoded here. Also determining next state may be a more complicated and not just be a simple sequence. | 37 |
| org.kuali.student.common.rice.StudentIdentityConstants | Line |
| It seems wrong to include DOCUMENT_TYPE_NAME in an identity constants class | 44 |
| org.kuali.student.common.rice.StudentWorkflowConstants | Line |
| This class needs to be combined with org.kuali.student.StudentWorkflowConstants class found in ks-lum-rice | 3 |
| org.kuali.student.common.search.service.impl.CrossSearchManager | Line |
| get the types for the params! | 233 |
| org.kuali.student.common.search.service.impl.SearchConfigValidator | Line |
| validate | 23 |
| org.kuali.student.common.ui.client.CommonUITest | Line |
| Auto-generated method stub | 121 |
| org.kuali.student.common.ui.client.configurable.mvc.DelayedToolView | Line |
| Auto-generated method stub | 145 |
| org.kuali.student.common.ui.client.configurable.mvc.LayoutController | Line |
| Auto-generated method stub | 594 |
| org.kuali.student.common.ui.client.configurable.mvc.binding.DynamicAttributeListBinding | Line |
| Auto-generated method stub | 84 |
| org.kuali.student.common.ui.client.configurable.mvc.binding.HasValueBinding | Line |
| Type checking to ensure that the value type of widget matches model defintion | 47 |
| org.kuali.student.common.ui.client.configurable.mvc.binding.MultiplicityItemBinding | Line |
| modify this method to use QueryPath.add to build paths, rather than string manipulation | 52 |
| org.kuali.student.common.ui.client.configurable.mvc.binding.RichTextBinding | Line |
| Should these defaults be set in server assembly defaults? Commenting type and state as it is not required for rich text qPath = QueryPath.parse(richTextRoot + "type"); model.set(qPath, "kuali.not.applicable"); | 57 |
| org.kuali.student.common.ui.client.configurable.mvc.layouts.MenuEditableSectionController | Line |
| Change to put in section header | 90 |
| org.kuali.student.common.ui.client.configurable.mvc.layouts.MenuSectionController | Line |
| add description to the menu item | 321 |
| org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityGroup | Line |
| might want fields and nested multiplicities at the same time. Remove if/else | 203 |
| Should copy widgets/bindings too? | 215 |
| org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityGroupItem | Line |
| Should this be an inner class in MultiplicityGroup? | 49 |
| org.kuali.student.common.ui.client.configurable.mvc.multiplicity.MultiplicityTable | Line |
| should just be 1 row def for a table - throw exception if > 1? | 75 |
| org.kuali.student.common.ui.client.configurable.mvc.sections.BaseSection | Line |
| Can we do this without checking for instanceof MG?? | 338 |
| May want to use different styles for field label if not enabled | 738 |
| Make sure this works with all (most?) types of widgets | 746 |
| org.kuali.student.common.ui.client.configurable.mvc.sections.MultiplicitySection | Line |
| --No comment-- | 69 |
| org.kuali.student.common.ui.client.configurable.mvc.views.SectionView | Line |
| review this, shouldn't it assign this.model = m? | 189 |
| Auto-generated method stub | 283 |
| org.kuali.student.common.ui.client.configurable.mvc.views.VerticalSectionView | Line |
| Auto-generated method stub | 112 |
| org.kuali.student.common.ui.client.event.ExportEvent | Line |
| Administrator - THIS METHOD NEEDS JAVADOCS | 31 |
| org.kuali.student.common.ui.client.mvc.Controller | Line |
| this logic needs to be reworked a bit... if an unchecked event has been bound locally, do we want to still fire it externally as well? | 343 |
| add some automatic view context setting here, get and set | 463 |
| Nina ??? Do we want to keep this seen in the light of the exportElements parameter | 600 |
| org.kuali.student.common.ui.client.mvc.ViewComposite | Line |
| Auto-generated method stub | 130 |
| org.kuali.student.common.ui.client.mvc.history.HistoryManager | Line |
| add the ability for view context to add a variety of additional attributes | 305 |
| org.kuali.student.common.ui.client.service.exceptions.OperationFailedException | Line |
| A better solution for exceptions on client side. | 23 |
| org.kuali.student.common.ui.client.util.ExportUtils | Line |
| Nina - This can't be the correct way of getting handle to Controller, isn't there a better way?? | 192 |
| org.kuali.student.common.ui.client.widgets.KSDatePicker | Line |
| 03/11/2009 - This widget currently only supports dates entered in the mm/dd/yyyy format. Known unresolved backspace bug in Firefox. | 36 |
| org.kuali.student.common.ui.client.widgets.KSErrorDialog | Line |
| maybe retrieve more error info | 141 |
| actually gather client context info, such as browser version, user id, etc | 146 |
| org.kuali.student.common.ui.client.widgets.KSItemLabel | Line |
| --No comment-- | 172 |
| does nothing on ok, ok is not currently used | 283 |
| org.kuali.student.common.ui.client.widgets.KSRichEditor | Line |
| implement i18n | 35 |
| org.kuali.student.common.ui.client.widgets.KSRichEditorAbstract | Line |
| implement with a clean toolbar and i18n | 27 |
| org.kuali.student.common.ui.client.widgets.field.layout.element.FieldElement | Line |
| Do a check here to change the type of label based on widget type eventually | 255 |
| show actual help window | 385 |
| does nothing on ok, ok is not currently used | 505 |
| better way to generate unique id here? | 587 |
| org.kuali.student.common.ui.client.widgets.field.layout.layouts.TableFieldLayout | Line |
| Auto-generated method stub | 118 |
| org.kuali.student.common.ui.client.widgets.field.layout.layouts.UnborderedHeadedLayout | Line |
| rename styles to remove references to course format and activity | 28 |
| org.kuali.student.common.ui.client.widgets.list.KSSelectedList | Line |
| ryan - THIS METHOD NEEDS JAVADOCS | 309 |
| org.kuali.student.common.ui.client.widgets.pagetable.GenericTableModel | Line |
| return dynamic size when loading with RPC | 188 |
| org.kuali.student.common.ui.client.widgets.search.KSPicker | Line |
| Reviewed in M6 cleanup, unknown: Update to also work with a KSLabelList that doesn't require pre-population of all list items | 581 |
| org.kuali.student.common.ui.client.widgets.search.SearchPanel | Line |
| pass search to the table | 326 |
| is this check needed here? probably. assuming string here | 331 |
| is this null check needed here? probably. assuming string here | 538 |
| make check more robust here/inserting params more robust do not pass to the search parameters that are empty | 539 |
| use message call here | 606 |
| Add handling of default value lists here | 614 |
| need to handle date and other types here, how they are converted for search, etc | 659 |
| need to handle date and other types here, how they are converted for search, etc | 693 |
| this should be a message key | 803 |
| is this null check needed here? probably. assuming string here | 867 |
| make check more robust here/inserting params more robust do not pass to the search parameters that are empty | 868 |
| org.kuali.student.common.ui.client.widgets.search.SearchResultsTable | Line |
| - there's a better way to do this | 132 |
| org.kuali.student.common.ui.client.widgets.searchtable.ResultRow | Line |
| Auto-generated method stub | 55 |
| org.kuali.student.common.ui.client.widgets.suggestbox.KSSuggestBox | Line |
| implement some form of focus handling for SuggestBox | 38 |
| ryan - THIS METHOD NEEDS JAVADOCS | 256 |
| org.kuali.student.common.ui.client.widgets.table.summary.SummaryTableRow | Line |
| Nina - Code Review - is it ok that i change to public... | 78 |
| org.kuali.student.common.ui.client.widgets.table.summary.SummaryTableSection | Line |
| handle custom widgets (multiplicity field widget initializer) | 235 |
| handle custom widgets (multiplicity field widget initializer) | 292 |
| org.kuali.student.common.ui.client.widgets.tabs.impl.KSTabPanelImpl | Line |
| Auto-generated method | 56 |
| Auto-generated method stub | 58 |
| Auto-generated method stub | 60 |
| Auto-generated method | 62 |
| Auto-generated method | 64 |
| Auto-generated method | 66 |
| org.kuali.student.common.ui.server.applicationstate.ApplicationStateManager | Line |
| The ApplicationStateManager needs to be reviewed as part of the code clean-up. | 19 |
| org.kuali.student.common.ui.server.gwt.AbstractDataService | Line |
| why do we have this reference in the base class???? | 50 |
| Igor : Why do we check for this when getting the data for programs? | 61 |
| org.kuali.student.common.ui.server.gwt.ExportDocumentDownload | Line |
| Nina how must we handle exceptions here?? do something here | 28 |
| org.kuali.student.common.ui.server.gwt.ExportGwtRpcServlet | Line |
| Nina do we want this as a system Property?? | 41 |
| org.kuali.student.common.ui.server.gwt.LogRpcGwtServlet | Line |
| Use sl4j for logging | 36 |
| org.kuali.student.common.ui.server.gwt.SecurityRpcGwtServlet | Line |
| Do we need to worry about permission details when checking by permission name | 112 |
| Need to determine if permission details are required. | 124 |
| Need to determine if permission details are required. | 149 |
| org.kuali.student.common.ui.shared.IdAttributes | Line |
| OBJECT_ID has no references | 10 |
| org.kuali.student.common.ui.theme.standard.client.CommonImagesImpl | Line |
| Auto-generated method stub | 97 |
| org.kuali.student.common.validator.BeanConstraintDataProvider | Line |
| fix it later. | 42 |
| org.kuali.student.common.validator.DefaultValidatorImpl | Line |
| parse fieldname from here | 358 |
| What happens when the field is not in the dataProvider? | 408 |
| There needs to be some mapping from PropertyUtils to the KS path Until then, this will only work for root level properties | 415 |
| figure out what data should go here instead of null | 545 |
| Implement lookup constraint | 554 |
| org.kuali.student.common.validator.old.BeanConstraintDataProvider | Line |
| fix it later. | 36 |
| Should not be ignoring exception | 53 |
| org.kuali.student.common.validator.old.Validator | Line |
| Change this to 'default' when the change is made in xml | 47 |
| Setup a mechanism to retrive referenced object structures nestedObjStruct = setupFactory.getObjectStructure(field .getFieldDescriptor().getObjectStructureRef()); | 227 |
| What happens when the field is not in the dataProvider? | 546 |
| Allow different processing based on the label | 600 |
| Implement lookup constraint | 659 |
| Implement TypeStateCase constraint | 663 |
| org.kuali.student.core.assembly.transform.ProposalWorkflowFilter | Line |
| this needs to be defined as a constant where all references will resolve | 135 |
| this needs to be defined as a constant where all references will resolve | 140 |
| org.kuali.student.core.comments.ui.client.widgets.commenttool.CommentTool | Line |
| use user id for now change to user name | 338 |
| org.kuali.student.core.comments.ui.client.widgets.decisiontool.DecisionPanel | Line |
| Auto-generated method stub | 433 |
| org.kuali.student.core.dictionary.service.TestBaseDictionary | Line |
| fix reg ex so it allows 4.0 but excludes 4.1, 4.2 etc... assertNotNull (v.processValidCharConstraint ("test", vc, null, "4.1")); | 180 |
| maybe allow these special characters | 277 |
| maybe allow these special characters | 325 |
| org.kuali.student.core.document.service.DocumentService | Line |
| CHECK THESE VALUES | 52 |
| org.kuali.student.core.document.ui.client.widgets.documenttool.DocumentList | Line |
| Reviewed in M6, future fix: this will fail if the document does not exist BUT the relation does, needs a check for existance before delete | 152 |
| org.kuali.student.core.document.ui.client.widgets.documenttool.DocumentTool | Line |
| Auto-generated method stub | 648 |
| org.kuali.student.core.enumerationmanagement.service.impl.EnumerationManagementServiceImpl | Line |
| Kamal - THIS METHOD NEEDS JAVADOCS | 290 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 296 |
| org.kuali.student.core.organization.assembly.OrgOrgRelationAssembler | Line |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 63 |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 98 |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 118 |
| Auto-generated method stub | 293 |
| org.kuali.student.core.organization.assembly.OrgPersonRelationAssembler | Line |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 65 |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 71 |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 110 |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 234 |
| Auto-generated method stub | 240 |
| org.kuali.student.core.organization.assembly.OrgPositionRestrictionAssembler | Line |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 61 |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 67 |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 91 |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 110 |
| Auto-generated method stub | 222 |
| org.kuali.student.core.organization.assembly.OrgProposalAssembler | Line |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 146 |
| Neerav Agrawal - THIS METHOD NEEDS JAVADOCS | 262 |
| org.kuali.student.core.organization.dao.impl.OrganizationDaoImpl | Line |
| Check query, not sure if matching against right orgKeyType value | 191 |
| org.kuali.student.core.organization.service.impl.OrganizationAssembler | Line |
| Use the person Service here Person person = personService.findPerson(orgPersonRelationInfo.getPersonId()); if (person == null) { throw new InvalidParameterException( "Person does not exist for id: " + orgPersonRelationInfo.getPersonId()); } orgPersonRelation.setPersonId(person.getId()); | 346 |
| org.kuali.student.core.organization.service.impl.OrganizationServiceImpl | Line |
| Flush out exceptions | 342 |
| org.kuali.student.core.organization.ui.client.mvc.controller.OrgApplicationManager | Line |
| Need to revisit for Org possibly | 90 |
| org.kuali.student.core.organization.ui.client.mvc.view.CommonConfigurer | Line |
| - when DOL is pushed farther down into LOBuilder, revert these 5 methods to returning void again. | 191 |
| org.kuali.student.core.personsearch.service.impl.QuickViewByGivenName | Line |
| what is this search for? | 120 |
| finish sorting | 154 |
| Recognize alternate sort columns | 166 |
| org.kuali.student.core.statement.service.impl.StatementAssembler | Line |
| goes through the list of statements in statementTreeViewInfo and extract the statement ids | 533 |
| Fix with LuService RC1.4 changes LuStatementTypeInfo stmtType = this.luService.getLuStatementType(statementInfo.getStatementTypeId()); stmt.setLuStatementType(stmtType); | 585 |
| Fix with LuService RC1.4 changes ReqComponentType type = | 607 |
| Fix with LuService RC1.4 changes stmt.setLuStatementType(this.luService.getLuStatementType(luNlStmt.getStatementTypeId())); | 618 |
| org.kuali.student.core.statement.service.impl.TestStatementServiceImpl | Line |
| This should test valid ReqCompFieldInfo types and values, too | 1450 |
| org.kuali.student.core.statement.ui.client.widgets.rules.ReqCompEditWidget | Line |
| use app context for text | 78 |
| not sure if this will work all the time due to parallel nature of this code but running out of time | 222 |
| save history | 460 |
| we should mix the custom widgets with regular widgets | 463 |
| org.kuali.student.core.statement.ui.client.widgets.rules.RuleManageWidget | Line |
| use application context for all labels | 27 |
| org.kuali.student.core.statement.ui.client.widgets.rules.RulePreviewWidget | Line |
| app context for labels | 136 |
| org.kuali.student.core.statement.ui.client.widgets.rules.RuleTable | Line |
| do we need simple panel? | 34 |
| org.kuali.student.core.statement.ui.client.widgets.rules.RuleTableManipulationWidget | Line |
| remove req. compon.t | 299 |
| org.kuali.student.core.statement.ui.client.widgets.rules.RulesUtil | Line |
| clone file descriptor fields | 178 |
| move to ks-lum and use constants in ReqComponentFieldTypes.java | 223 |
| org.kuali.student.core.statement.ui.client.widgets.rules.StatementVO | Line |
| do we really need this and do we need to duplicate this in sub rules? | 35 |
| remove after refactoring rule table related classes, removing StatementVO | 40 |
| org.kuali.student.core.statement.ui.client.widgets.rules.Token | Line |
| use AndOrButton class int instead? | 20 |
| org.kuali.student.core.subjectcode.service.impl.MockSubjectCodeServiceImpl | Line |
| Auto-generated method stub | 21 |
| Auto-generated method stub | 29 |
| Auto-generated method stub | 38 |
| Auto-generated method stub | 47 |
| Auto-generated method stub | 54 |
| Auto-generated method stub | 62 |
| Auto-generated method stub | 69 |
| Auto-generated method stub | 78 |
| Auto-generated method stub | 85 |
| org.kuali.student.core.workflow.ui.client.views.CollaboratorSectionView | Line |
| Is there a way to get first, last name from picker w/o parsing display value | 406 |
| org.kuali.student.core.workflow.ui.client.widgets.WorkflowUtilities | Line |
| This should come from the ReferenceModel like it does in CommentTool | 93 |
| This static class should be moved to a utility at some point | 304 |
| currently causing a lot of duplicate calls and has to be used carefully. Commented out for now. | 1514 |
| org.kuali.student.lum.common.client.lo.CategoryDataUtil | Line |
| - doesn't work on the client; what to do? | 27 |
| - LoCategoryInfoAssembler, w/ an assemble method so we can just do categoriesData.add(LoCategoryInfoAssembler.assemble(cat)) instead of all the above | 46 |
| - this should't be necessary when DOL pushed down into LOPicker and its LOCategoryBuilder catInfo.setAttributes(catHelper.getAttributes()); | 69 |
| - LoCategoryInfoAssembler, w/ a disassemble method so we can just do categoriesData.add(LoCategoryInfoAssembler.disassemble(catData)) instead of all the above | 75 |
| org.kuali.student.lum.common.client.lo.LOCategoryBuilder | Line |
| show actual help window | 230 |
| do we need null checks? | 379 |
| - need to somehow ensure that categoryTypeMap is initialized before redraw() | 657 |
| - need to somehow ensure that categoryTypeMap is initialized before redraw() String typeName = "ERROR: uninitialized categoryTypeMap"; if (null != categoryTypeMap) { typeName = categoryTypeMap.get(typeKey).getName(); } categoryTable.setWidget(row, col++, new KSLabel(name + CATEGORY_TYPE_SEPARATOR + typeName)); KSLabel deleteLabel = new KSLabel("[x]"); deleteLabel.addStyleName("KS-LOBuilder-Search-Link"); deleteLabel.addClickHandler(deleteHandler); categoryTable.setWidget(row, col++, deleteLabel); row++; col = 0; | 666 |
| org.kuali.student.lum.common.client.lo.OutlineNode | Line |
| M4 - some way to subclass OutlineNode? | 27 |
| org.kuali.student.lum.common.server.LoCategoryDataService | Line |
| Check that only LO categories are coming through this way. LOs are persisted only in the context of a CLU? | 50 |
| org.kuali.student.lum.common.server.StatementUtil | Line |
| create a constants class that can be shared across view, server, etc. | 24 |
| create a constants class that can be shared across view, server, etc. | 33 |
| org.kuali.student.lum.course.service.assembler.CourseAssembler | Line |
| Use CluAssemblerUtils | 774 |
| Auto-generated method stub | 777 |
| This is pretty much a copy of the FormatAssembler's disassembleActivities code... maybe can be made generic | 959 |
| This is pretty much a copy of the disassembleJoints code... maybe can be made generic | 1142 |
| org.kuali.student.lum.course.service.impl.CourseDataGenerator | Line |
| make it return A, B, C... | 229 |
| org.kuali.student.lum.course.service.impl.TestCourseServiceImpl | Line |
| - check its contents | 170 |
| - check its contents | 175 |
| - check effective/expiration dates | 184 |
| - check feeInfo | 186 |
| - check joints | 188 |
| - check metaInfo | 189 |
| - check variotions | 217 |
| org.kuali.student.lum.lo.dao.impl.LoDaoImpl | Line |
| - will need more general logic here when we have relationships other than "includes" hopefully dictionary-driven List<Lo> parents = getIncludingLos(loId); for (Lo parent : parents) { parent.getRelatedLos().remove(child); update(parent); } | 101 |
| - gotta be a constant somewhere, or perhaps pull from dictionary | 116 |
| - gotta be a constant somewhere, or perhaps pull from dictionary | 123 |
| - "&& [not a top-level LO for another CLU]" when LO's are reused throw new DependentObjectsExistException("LoLoRelation(" + loLoRelationId + ") cannot be deleted without orphaning Lo(s)."); } | 191 |
| - remove when logic has been completely pilfered as appropriate | 238 |
| - null out hierarchy | 334 |
| org.kuali.student.lum.lu.assembly.CluSetManagementAssembler | Line |
| Split out CluInfo assembly to its own class | 53 |
| Auto-generated method stub | 518 |
| org.kuali.student.lum.lu.service.impl.LuServiceAssembler | Line |
| dto.setCluCriteria() | 315 |
| org.kuali.student.lum.lu.service.impl.LuServiceImpl | Line |
| is the is the best method of doing this? what if the user passes in a new made up id, does that mean we have orphaned richtexts? | 1310 |
| maybe this is unnecessary, contract specifies not null | 1692 |
| maybe this is unnecessary, contract specifies not null | 1738 |
| Auto-generated method stub | 2262 |
| move all of these procedural custom searches to a new search manager | 3514 |
| Is it possible we need to search up the cluset hierarchies? If Cluset A contains clu 1 and cluset B contains cluset A, do we also return cluset B as a dependency? | 3551 |
| check if this should be set from outside | 3968 |
| org.kuali.student.lum.lu.ui.course.client.configuration.CourseProposalConfigurer | Line |
| people and permissions will use a different button than continue | 198 |
| customize multiplicity and change "Percentage" label into LUConstants.AMOUNT | 1063 |
| customize multiplicity and change "Percentage" label into LUConstants.AMOUNT | 1085 |
| org.kuali.student.lum.lu.ui.course.client.configuration.CourseSummaryConfigurer | Line |
| next 3 methods below should be moved into some kind of multiplicity helper class | 1001 |
| Auto-generated method stub | 1223 |
| org.kuali.student.lum.lu.ui.course.client.controllers.CourseProposalController | Line |
| make msg | 163 |
| Is it possible to cut down on this validation so it doesn't have to validate every time. | 936 |
| Is this correct... do we want to stop view change if we can't restore the data? Possibly traps the user if we don't it messes up saves, possibly warn the user that it failed and continue? | 1183 |
| org.kuali.student.lum.lu.ui.course.client.requirements.CourseRequirementsManageView | Line |
| cache it for each statement type? | 117 |
| until we figure out how to detect changes, always return true | 236 |
| how to check whether rule changed or not? !(ruleManageWidget.getLogicExpression().equals(originalLogicExpression) && getAllReqCompNLs().equals(originalReqCompNL)); | 248 |
| ? | 290 |
| org.kuali.student.lum.lu.ui.course.client.requirements.CourseRequirementsViewController | Line |
| show dialog if user clicks on a menu from Manage Rules page | 93 |
| org.kuali.student.lum.lu.ui.course.client.widgets.CourseWorkflowActionList | Line |
| use messages | 99 |
| Inactivate | 133 |
| create Retire dialog | 166 |
| use message e.g. activateCurrentInstr, activateModificationInstr | 168 |
| create Inactivate dialog | 172 |
| message | 179 |
| message | 182 |
| use messages | 193 |
| add Retire and Inactivate Dialogs | 337 |
| org.kuali.student.lum.lu.ui.dependency.client.controllers.DependencyAnalysisController | Line |
| Auto-generated method stub | 43 |
| org.kuali.student.lum.lu.ui.dependency.client.views.DependencyAnalysisView | Line |
| This should not use hard-coded result columns | 264 |
| This code copied from program screens, need common reusable code | 380 |
| This code copied from Program summary, need to create reusable version for this and program summary | 530 |
| This code copied from Program summary, need to create reusable version for this and program summary | 542 |
| org.kuali.student.lum.lu.ui.dependency.client.widgets.DependencyResultPanel | Line |
| This may need to be replaced with document to surface handle print/export | 66 |
| org.kuali.student.lum.lu.ui.main.client.configuration.CurriculumHomeConfigurer | Line |
| Fix to improve performance, so permissions don't have to be loaded every time | 58 |
| org.kuali.student.lum.lu.ui.tools.client.configuration.CluSetsConfigurer | Line |
| add cluset and clurange here | 132 |
| change this to custom layoutcontroller here to support multiple model ids? | 226 |
| change this to custom layoutcontroller here to support multiple model ids? | 312 |
| retrieve cluset by id and | 324 |
| retrieve cluset by id and | 362 |
| - when DOL is pushed farther down into LOBuilder, revert these 5 methods to returning void again. | 544 |
| - when DOL is pushed farther down into LOBuilder, revert these 5 methods to returning void again. | 546 |
| org.kuali.student.lum.lu.ui.tools.client.widgets.SearchBackedTable | Line |
| use this as a token to get a message from message service instead | 149 |
| org.kuali.student.lum.program.client.major.MajorManager | Line |
| we really need to think about how this works rather than add these if/then cases, but we are out of time | 85 |
| we really need to think about how this works rather than add these if/then cases, but we are out of time | 161 |
| org.kuali.student.lum.program.client.major.edit.MajorEditController | Line |
| Error message are being sent back from the server but we do not have time to implement code to display them (we wanted to implement a lightbox for this). | 562 |
| org.kuali.student.lum.program.client.major.proposal.MajorProposalInformationEditConfiguration | Line |
| get title from ProgramProperties | 25 |
| org.kuali.student.lum.program.client.major.view.MajorViewController | Line |
| Change to program and copy msgs | 50 |
| retire is not implemented yet for program | 97 |
| PLEASE REVIEW. Should we be passing values from async calls to light boxes using instance variables like this? (we are doing this in course as well) | 352 |
| PLEASE REVIEW. If this async call runs slow, will the box remain visible? Is this an issue? Answer: Yes, it might be an issue, possible solution might to block user action w/progress bar until finished. | 385 |
| org.kuali.student.lum.program.client.requirements.ProgramRequirementsDataModel | Line |
| how can we reliably know that we need to reload rules (or not) String programId = (model == null ? null : (String)((DataModel)model).getRoot().get("id")); String modelProgramId = ((DataModel)modelIn).getRoot().get(ProgramConstants.ID); if ((modelProgramId == null) || (!modelProgramId.equals(programId))) { | 71 |
| clonedProgReqInfo.setAttributes(); | 462 |
| clonedProgReqInfo.setLearningObjectives(); | 463 |
| clonedProgReqInfo.setMetaInfo(); | 464 |
| org.kuali.student.lum.program.client.requirements.ProgramRequirementsManageView | Line |
| cache it for each statement type? | 98 |
| until we figure out how to detect changes, always return true | 227 |
| how to check whether rule changed or not? !(ruleManageWidget.getLogicExpression().equals(originalLogicExpression) && getAllReqCompNLs().equals(originalReqCompNL)); | 239 |
| ? | 281 |
| org.kuali.student.lum.program.client.requirements.ProgramRequirementsSummaryView | Line |
| rework to use Configurer if possible | 493 |
| org.kuali.student.lum.program.client.requirements.ProgramRequirementsViewController | Line |
| show dialog if user clicks on a menu from Manage Rules page | 100 |
| put back if we will NOT reset rules every time user comes to PREVIEW page above... | 142 |
| org.kuali.student.lum.program.server.CoreProgramDataService | Line |
| find a better way to get this search, param and resultcolumn names | 85 |
| org.kuali.student.lum.program.server.MajorDisciplineDataService | Line |
| Just Major Discipline for now - need to check for other types later | 35 |
| org.kuali.student.lum.program.server.MajorDisciplineProposalDataService | Line |
| Just Major Discipline for now - need to check for other types later | 47 |
| Auto-generated method stub | 145 |
| org.kuali.student.lum.program.server.MajorDisciplineProposalRpcServlet | Line |
| temporary fix - see KSLUM 1421 | 133 |
| org.kuali.student.lum.program.server.MajorDisciplineRpcServlet | Line |
| temporary fix - see KSLUM 1421 | 169 |
| org.kuali.student.lum.program.server.transform.CoreCredentialProgramFilter | Line |
| find a better way to get search, param and resultcolumn names | 66 |
| org.kuali.student.lum.program.service.ProgramService | Line |
| CHECK THESE VALUES | 56 |
| org.kuali.student.lum.program.service.assembler.MajorDisciplineAssembler | Line |
| - is it an error if there's more than one core program? | 97 |
| IDs for objects w/o ids | 135 |
| Check for ProgramAssemblerConstants.CERTIFICATE_RESULTS too | 219 |
| org.kuali.student.lum.program.service.assembler.ProgramAssemblerUtils | Line |
| maybe this should be in CluAssemblerUtils?? | 113 |
| maybe this should be in CluAssemblerUtils?? | 130 |
| - do we need to validate that? | 1076 |
| org.kuali.student.lum.program.service.impl.ProgramDataGeneratorUtils | Line |
| maybe need it later | 21 |
| org.kuali.student.lum.program.service.impl.ProgramServiceImpl | Line |
| Auto-generated method stub | 122 |
| find a place to put a shared StatementUtil | 241 |
| find a place to put a shared StatementUtil | 279 |
| Auto-generated method stub | 556 |
| Auto-generated method stub | 587 |
| Auto-generated method stub | 617 |
| Auto-generated method stub | 678 |
| Auto-generated method stub | 685 |
| Auto-generated method stub | 693 |
| Auto-generated method stub | 702 |
| Auto-generated method stub | 740 |
| Auto-generated method stub | 749 |
| Auto-generated method stub | 757 |
| Auto-generated method stub | 844 |
| Auto-generated method stub | 883 |
| Auto-generated method stub | 930 |
| Auto-generated method stub | 955 |
| Auto-generated method stub | 987 |
| Auto-generated method stub | 994 |
| Auto-generated method stub | 1002 |
| Auto-generated method stub | 1009 |
| Auto-generated method stub | 1017 |
| Auto-generated method stub | 1024 |
| Auto-generated method stub | 1033 |
| Auto-generated method stub | 1042 |
| Auto-generated method stub | 1049 |
| - when CRUD for a second ProgramInfo is implemented, pull common code up from its process*() and this | 1067 |
| org.kuali.student.lum.program.service.impl.TestProgramServiceImpl | Line |
| catalog descr | 221 |
| catalog pub targets | 222 |
| effectiveDate Calendar effectiveDate = GregorianCalendar.getInstance(); effectiveDate.set(1984, 7, 1, 0, 0, 0); Date testDate = new Date(effectiveDate.getTimeInMillis()); assertTrue(major.getEffectiveDate().compareTo(testDate) == 0); | 318 |
| catalog descr | 335 |
| createTime Calendar createTime = GregorianCalendar.getInstance(); createTime.set(2009, 4, 7, 12, 5, 36); testDate = new Date(createTime.getTimeInMillis()); assertTrue(major.getEffectiveDate().compareTo(testDate) == 0); | 399 |
| assertEquals("ANTH", createdMD.getCode()); | 493 |
| effectiveDate Calendar effectiveDate = GregorianCalendar.getInstance(); effectiveDate.set(1984, 7, 1, 0, 0, 0); Date testDate = new Date(effectiveDate.getTimeInMillis()); assertTrue(createdMD.getEffectiveDate().compareTo(testDate) == 0); | 524 |
| assertEquals("00f5f8c5-fff1-4c8b-92fc-789b891e0849", createdMD.getOrgCoreProgram().getId()); | 561 |
| createTime Calendar createTime = GregorianCalendar.getInstance(); createTime.set(2009, 4, 7, 12, 5, 36); testDate = new Date(createTime.getTimeInMillis()); assertTrue(createdMD.getEffectiveDate().compareTo(testDate) == 0); | 618 |
| checkReqComponentType(reqComponent.getRequiredComponentType(), reqComponent2.getRequiredComponentType()); | 754 |
| add version update | 1063 |
| add version update | 1179 |
| this should actually be passing; get working again after today's change of AdminOrgInfo's to those orgs ID's in Program-related DTOs assertEquals("testOrgId", target.getDivisionsContentOwner().get(0)); | 1291 |
| update versioning | 1415 |
| org.kuali.student.lum.service.assembler.CluAssemblerUtils | Line |
| Check this for proper handling of multiple CluResultInfos? If this is not a create, lookup the results for this clu | 84 |
| Auto-generated method stub | 190 |
| org.kuali.student.lum.statement.config.context.AtpContextImplTest | Line |
| Auto-generated method stub | 133 |
| Auto-generated method stub | 143 |
| Auto-generated method stub | 153 |
| Auto-generated method stub | 162 |
| Auto-generated method stub | 170 |
| Auto-generated method stub | 184 |
| Auto-generated method stub | 192 |
| Auto-generated method stub | 199 |
| Auto-generated method stub | 207 |
| Auto-generated method stub | 213 |
| Auto-generated method stub | 221 |
| Auto-generated method stub | 229 |
| Auto-generated method stub | 237 |
| Auto-generated method stub | 245 |
| Auto-generated method stub | 253 |
| Auto-generated method stub | 260 |
| Auto-generated method stub | 269 |
| Auto-generated method stub | 277 |
| Auto-generated method stub | 285 |
| Auto-generated method stub | 293 |
| Auto-generated method stub | 301 |
| Auto-generated method stub | 308 |
| Auto-generated method stub | 317 |
| Auto-generated method stub | 325 |
| Auto-generated method stub | 333 |
| Auto-generated method stub | 342 |
| Auto-generated method stub | 351 |
| Auto-generated method stub | 360 |
| Auto-generated method stub | 370 |
| Auto-generated method stub | 381 |
| Auto-generated method stub | 392 |
| Auto-generated method stub | 401 |
| Auto-generated method stub | 410 |
| Auto-generated method stub | 419 |
| Auto-generated method stub | 428 |
| Auto-generated method stub | 435 |
| Auto-generated method stub | 444 |
| Auto-generated method stub | 451 |
| Auto-generated method stub | 459 |
| Auto-generated method stub | 466 |
| Auto-generated method stub | 475 |
| Auto-generated method stub | 484 |
| Auto-generated method stub | 491 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 497 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 503 |
| org.kuali.student.lum.statement.config.context.LrcContextImplTest | Line |
| Auto-generated method stub | 207 |
| Auto-generated method stub | 219 |
| Auto-generated method stub | 228 |
| Auto-generated method stub | 236 |
| Auto-generated method stub | 245 |
| Auto-generated method stub | 253 |
| Auto-generated method stub | 260 |
| Auto-generated method stub | 269 |
| Auto-generated method stub | 277 |
| Auto-generated method stub | 285 |
| Auto-generated method stub | 293 |
| Auto-generated method stub | 300 |
| Auto-generated method stub | 308 |
| Auto-generated method stub | 316 |
| Auto-generated method stub | 324 |
| Auto-generated method stub | 332 |
| Auto-generated method stub | 339 |
| Auto-generated method stub | 347 |
| Auto-generated method stub | 355 |
| Auto-generated method stub | 371 |
| Auto-generated method stub | 388 |
| Auto-generated method stub | 402 |
| Auto-generated method stub | 410 |
| Auto-generated method stub | 422 |
| Auto-generated method stub | 431 |
| Auto-generated method stub | 438 |
| Auto-generated method stub | 447 |
| Auto-generated method stub | 454 |
| Auto-generated method stub | 462 |
| Auto-generated method stub | 469 |
| Auto-generated method stub | 478 |
| Auto-generated method stub | 487 |
| Auto-generated method stub | 494 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 500 |
| Kamal - THIS METHOD NEEDS JAVADOCS | 506 |
| org.kuali.student.lum.statement.config.context.LuContextImplTest | Line |
| Auto-generated method stub | 292 |
| Auto-generated method stub | 302 |
| Auto-generated method stub | 313 |
| Auto-generated method stub | 322 |
| Auto-generated method stub | 332 |
| Auto-generated method stub | 342 |
| Auto-generated method stub | 354 |
| Auto-generated method stub | 365 |
| Auto-generated method stub | 375 |
| Auto-generated method stub | 386 |
| Auto-generated method stub | 396 |
| Auto-generated method stub | 406 |
| Auto-generated method stub | 418 |
| Auto-generated method stub | 427 |
| Auto-generated method stub | 436 |
| Auto-generated method stub | 445 |
| Auto-generated method stub | 454 |
| Auto-generated method stub | 463 |
| Auto-generated method stub | 472 |
| Auto-generated method stub | 481 |
| Auto-generated method stub | 490 |
| Auto-generated method stub | 499 |
| Auto-generated method stub | 508 |
| Auto-generated method stub | 517 |
| Auto-generated method stub | 526 |
| Auto-generated method stub | 535 |
| Auto-generated method stub | 544 |
| Auto-generated method stub | 553 |
| Auto-generated method stub | 561 |
| Auto-generated method stub | 576 |
| Auto-generated method stub | 584 |
| Auto-generated method stub | 592 |
| Auto-generated method stub | 601 |
| Auto-generated method stub | 609 |
| Auto-generated method stub | 617 |
| Auto-generated method stub | 626 |
| Auto-generated method stub | 635 |
| Auto-generated method stub | 644 |
| Auto-generated method stub | 651 |
| Auto-generated method stub | 659 |
| Auto-generated method stub | 667 |
| Auto-generated method stub | 675 |
| Auto-generated method stub | 683 |
| Auto-generated method stub | 692 |
| Auto-generated method stub | 700 |
| Auto-generated method stub | 708 |
| Auto-generated method stub | 716 |
| Auto-generated method stub | 723 |
| Auto-generated method stub | 732 |
| Auto-generated method stub | 741 |
| Auto-generated method stub | 758 |
| Auto-generated method stub | 767 |
| Auto-generated method stub | 775 |
| Auto-generated method stub | 782 |
| Auto-generated method stub | 790 |
| Auto-generated method stub | 798 |
| Auto-generated method stub | 807 |
| Auto-generated method stub | 816 |
| Auto-generated method stub | 825 |
| Auto-generated method stub | 832 |
| Auto-generated method stub | 841 |
| Auto-generated method stub | 848 |
| Auto-generated method stub | 856 |
| Auto-generated method stub | 863 |
| Auto-generated method stub | 871 |
| Auto-generated method stub | 878 |
| Auto-generated method stub | 887 |
| Auto-generated method stub | 894 |
| Auto-generated method stub | 902 |
| Auto-generated method stub | 910 |
| Auto-generated method stub | 916 |
| Auto-generated method stub | 924 |
| Auto-generated method stub | 932 |
| Auto-generated method stub | 941 |
| Auto-generated method stub | 949 |
| Auto-generated method stub | 957 |
| Auto-generated method stub | 965 |
| Auto-generated method stub | 973 |
| Auto-generated method stub | 982 |
| Auto-generated method stub | 990 |
| Auto-generated method stub | 999 |
| Auto-generated method stub | 1008 |
| Auto-generated method stub | 1017 |
| Auto-generated method stub | 1026 |
| Auto-generated method stub | 1034 |
| Auto-generated method stub | 1042 |
| Auto-generated method stub | 1049 |
| Auto-generated method stub | 1058 |
| Auto-generated method stub | 1067 |
| Auto-generated method stub | 1076 |
| Auto-generated method stub | 1085 |
| Auto-generated method stub | 1095 |
| Auto-generated method stub | 1105 |
| Auto-generated method stub | 1116 |
| Auto-generated method stub | 1127 |
| Auto-generated method stub | 1138 |
| Auto-generated method stub | 1149 |
| Auto-generated method stub | 1160 |
| Auto-generated method stub | 1169 |
| Auto-generated method stub | 1179 |
| Auto-generated method stub | 1190 |
| Auto-generated method stub | 1199 |
| Auto-generated method stub | 1208 |
| Auto-generated method stub | 1217 |
| Auto-generated method stub | 1226 |
| Auto-generated method stub | 1235 |
| Auto-generated method stub | 1244 |
| Auto-generated method stub | 1253 |
| Auto-generated method stub | 1262 |
| Auto-generated method stub | 1271 |
| Auto-generated method stub | 1277 |
| Auto-generated method stub | 1283 |
| Auto-generated method stub | 1292 |
| Auto-generated method stub | 1299 |
| Auto-generated method stub | 1308 |
| Auto-generated method stub | 1315 |
| Auto-generated method stub | 1323 |
| Auto-generated method stub | 1330 |
| Auto-generated method stub | 1339 |
| Auto-generated method stub | 1348 |
| Auto-generated method stub | 1355 |
| Auto-generated method stub | 1365 |
| Auto-generated method stub | 1375 |
| Auto-generated method stub | 1395 |
| Auto-generated method stub | 1404 |
| Auto-generated method stub | 1413 |
| Auto-generated method stub | 1423 |
| Auto-generated method stub | 1432 |
| Auto-generated method stub | 1442 |
| org.kuali.student.lum.statement.config.context.OrganizationContextImplTest | Line |
| Auto-generated method stub | 124 |
| Auto-generated method stub | 136 |
| Auto-generated method stub | 148 |
| Auto-generated method stub | 157 |
| Auto-generated method stub | 166 |
| Auto-generated method stub | 174 |
| Auto-generated method stub | 182 |
| Auto-generated method stub | 191 |
| Auto-generated method stub | 200 |
| Auto-generated method stub | 207 |
| Auto-generated method stub | 215 |
| Auto-generated method stub | 224 |
| Auto-generated method stub | 233 |
| Auto-generated method stub | 240 |
| Auto-generated method stub | 249 |
| Auto-generated method stub | 258 |
| Auto-generated method stub | 268 |
| Auto-generated method stub | 277 |
| Auto-generated method stub | 286 |
| Auto-generated method stub | 295 |
| Auto-generated method stub | 304 |
| Auto-generated method stub | 311 |
| Auto-generated method stub | 320 |
| Auto-generated method stub | 330 |
| Auto-generated method stub | 339 |
| Auto-generated method stub | 348 |
| Auto-generated method stub | 358 |
| Auto-generated method stub | 366 |
| Auto-generated method stub | 372 |
| Auto-generated method stub | 389 |
| Auto-generated method stub | 398 |
| Auto-generated method stub | 408 |
| Auto-generated method stub | 417 |
| Auto-generated method stub | 426 |
| Auto-generated method stub | 435 |
| Auto-generated method stub | 444 |
| Auto-generated method stub | 453 |
| Auto-generated method stub | 462 |
| Auto-generated method stub | 473 |
| Auto-generated method stub | 485 |
| Auto-generated method stub | 495 |
| Auto-generated method stub | 507 |
| Auto-generated method stub | 516 |
| Auto-generated method stub | 525 |
| Auto-generated method stub | 535 |
| Auto-generated method stub | 545 |
| Auto-generated method stub | 551 |
| Auto-generated method stub | 557 |
| Auto-generated method stub | 566 |
| Auto-generated method stub | 573 |
| Auto-generated method stub | 582 |
| Auto-generated method stub | 589 |
| Auto-generated method stub | 597 |
| Auto-generated method stub | 604 |
| Auto-generated method stub | 613 |
| Auto-generated method stub | 622 |
| Auto-generated method stub | 629 |
| org.kuali.student.lum.ui.requirements.util.StatementServiceMock | Line |
| Auto-generated method stub | 357 |
| Auto-generated method stub | 366 |
| org.kuali.student.lum.workflow.CoursePostProcessorBase | Line |
| set states of other approved courses to superseded | 139 |
| We are not able to reuse the code in CourseStateUtil for dependency reason. | 177 |
| org.kuali.student.lum.workflow.ProgramPostProcessorBase | Line |
| Why is this method implemented in course post processor? it might be important for program as well | 35 |
| org.kuali.student.lum.workflow.StatementUtil | Line |
| create a constants class that can be shared across view, server, etc. | 24 |
| create a constants class that can be shared across view, server, etc. | 33 |
| org.kuali.student.lum.workflow.qualifierresolver.CocOrgTypeQualifierResolver | Line |
| The following annotation needs to be udpated on role definition. | 29 |