Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-5488

Conditional properties using @{} syntax cause views to blow up when ReloadingDataDictionary reloads a view

    Details

    • Type: Bug Fix
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0-b1, 2.0
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KRAD
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      To reproduce:

      1. Configure the reloading DD as per https://wiki.kuali.org/x/by6REg
      2. From main portal page open 'Test View 1' – it will load fine
      3. Make a trivial whitespace change in Travel-testView1.xml to get it to be reloaded
      4. Observe in the server log that it did indeed reload – look for something like: INFO org.kuali.rice.krad.datadictionary.DataDictionary - reloading dictionary configuration for Travel-testView1.xml
      5. go back to the main portal page and open 'Test View 1'.

      Expected result: it opens again as before
      Actual result: incident report

      Here's the bottom-most 'Caused by':

      Caused by: java.lang.IllegalArgumentException: Invalid boolean value [@

      {#line?.subAccount eq 'i84n'}

      ]
      at org.springframework.beans.propertyeditors.CustomBooleanEditor.setAsText(CustomBooleanEditor.java:124)
      at org.springframework.beans.TypeConverterDelegate.doConvertTextValue(TypeConverterDelegate.java:416)
      at org.springframework.beans.TypeConverterDelegate.doConvertValue(TypeConverterDelegate.java:388)
      at org.springframework.beans.TypeConverterDelegate.convertIfNecessary(TypeConverterDelegate.java:157)
      at org.springframework.beans.BeanWrapperImpl.convertIfNecessary(BeanWrapperImpl.java:447)
      ... 90 more

        Attachments

          Activity

          Hide
          jcoltrin Jessica Coltrin (Inactive) added a comment -

          Fixed, according to Peter.

          Show
          jcoltrin Jessica Coltrin (Inactive) added a comment - Fixed, according to Peter.
          Hide
          jcoltrin Jessica Coltrin (Inactive) added a comment -

          Closing since these items are now in the release notes.

          Show
          jcoltrin Jessica Coltrin (Inactive) added a comment - Closing since these items are now in the release notes.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: