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

AddPersonController needs to be refactored to not use instance fields

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Should
    • Resolution: Fixed
    • Affects Version/s: OpenMRS 1.6.5, OpenMRS 1.7.3, OpenMRS 1.8.3, OpenMRS 1.9.0, OpenMRS 1.9.7, Platform 1.10.0
    • Fix Version/s: OpenMRS 1.8.5, Platform 2.0.0
    • Component/s: None
    • Labels:
      None
    • Complexity:
      Low

      Description

      AddPersonController uses instance fields to store request parameters. This is a no-no because controllers are singletons.

        Gliffy Diagrams

          Attachments

            Attachments-Category-Modification

              Activity

                People

                • Assignee:
                  approce Roman Zayats
                  Reporter:
                  mogoodrich Mark Goodrich
                  Watchers:
                  ashish kumar, Ben Wolfe, chelsea komlo, Daniel Kayiwa, Gaurav Agrawal, Mark Goodrich, Roman Zayats, Shefali K, Stephen Lorenz
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  9 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Time Tracking

                    Estimated:
                    Original Estimate - 3 hours
                    3h
                    Remaining:
                    Time Spent - 2 hours Remaining Estimate - 1 hour
                    1h
                    Logged:
                    Time Spent - 2 hours Remaining Estimate - 1 hour
                    2h