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!

        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