Moved array to a variable

NYCPlanning · Apr 23, 2024 · 8aadc56 · 8aadc56
1 parent 94cc054
commit 8aadc56
Showing 1 changed file with 11 additions and 17 deletions.
diff --git a/app/controllers/bookmarks.js b/app/controllers/bookmarks.js
@@ -4,6 +4,15 @@ import { inject as service } from '@ember/service';
 import { computed as computedProp } from '@ember/object';
 import { Promise } from 'rsvp';
 
+const QUERY_PARAMS_LIST = [
+  'selectedZoning',
+  'selectedOverlays',
+  'selectedFirm',
+  'selectedPfirm',
+  'selectedCouncilDistricts',
+  'selectedLayerGroup',
+];
+
 export default Controller.extend({
   mainMap: service(),
   metrics: service(),
@@ -60,15 +69,7 @@ export default Controller.extend({
       });
 
       const queryParams = {};
-      [
-        'layer-groups',
-        'selectedZoning',
-        'selectedOverlays',
-        'selectedFirm',
-        'selectedPfirm',
-        'selectedCouncilDistricts',
-        'selectedLayerGroup',
-      ].forEach((selected) => {
+      ['layer-groups', ...QUERY_PARAMS_LIST].forEach((selected) => {
         queryParams[selected] = this.router.currentRoute.queryParams[selected]
           ? JSON.parse(this.router.currentRoute.queryParams[selected])
           : undefined;
@@ -142,14 +143,7 @@ export default Controller.extend({
         });
       });
 
-      [
-        'selectedZoning',
-        'selectedOverlays',
-        'selectedFirm',
-        'selectedPfirm',
-        'selectedCouncilDistricts',
-        'selectedLayerGroup',
-      ].forEach((selected) => {
+      QUERY_PARAMS_LIST.forEach((selected) => {
         this.router.currentRoute.queryParams[selected] =
           layerToLoad.queryParams[selected];
       });