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

Determine if we can use joda-time as part of our api instead of built-in java date/time types

    Details

    • Type: Task
    • Status: Closed
    • Priority: Critical
    • Resolution: Complete
    • Affects Version/s: None
    • Fix Version/s: 2.0.0-m7, 2.0
    • Component/s: Analysis
    • Labels:
      None
    • Rice Module:
      Rice Core, KEW

      Description

      This was discussed in the KTI meeting on June 3, 2009. See notes here:

      https://test.kuali.org/confluence/x/ngCcCg

      Since we are targeting 1.1 for compatibility, if we want to make an API change we should be doing this for the 1.1 release.

      Mentioned in the KTI was that this involves more than just the service API as these sorts of things are referenced elsewhere throughout the KNS and other modules.

      So, what need to be done here is some analysis on the impact of such a change and then a formal enhancement request written up that can be presented to the KTi for review.

      In essence, this work will probably be to eliminate references to java.sql.Date and java.sql.Timestamp everywhere in the code (I can't, off the top of my head imagine why they would be needed). We need to verify that OJB/JPA can handle fields that are java.util.Date being mapped to the database.

        Attachments

          Issue Links

            Activity

            Hide
            ewestfal Eric Westfall added a comment -

            I think I've modified as much as I can. The InactivatableFromTo covers a lot of the remaining work though, so i separated that out into a separate jira (KULRICE-5396).

            Show
            ewestfal Eric Westfall added a comment - I think I've modified as much as I can. The InactivatableFromTo covers a lot of the remaining work though, so i separated that out into a separate jira ( KULRICE-5396 ).
            Hide
            ewestfal Eric Westfall added a comment -

            Created and linked the KRDOC issue. So i'm going to resolve this issue as incomplete since there is work remaining that was farmed out to another jira.

            Show
            ewestfal Eric Westfall added a comment - Created and linked the KRDOC issue. So i'm going to resolve this issue as incomplete since there is work remaining that was farmed out to another jira.
            Hide
            riceci Rice-CI User (Inactive) added a comment -

            Integrated in rice-trunk-nightly #123 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/123/)
            KULRICE-3166 - whoops, missed some issues in the krms integration tests from my last commit
            KULRICE-3166 - converted krms api and framework over to use joda-time
            KULRICE-3166 - created joda-time LocalDateAdapter
            KULRICE-3166 - integrated joda-time into the core apis

            Show
            riceci Rice-CI User (Inactive) added a comment - Integrated in rice-trunk-nightly #123 (See http://ci.rice.kuali.org/job/rice-trunk-nightly/123/ ) KULRICE-3166 - whoops, missed some issues in the krms integration tests from my last commit KULRICE-3166 - converted krms api and framework over to use joda-time KULRICE-3166 - created joda-time LocalDateAdapter KULRICE-3166 - integrated joda-time into the core apis
            Hide
            ewestfal Eric Westfall added a comment -

            Thought it more appropriate to resolve this issue as complete since this particular jira has been completed.

            Show
            ewestfal Eric Westfall added a comment - Thought it more appropriate to resolve this issue as complete since this particular jira has been completed.
            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:
                ewestfal Eric Westfall
                Reporter:
                ewestfal Eric Westfall
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

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