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

ConceptNumeric incorrectly copies collections

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Must
    • Resolution: Fixed
    • Affects Version/s: Platform 1.10.1
    • Fix Version/s: Platform 1.10.2
    • Component/s: Concept Dictionary
    • Labels:
      None
    • Complexity:
      Low

      Description

      The constructor of ConceptNumeric takes an existing concept and deep copies all the attributes.

      line number 110

      ConceptNumeric.java
      
      public ConceptNumeric(Concept c) {
      {
        ....    
        for (ConceptSet cSet : this.getConceptSets()) {
           cSet.setConcept(this);
        }
        .....
      }
      

      The idea, I assume, is to change the parent for the members of the given concept to the new one. However, the above code will set the new concept as member of the given concept.

      Please see discussion here: http://tinyurl.com/pfhzmu7

        Attachments

          Activity

            People

            • Assignee:
              angshuonline Angshuman Sarkar
              Reporter:
              angshuonline Angshuman Sarkar
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 4 hours
                4h
                Remaining:
                Remaining Estimate - 4 hours
                4h
                Logged:
                Time Spent - Not Specified
                Not Specified