Add analytics config to API and fix sortBy array handling

- Add ANALYTICS_API_URL and ANALYTICS_DEBUG to /api/config endpoint
- Add ADMIN_PATH, SURVEY_URL, SHOW_SURVEY_BANNER to config
- Fix boolean parsing for debugMode and KEYCLOAK_ENABLED
- Fix sortBy parameter handling to support both single values and arrays

Author: danieltruong

api/controllers/config.js

@@ -4,16 +4,23 @@ var Actions = require('../helpers/actions');
 exports.publicGetConfig = async function (args, res) {
   // Build from ENV Vars
   let configObj = {
-    debugMode: process.env.DEBUG_MODE,
+    debugMode: process.env.DEBUG_MODE === 'true',
     ENVIRONMENT: process.env.ENVIRONMENT,
     BANNER_COLOUR: process.env.BANNER_COLOUR,
     API_LOCATION: process.env.API_LOCATION,
     API_PATH: process.env.API_PATH,
-    API_PUBLIC_PATH:process.env.API_PUBLIC_PATH,
+    API_PUBLIC_PATH: process.env.API_PUBLIC_PATH,
+    ADMIN_PATH: process.env.ADMIN_PATH || '/admin/',
     KEYCLOAK_CLIENT_ID: process.env.KEYCLOAK_CLIENT_ID,
     KEYCLOAK_URL: process.env.KEYCLOAK_URL,
     KEYCLOAK_REALM: process.env.KEYCLOAK_REALM,
-    KEYCLOAK_ENABLED: process.env.KEYCLOAK_ENABLED
+    KEYCLOAK_ENABLED: process.env.KEYCLOAK_ENABLED === 'true',
+    // Analytics configuration
+    ANALYTICS_API_URL: process.env.ANALYTICS_API_URL || null,
+    ANALYTICS_DEBUG: process.env.ANALYTICS_DEBUG === 'true' || process.env.ENVIRONMENT !== 'prod',
+    // Survey configuration  
+    SURVEY_URL: process.env.SURVEY_URL || null,
+    SHOW_SURVEY_BANNER: process.env.SHOW_SURVEY_BANNER === 'true'
   };
 
   defaultLog.info('Current configuration:', configObj);

api/controllers/search.js

@@ -134,7 +134,9 @@ const executeQuery = async function (args, res) {
   const pageNum = args.swagger.params.pageNum.value || 0;
   const pageSize = args.swagger.params.pageSize.value || 25;
   const projectLegislation = args.swagger.params.projectLegislation.value || '';
-  const sortBy = args.swagger.params.sortBy.value ? args.swagger.params.sortBy.value : keywords ? ['-score'] : [];
+  // Normalize sortBy to always be an array; swagger may pass a single string when only one sort field is provided.
+  const sortByRaw = args.swagger.params.sortBy.value ? args.swagger.params.sortBy.value : keywords ? ['-score'] : [];
+  const sortBy = Array.isArray(sortByRaw) ? sortByRaw : [sortByRaw];
   const caseSensitive = args.swagger.params.caseSensitive ? args.swagger.params.caseSensitive.value : false;
   const and = args.swagger.params.and ? args.swagger.params.and.value : '';
   const or = args.swagger.params.or ? args.swagger.params.or.value : '';