Uploaded image for project: 'OpenMRS Core'
  1. OpenMRS Core
  2. TRUNK-324

Go through the TODO items code and create tickets out of them

    XMLWordPrintable

    Details

    • Complexity:
      Low

      Description

      There are a good number of TODO items in the core code. They aren't doing much good in this state. These items should be examined and tickets made for them to be fixed.

      1. Do a global search for the word "TODO" in the api and web projects
      2. Add a ticket with as much description as possible. (file it was in, line number, and the TODO wording)
      3. Link those tickets to this ticket.
      4. Someone with more knowledge of the codebase can fill in more details around the ticket.

      If you're feeling frisky, go ahead and create pull requests for the tickets that you've made!

        Gliffy Diagrams

          Attachments

            Issue Links

            1.
            Need to test the null case Sub-task Closed Richard Meinsen  
            2.
            Convenience method to allow us to change the configuration more easily Sub-task Ready for Work Unassigned  
            3.
            Remove commented code about Throw an exception if there is no location to fall back on Sub-task Closed Yusuf Karim  
            4.
            Delete related objects before deleting the obs Sub-task Needs Assessment Unassigned  
            5.
            Remove code comment about Add changedBy and DateChanged columns to table patient_identifier_type Sub-task Closed tendo kiiza Martyn

            0%

            Original Estimate - 20 minutes
            Remaining Estimate - 20 minutes
            6.
            Copy instead of moving Sub-task Needs Assessment Unassigned  
            7.
            Remove TODO comment and deprecate method Sub-task Closed Jai Tatia  
            8.
            Possibly remove this method Sub-task Closed Unassigned  
            9.
            This ADT A28 handler does NOT currently handle ALL possible segments. Sub-task Ready for Work Unassigned  
            10.
            Remove code comment about Add a global property that enables different behavior here Sub-task Closed Yusuf Karim  
            11.
            Remove code comments about Fail if minOccurs is greater than 1 Sub-task Closed Piyush Kundnani  
            12.
            Remove code TODO about Should take care of the case where the user is using removable media Sub-task Closed Collin Grimm

            100%

            Original Estimate - Not Specified Original Estimate - Not Specified
            Time Spent - 20 minutes
            13.
            Get rid of hack that allows first component to be an integer Sub-task Code Review (Initial) Juliet Wamalwa  
            14.
            Remove TODO comments about Check patient has at least one "sufficient" identifier Sub-task Closed Collin Grimm

            100%

            Original Estimate - Not Specified Original Estimate - Not Specified
            Time Spent - 32 minutes
            15.
            Remove code comment about Eliminate unnecessary methods Sub-task Closed Collin Grimm

            100%

            Original Estimate - Not Specified Original Estimate - Not Specified
            Time Spent - 20 minutes
            16.
            Remove commented code about Re-write the hashcode() method Sub-task Closed tendo kiiza Martyn

            0%

            Original Estimate - 1 day, 2 hours
            Remaining Estimate - 1 day, 2 hours
            17.
            Copy instead of Moving Sub-task Needs Assessment Unassigned  
            18.
            Copy instead of moving Sub-task Needs Assessment Unassigned  

              Activity

                People

                Assignee:
                isaaclin isaac lin
                Reporter:
                bwolfe Ben Wolfe
                Votes:
                2 Vote for this issue
                Watchers:
                18 Start watching this issue

                  Dates

                  Created:
                  Updated:

                    Time Tracking

                    Estimated:
                    Original Estimate - 1 day, 2 hours, 20 minutes Original Estimate - 1 day, 2 hours, 20 minutes
                    1d 2h 20m
                    Remaining:
                    Time Spent - 2 days, 3 hours, 42 minutes Remaining Estimate - 1 day, 2 hours, 20 minutes
                    1d 2h 20m
                    Logged:
                    Time Spent - 2 days, 3 hours, 42 minutes Remaining Estimate - 1 day, 2 hours, 20 minutes
                    2d 3h 42m