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

Make sure Detailed Search save/restore works for Document Search

    Details

    • Type: Task Task
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0-b1, 2.0
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Similar issues:
      KULRICE-5630Make sure Document Search is working with ranged searches
      KULRICE-7483make doc search work like other lookups in terms of result limits
      KULRICE-2705The "detailed" link on the document search is not working
      KULRICE-5740Verify that super user document search is working properly after the doc search framework refactoring
      KULRICE-5911Rice Dev: SuperUser and Detailed Doc Search Not working in IE9 and Safari
      KULRICE-1212Saved searches from detailed search screen don't work
      KULRICE-7119Detailed document search will throw an RTE if no values are entered for search fields and tried to do a search..
      KULRICE-11664DocumentBase JPA versus OJB change regarding saving/restoring documentHeader
      KULRICE-6018Document the document search framework changes
      KULRICE-5751"Clear Named Searches" no longer works in document search
    • Rice Module:
      KEW
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      i think the saves works, just not the restore, but it's something to look at
      there was actually a weird issue where it looked like it was maybe entering more than one item in KREW_USR_OPTN_T for each save of a search
      i think i mention that in KULRICE-5056 somewhere
      but i did implement the persistence so that it serializes the DocumentLookupCritieria to XML using JAXB and then saves that as the value
      the only concern i have there is that i think we only have like 2000 or 4000 characters to store for the serialized value of the saved search, and XML isn't exactly the most compact format
      was thinking it be great to turn it into JSON instead but didn't want to go inserting new libraries just to deal with persistence of saved search values

        Issue Links

          Activity

          Hide
          Rice-CI User (Inactive) added a comment -

          Integrated in rice-trunk-nightly #221 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/221/)
          Merge branch 'KULRICE-5633'

          Show
          Rice-CI User (Inactive) added a comment - Integrated in rice-trunk-nightly #221 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/221/ ) Merge branch ' KULRICE-5633 ' KULRICE-5633 : KULRICE-5633 better coverage of doclookupcriteriatranslator
          Hide
          Rice-CI User (Inactive) added a comment -

          Integrated in rice-trunk-nightly #222 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/222/)
          Merge branch 'KULRICE-5633'

          Show
          Rice-CI User (Inactive) added a comment - Integrated in rice-trunk-nightly #222 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/222/ ) Merge branch ' KULRICE-5633 ' KULRICE-5633 : KULRICE-5633 converted doc search serialization to JSON
          Hide
          Aaron Hamid (Inactive) added a comment -

          Fixed. DocumentLookupCriteria are now saved as JSON thanks to the Jackson library already included by Rice.

          Show
          Aaron Hamid (Inactive) added a comment - Fixed. DocumentLookupCriteria are now saved as JSON thanks to the Jackson library already included by Rice.
          Hide
          Rice-CI User (Inactive) added a comment -

          Integrated in rice-trunk-nightly #223 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/223/)
          Merge branch 'KULRICE-5633'

          • KULRICE-5633:
            rename unused Groovy class; whitespace/charset fixes for GenericDaoOjb.java
            KULRICE-5633 added test for searching against doc attrs; omit null values from criteria json
          Show
          Rice-CI User (Inactive) added a comment - Integrated in rice-trunk-nightly #223 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/223/ ) Merge branch ' KULRICE-5633 ' KULRICE-5633 : rename unused Groovy class; whitespace/charset fixes for GenericDaoOjb.java KULRICE-5633 added test for searching against doc attrs; omit null values from criteria json
          Hide
          Jessica Coltrin (Inactive) added a comment -

          Closing since these items are now in the release notes.

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

            People

            • Assignee:
              Aaron Hamid (Inactive)
              Reporter:
              Jeremy Hanson
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 2 days
                2d
                Remaining:
                Remaining Estimate - 2 days
                2d
                Logged:
                Time Spent - Not Specified
                Not Specified

                  Structure Helper Panel