Details

    • Rice Module:
      KEW

      Description

      tried using the kra org review template and creating my own template with a delegation template. in both cases, i created a rule, then tried to create the delegation. but, when i filled stuff out and tried to blank approve the delegation i got this, even though i had selected the rule i was delegating on to start with...
      Fatal Workflow Error has occured: org.springframework.dao.DataIntegrityViolationException: OJB operation; SQL []; ORA-01400: cannot insert NULL into ("KULREG"."EN_DLGN_RSP_T"."DLGN_RULE_BASE_VAL_ID") ; nested exception is java.sql.SQLException: ORA-01400: cannot insert NULL into ("KULREG"."EN_DLGN_RSP_T"."DLGN_RULE_BASE_VAL_ID") . Contact Us for Assistance

        Attachments

          Issue Links

            Activity

            Hide
            ewestfal Eric Westfall added a comment -

            Actually, to keep this consistent with the implementation of validateRoutingData, I moved the isRequired check to surround the validation portion of validateRuleData.

            Show
            ewestfal Eric Westfall added a comment - Actually, to keep this consistent with the implementation of validateRoutingData, I moved the isRequired check to surround the validation portion of validateRuleData.
            Hide
            ahamid Aaron Hamid (Inactive) added a comment -

            It seems like this should be guarded against in a consistent fashion if possible, or the column should just be set nullable. A constraint violation is a pretty heinous diagnostic. What do you think?

            Show
            ahamid Aaron Hamid (Inactive) added a comment - It seems like this should be guarded against in a consistent fashion if possible, or the column should just be set nullable. A constraint violation is a pretty heinous diagnostic. What do you think?
            Hide
            ewestfal Eric Westfall added a comment -

            This particular error should be resolved now on kfs-release-2-1-br. It did not require any changes in the Rice code so no rice/kew jars will need to be rebuilt.

            Show
            ewestfal Eric Westfall added a comment - This particular error should be resolved now on kfs-release-2-1-br. It did not require any changes in the Rice code so no rice/kew jars will need to be rebuilt.
            Hide
            ewestfal Eric Westfall added a comment -

            Responding to Aaron's last comment...

            The current implementation is terrible. I think the KEW code should be responsible for determining how to persist "empty" extension values and not assume that the originating code understands the quirks of the underlying data model. Incidentally, I think the whole WorkflowAttribute contract and the interactions within the system are way too complex. The perfect example of this is that it's assumed in the contract that when validateRuleData is called, that the local variables to hold the extension values will be stored in local variables on the attribute. Then, a subsequent call to getExtensionValues is used to extract the values from the attribute. WorkflowAttribute is currently way too stateful and the interface is beyond confusing. I've wanted to address this for awhile, I think I'll create a JIRA issue for 0.9.4

            Show
            ewestfal Eric Westfall added a comment - Responding to Aaron's last comment... The current implementation is terrible. I think the KEW code should be responsible for determining how to persist "empty" extension values and not assume that the originating code understands the quirks of the underlying data model. Incidentally, I think the whole WorkflowAttribute contract and the interactions within the system are way too complex. The perfect example of this is that it's assumed in the contract that when validateRuleData is called, that the local variables to hold the extension values will be stored in local variables on the attribute. Then, a subsequent call to getExtensionValues is used to extract the values from the attribute. WorkflowAttribute is currently way too stateful and the interface is beyond confusing. I've wanted to address this for awhile, I think I'll create a JIRA issue for 0.9.4
            Hide
            ewestfal Eric Westfall added a comment -

            Here's the issue I just created to attempt to address the issues which led to this problem: KULRICE-1342

            Show
            ewestfal Eric Westfall added a comment - Here's the issue I just created to attempt to address the issues which led to this problem: KULRICE-1342

              People

              • Assignee:
                ahamid Aaron Hamid (Inactive)
                Reporter:
                jkneal Jerry Neal (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Due:
                  Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 day, 2 hours
                  1d 2h