Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-7571

Entering an document type qualifier on a Role document causes stack trace on add

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.1.2
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-9005On missing role qualifiers, rice throws stack trace instead of error message.
      KULRICE-7896NullPointerException if no role qualifier added on Person document
      KULRICE-7263Passing Collections.emptyMap() as role qualifier causes failures in KIM
      KULRICE-7082KIM role document not handling missing qualifiers on role members
      KULRICE-12836Role qualifiers lost on role maintenance document
      KULRICE-7711Role qualifier validation being ignored when assigning a role as a member
      KULRICE-8477Role Maintenance Document: Add ability to search on Role Qualifiers
      KULRICE-9165Person document not displaying role qualifiers
      KULRICE-9199Unable to inactivate roles of type unitHierarchy or unit from person maintenance docs
      KULRICE-4153Inactive qualifiers cause problems on existing roles
    • Rice Module:
      KIM
    • Application Requirement:
      KFS
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      Easy to reproduce on KFS.

      1) Edit role 28
      2) Enter khuntley in the Assignees Name field.
      3) Enter "sdfdsfds" in the Document Type Name field.
      4) Click the add button for the line.

      This results in a SOAP message stack trace on the Rice server. However, on the KFS server, this shows in the logs. It looks like it is having a problem converting the null document type returned into a DTO for sending back across the bus.

      Caused by: java.lang.NullPointerException
      	at org.kuali.rice.kew.doctype.bo.DocumentType.from(DocumentType.java:1684)
      	at org.kuali.rice.kew.service.impl.KEWModuleService.getExternalizableBusinessObject(KEWModuleService.java:76)
      	at org.kuali.rice.krad.service.impl.BusinessObjectServiceImpl.getReferenceIfExists(BusinessObjectServiceImpl.java:306)
      	at sun.reflect.GeneratedMethodAccessor3198.invoke(Unknown Source)
      	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
      	at java.lang.reflect.Method.invoke(Method.java:616)
      	at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:318)
      	at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:196)
      	at $Proxy64.getReferenceIfExists(Unknown Source)
      	at org.kuali.rice.krad.service.impl.DictionaryValidationServiceImpl.validateReferenceExists(DictionaryValidationServiceImpl.java:455)
      	at org.kuali.rice.krad.service.impl.DictionaryValidationServiceImpl.validateReferenceExistsAndIsActive(DictionaryValidationServiceImpl.java:626)
      	at org.kuali.rice.kns.kim.type.DataDictionaryTypeServiceBase.validateReferencesExistAndActive(DataDictionaryTypeServiceBase.java:291)
      	at org.kuali.rice.kns.kim.type.DataDictionaryTypeServiceBase.validateAttributes(DataDictionaryTypeServiceBase.java:176)
      	at sun.reflect.GeneratedMethodAccessor8403.invoke(Unknown Source)
      	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
      	at java.lang.reflect.Method.invoke(Method.java:616)
      	at org.apache.cxf.service.invoker.AbstractInvoker.performInvocation(AbstractInvoker.java:173)
      	at org.apache.cxf.service.invoker.AbstractInvoker.invoke(AbstractInvoker.java:89)
      	... 45 more
      

        Issue Links

          Activity

          Hide
          Jessica Coltrin (Inactive) added a comment -

          moving to 2.1.2

          Show
          Jessica Coltrin (Inactive) added a comment - moving to 2.1.2
          Hide
          Peter Giles (Inactive) added a comment -

          Added an integration test to prevent regression.

          Show
          Peter Giles (Inactive) added a comment - Added an integration test to prevent regression.

            People

            • Assignee:
              Peter Giles (Inactive)
              Reporter:
              Jonathan Keller
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Structure Helper Panel