Fix SUPReMM data warehouse export regression (ubccr#1520)

ryanrath · Mar 31, 2021 · df89701 · df89701
1 parent 1528398
commit df89701
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 2 deletions.
diff --git a/classes/DataWarehouse/Data/BatchDataset.php b/classes/DataWarehouse/Data/BatchDataset.php
@@ -202,7 +202,7 @@ private function getNextRow()
         $row = [];
 
         foreach ($this->fields as $field) {
-            $key = $field['name'];
+            $key = $field['alias'];
             $row[] = $field['anonymize']
                 ? $this->anonymizeField($rawRow[$key])
                 : $rawRow[$key];

diff --git a/classes/DataWarehouse/Data/RawStatisticsConfiguration.php b/classes/DataWarehouse/Data/RawStatisticsConfiguration.php
@@ -187,6 +187,11 @@ public function getBatchExportFieldDefinitions($realm)
         $fields = [];
 
         foreach ($this->getQueryFieldDefinitions($realm) as $field) {
+            // Skip "ignore" and "analysis" dtype
+            if (isset($field['dtype']) && in_array($field['dtype'], ['ignore', 'analysis'])) {
+                continue;
+            }
+
             $export = isset($field['batchExport']) ? $field['batchExport'] : false;
             if ($export === false) {
                 continue;
@@ -208,6 +213,7 @@ public function getBatchExportFieldDefinitions($realm)
 
             $fields[] = [
                 'name' => $field['name'],
+                'alias' => isset($field['alias']) ? $field['alias'] : $field['name'],
                 'display' => $display,
                 'anonymize' => ($export === 'anonymize'),
                 'documentation' => $field['documentation']

diff --git a/tests/artifacts/xdmod/schema/warehouse-export/GET-realms.schema.json b/tests/artifacts/xdmod/schema/warehouse-export/GET-realms.schema.json
@@ -51,6 +51,9 @@
                 "name": {
                     "type": "string"
                 },
+                "alias": {
+                    "type": "string"
+                },
                 "display": {
                     "type": "string"
                 },

diff --git a/tests/integration/lib/Rest/WarehouseExportControllerTest.php b/tests/integration/lib/Rest/WarehouseExportControllerTest.php
@@ -396,7 +396,6 @@ public function testDeleteRequests($role, $httpCode, $schema)
             $ids[] = $datum['id'];
         }
         $data = json_encode($ids);
-        //$this->assertTrue(false, json_encode($beforeContent['data']));
 
         // Delete all existing requests.
         list($content, $info, $headers) = self::$helpers[$role]->delete('rest/warehouse/export/requests', null, $data);