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

XmlObjectSerializerServiceImpl has trouble w/ XStream 1.4.1 and CGLIB proxies

    Details

    • Type: Bug Fix
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: Backlog
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Labels:
    • Rice Module:
      KNS
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      I had put an attempted fix in on this issue previously due to these proxy serialization issues (See KULRICE-5566) but that created another problem w/ circular references when there were bidirectional proxied OJB relationships. The nature of the problems w/ XStream 1.4.1 seems to relate to the CGLIB support being removed (see http://xstream.codehaus.org/faq.html#Serialization_CGLIB), but my attempt at configuring XStream w/ the CGLIBMapper and CGLIBEnhancedConverter resulted in an exception that CGLIBMapper$Marker couldn't be instantiated as it doesn't have a no args constructor.

      To reproduce the problem (which I am thinly documenting here as I don't have things setup to repro it currently) pull in r21210 of XmlObjectSerializerServiceImpl.java and set the XStream version in the root pom to 1.4.1 – you'll also have to make a small change to SerializerServiceBase.java to deal with a renamed method (see line 105, serializableFieldsFor -> fieldsFor). Then:

      1. go to Rice portal Main tab
      2. Under KRMS Rules, click the Agenda Lookup link
      3. Do an empty search
      4. Click the "edit" link on My Fabulous Agenda
      5. Set the Campus to "BL"
      6. Click the Save button

        Attachments

          Issue Links

            Activity

            Hide
            jjhanso Jeremy Hanson added a comment -

            We should test this with xstream 1.4.2. That came out in November of 2011.

            Show
            jjhanso Jeremy Hanson added a comment - We should test this with xstream 1.4.2. That came out in November of 2011.
            Hide
            peris Peri Subrahmanya added a comment -

            Jeremy - have you guys verified with xstream 1.4.2?

            Show
            peris Peri Subrahmanya added a comment - Jeremy - have you guys verified with xstream 1.4.2?
            Hide
            peris Peri Subrahmanya added a comment -

            Eric/Peter - We are running into a similar issue; We are using the latest XStream and create of a new maint. document is fine but when we try to edit throws an exception. The maint. doc is for a table that has a reference to another table.

            Any workaround?

            Show
            peris Peri Subrahmanya added a comment - Eric/Peter - We are running into a similar issue; We are using the latest XStream and create of a new maint. document is fine but when we try to edit throws an exception. The maint. doc is for a table that has a reference to another table. Any workaround?
            Hide
            gilesp Peter Giles (Inactive) added a comment -

            I'm not sure if we ever got around to trying out 1.4.2. Perhaps Sona did, I seem to recall her trying to update the XStream version for some reason. Sona, can you comment?

            Show
            gilesp Peter Giles (Inactive) added a comment - I'm not sure if we ever got around to trying out 1.4.2. Perhaps Sona did, I seem to recall her trying to update the XStream version for some reason. Sona, can you comment?
            Hide
            sonam Sona Sona (Inactive) added a comment -

            There was an issue related to RulesDelegationBo where I thought it was a problem with serialization of nested properties but that was not the case so I did not update the XStream version.

            Show
            sonam Sona Sona (Inactive) added a comment - There was an issue related to RulesDelegationBo where I thought it was a problem with serialization of nested properties but that was not the case so I did not update the XStream version.

              People

              • Assignee:
                ewestfal Eric Westfall
                Reporter:
                gilesp Peter Giles (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated: