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

Deprecate PersistenceStructureService and related old metadata services

    Details

    • Type: Task
    • Status: Closed
    • Priority: Critical
    • Resolution: Complete
    • Affects Version/s: None
    • Fix Version/s: 2.4
    • Component/s: Development, JPA, Roadmap
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KNS, KRAD
    • KRAD Feature Area:
      Data Dictionary
    • Sprint:
      Sprint 2
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Include in Release Notes?:
      Yes

      Description

      Deprecate PersistenceStructureService and some of the other related pieces in preference for the new DataDictionary being built as part of the krad-data module.

        Attachments

          Activity

          Hide
          jkeller Jonathan Keller added a comment -

          Most of the core classes have been deprecated. Need to check where they were being used to see if there are methods on other services which were simple proxies to the persistence structure service.

          Show
          jkeller Jonathan Keller added a comment - Most of the core classes have been deprecated. Need to check where they were being used to see if there are methods on other services which were simple proxies to the persistence structure service.
          Hide
          jkeller Jonathan Keller added a comment -

          Would this include PBOB?

          Show
          jkeller Jonathan Keller added a comment - Would this include PBOB?
          Hide
          ewestfal Eric Westfall added a comment -

          Yes it would.

          Show
          ewestfal Eric Westfall added a comment - Yes it would.
          Hide
          ewestfal Eric Westfall added a comment -

          Given that though, we should discuss whether or not it makes sense to introduce a similar concept. I wish Java supported mixins, because then we could easily add behavior such as "GloballyUnique" and "Versioned". Instead what we would want them to do now is implement the GloballyUnique and Versioned interfaces. We could provide a "UniqueAndGloballyVersioned" parent class that they could extend from, but that seems like a hack.

          Show
          ewestfal Eric Westfall added a comment - Given that though, we should discuss whether or not it makes sense to introduce a similar concept. I wish Java supported mixins, because then we could easily add behavior such as "GloballyUnique" and "Versioned". Instead what we would want them to do now is implement the GloballyUnique and Versioned interfaces. We could provide a "UniqueAndGloballyVersioned" parent class that they could extend from, but that seems like a hack.
          Hide
          jkeller Jonathan Keller added a comment -

          So, we have to provide interfaces and a Support/Helper base class which they can use if desired.

          Show
          jkeller Jonathan Keller added a comment - So, we have to provide interfaces and a Support/Helper base class which they can use if desired.
          Hide
          jkeller Jonathan Keller added a comment -

          OK - I've gone through and removed all direct references which I can find to the Persistence[Structure]Service in code which is not exclusive to the KNS or the LegacyDataAdapter. Additional methods added to the LDA and DataObjectWrapper as needed.

          Show
          jkeller Jonathan Keller added a comment - OK - I've gone through and removed all direct references which I can find to the Persistence[Structure]Service in code which is not exclusive to the KNS or the LegacyDataAdapter. Additional methods added to the LDA and DataObjectWrapper as needed.

            People

            • Assignee:
              jkeller Jonathan Keller
              Reporter:
              ewestfal Eric Westfall
            • Votes:
              0 Vote for this issue
              Watchers:
              4 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