level2_concept_name condition duplicated

[jakewendt]

Just reading some code and I'm pretty sure that level2_concept_name is duplicated in src/main/resources/resources/cdmresults/sql/report/condition/treemap.sql. The second one should be level1_concept_name like in src/main/resources/resources/cdmresults/sql/report/conditionera/treemap.sql

  CONCAT(
    isNull(concept_hierarchy.level4_concept_name, 'NA'), '||',
    isNull(concept_hierarchy.level3_concept_name, 'NA'), '||',
    isNull(concept_hierarchy.level2_concept_name, 'NA'), '||',
    isNull(concept_hierarchy.level2_concept_name, 'NA'), '||',       <---- should be level1_concept_name
    isNull(concept_hierarchy.concept_name, 'NA')
  ) AS concept_path,

[chrisknoll]

I agree, that seems unlikely to be by design, but let me review the concept_hierarchy to see if there's something special about condition hierarchy that we'd want to not use level 1 (and if that was the case, I don't know why they just didn't omit the lvl 1 from the concatination (unless the front end expects 4 levels always). In any case, this is a good catch and let me investigate.