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

improve error handling in KualiAction.performLookup(..)

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.2
    • Fix Version/s: 1.0.3
    • Component/s: Development
    • Labels:
      None
    • Similar issues:
      KULRICE-10454Improve handling of the binding errors
      KULRICE-8558improve error handling in KualiDocumentActionBase.docHandler(..)
      KULRICE-9599Validation framework improvement for handling collections differently
      KULRICE-9238Improve approach for asset handling in KRAD
      KULRICE-2346Improve how the KSB handles "dead" services
      KULRICE-4177Improve error handling on person doc when checking permissions for adding roles
      KULRICE-12551Improvements on handling of template options
      KULRICE-3247Improve handling in KualiDecimal constructor
      KULRICE-4543Exception and incident handling framework improvements
      KULRICE-3672Need to improve the way Rice handles remote applications when they are not accessible.
    • Rice Module:
      Rice Core

      Description

      I would have been able to diagnose a problem easier if Rice had better error handling in one particular place. I would suggest the following changes.

      Replace
      LOG.debug( "lookupParams: " + lookupParams );
      with
      LOG.debug( "lookupParams: " + Arrays.toString(lookupParams) );

      and
      String[] keyValue = lookupParams[i].split(KNSConstants.FIELD_CONVERSION_PAIR_SEPARATOR);
      with
      String[] keyValue = lookupParams[i].split(KNSConstants.FIELD_CONVERSION_PAIR_SEPARATOR);
      if (keyValue.length != 2) throw new RuntimeException("malformed field conversion pair: " + Arrays.toString(keyValue));

        Activity

        Hide
        Garey Taylor added a comment -

        Added changes.

        Show
        Garey Taylor added a comment - Added changes.

          People

          • Assignee:
            Unassigned
            Reporter:
            Ken Geis
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Structure Helper Panel