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

Move field org.openmrs.aop.RequiredDataAdvice.fieldAccess to right place and add classname

    XMLWordPrintable

    Details

    • Complexity:
      Medium

      Description

      put this somewhere and do it right and add class name, etc
      do this with an annotation on the field? or on the method?

      File: /openmrs/api/src/main/java/org/openmrs/aop/RequiredDataAdvice.java
      Line: 82
      Patch:

      Index: api/src/main/java/org/openmrs/aop/RequiredDataAdvice.java
      ===================================================================
      — api/src/main/java/org/openmrs/aop/RequiredDataAdvice.java (revision 26658)
      +++ api/src/main/java/org/openmrs/aop/RequiredDataAdvice.java (working copy)
      @@ -79,6 +79,7 @@
      */
      public class RequiredDataAdvice implements MethodBeforeAdvice {

      + // TODO Issue was added: TRUNK-3179
      // TODO put this somewhere and do it right and add class name, etc
      // TODO do this with an annotation on the field? or on the method?
      protected static List<String> fieldAccess = new ArrayList<String>();

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              lluismf Lluis Martinez
              Reporter:
              nullakilla Anton Alekseiev
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 2 hours
                  2h