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

deprecated method Encounter.setProvider against 1.9 needs to be adjusted

    Details

    • Type: Bug
    • Status: Ready for Work
    • Priority: Must
    • Resolution: Unresolved
    • Affects Version/s: OpenMRS 1.9.0
    • Fix Version/s: 1.9.x
    • Component/s: None
    • Complexity:
      Low

      Description

      OpenMRS1.9

      If you pass deprecated method encounter.setProvider(User) a User that does not have a Provider object associated, this throws an error. This isn't good backwards-compatibility deprecation for module support.

      I'd propose that this method now create the User's Provider object if it doesn't exist on the assumption that if someone meant that a given User be the Encounter's provider, its probably OK to make that User a Provider.

      TODOs

      • Add a convenience method to ProviderService that takes a user and creates a provider record for them.
      • Confirm with Dave/Mark if they really need this to behave as Dave is describing, create a patch that does this only if you switch a global property to true (but have it default to false).
      • Create a Providers wiki page and with a groovy script that will show which of the users don't have provider records, and another script that will add provider records for those users

        Gliffy Diagrams

          Attachments

            Attachments-Category-Modification

              Activity

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  dthomas Dave Thomas
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  16 Start watching this issue

                  Dates

                  • Created:
                    Updated:

                    Time Tracking

                    Estimated:
                    Original Estimate - Not Specified
                    Not Specified
                    Remaining:
                    Remaining Estimate - 0 minutes
                    0m
                    Logged:
                    Time Spent - 1 day, 1 hour
                    1d 1h