Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: 2.4
    • Component/s: Analysis
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KRAD
    • KRAD Feature Area:
      Lookup
    • Application Requirement:
      Rice
    • Sprint:
      2.4.0-m2 Sprint 2
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Include in Release Notes?:
      Yes

      Description

      Review LookupUtils to see if it's used by KRAD. If not move to KNS Utils class.

        Attachments

          Issue Links

            Activity

            cniesen Claus Niesen created issue -
            cniesen Claus Niesen made changes -
            Field Original Value New Value
            Remote Link This issue links to "Found by code review rice-269 (Web Link)" [ 25000 ]
            cniesen Claus Niesen made changes -
            Epic Link KULRICE-8800 [ 109084 ]
            cniesen Claus Niesen made changes -
            Rank Ranked higher
            cniesen Claus Niesen made changes -
            Assignee Martin Taylor [ mztaylor ]
            cniesen Claus Niesen made changes -
            Sprint 2.4.0-m2 Sprint 2 [ 63 ]
            cniesen Claus Niesen made changes -
            Rank Ranked lower
            mztaylor Martin Taylor (Inactive) made changes -
            Hide
            mztaylor Martin Taylor (Inactive) added a comment -

            Ran find usage on LookupUtils. Found 42 usages, 38 of which are in krad packages (see attached screenshot).

            LookupUtils calls in KRAD packages include:

            • LookupUtils#getActiveDateTimestampForCriteria
            • LookupUtils#removeExternalizableBusinessObjectFieldValues
            • LookupUtils#scrubQueryCharacters
            • LookupUtils#preprocessDateFields
            • LookupUtils#forceUppercase
            Show
            mztaylor Martin Taylor (Inactive) added a comment - Ran find usage on LookupUtils. Found 42 usages, 38 of which are in krad packages (see attached screenshot). LookupUtils calls in KRAD packages include: LookupUtils#getActiveDateTimestampForCriteria LookupUtils#removeExternalizableBusinessObjectFieldValues LookupUtils#scrubQueryCharacters LookupUtils#preprocessDateFields LookupUtils#forceUppercase
            mztaylor Martin Taylor (Inactive) made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            mztaylor Martin Taylor (Inactive) logged work - 13/Sep/13 11:37 AM
            • Time Spent:
              30 minutes
               
              - review ticket
              - review class and usage
              - gather details regarding usage
              - update ticket accordingly
            mztaylor Martin Taylor (Inactive) made changes -
            Status In Progress [ 3 ] Open [ 1 ]
            mztaylor Martin Taylor (Inactive) made changes -
            Remaining Estimate 3 hours [ 10800 ] 2 hours, 30 minutes [ 9000 ]
            Time Spent 30 minutes [ 1800 ]
            Worklog Id 88439 [ 88439 ]
            Hide
            mztaylor Martin Taylor (Inactive) added a comment -

            Reviewed LookupUtils, several calls are being made to krad packages therefore not moving to kns library.

            Show
            mztaylor Martin Taylor (Inactive) added a comment - Reviewed LookupUtils, several calls are being made to krad packages therefore not moving to kns library.
            mztaylor Martin Taylor (Inactive) made changes -
            Status Open [ 1 ] Resolved [ 5 ]
            Resolution Won't Fix [ 2 ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Fix Version/s 2.4.0-m2 [ 17036 ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Fix Version/s 2.4.0-m2 [ 17036 ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Status Resolved [ 5 ] Closed [ 6 ]

              People

              • Assignee:
                mztaylor Martin Taylor (Inactive)
                Reporter:
                cniesen Claus Niesen
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 3 hours
                  3h
                  Remaining:
                  Time Spent - 30 minutes Remaining Estimate - 2 hours, 30 minutes
                  2h 30m
                  Logged:
                  Time Spent - 30 minutes Remaining Estimate - 2 hours, 30 minutes
                  30m