From 5584e65e6f39b783a604db708c768b71b3868bff Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=B8rgen=20Thorsnes?= <jorgen.thorsnes@gmail.com>
Date: Tue, 26 Sep 2023 14:31:17 +0200
Subject: [PATCH] standard fix

---
 lib/get-fint-data.js | 84 +++++++++++++++++++++-----------------------
 lib/get-pifu-data.js |  2 +-
 2 files changed, 41 insertions(+), 45 deletions(-)

diff --git a/lib/get-fint-data.js b/lib/get-fint-data.js
index dc2c99f..4221053 100644
--- a/lib/get-fint-data.js
+++ b/lib/get-fint-data.js
@@ -62,7 +62,7 @@ const getFaggrupper = async (feidenavn) => {
   }
   const fintData = await callFint(query)
 
-  if (!fintData.elev) throw { response: { status: 500, data: `Fant ingen elev med feidenavn ${feidenavn}` } }
+  if (!fintData.elev) throw Error(`Fant ingen elev med feidenavn ${feidenavn}`)
 
   // repack to match pifu data (dirty solution 👹)
 
@@ -108,50 +108,46 @@ const getFaggrupper = async (feidenavn) => {
 
   */
 
-  try {
-    let faggrupper = []
-    for (const elevforhold of fintData.elev.elevforhold) {
-      const repackedFag = elevforhold.faggruppemedlemskap.map(medlemskap => {
-        // repack to match pifu data
-        const kortnavn = elevforhold.skole.organisasjon.kortnavn.startsWith('OF-') ? elevforhold.skole.organisasjon.kortnavn.replace('OF-', '') : elevforhold.skole.organisasjon.kortnavn // Make it match pifu data
-        const faggruppe = medlemskap.faggruppe
-        const faggruppeInfo = {
-          id: faggruppe.systemId.identifikatorverdi,
-          type: 'faggruppe',
-          name: faggruppe.navn,
-          groupId: `${kortnavn}:${faggruppe.navn}`,
-          schoolId: elevforhold.skole.skolenummer.identifikatorverdi,
-          schoolName: elevforhold.skole.navn
-        }
-        const grep = {
-          kode: faggruppe.fag.systemId.identifikatorverdi,
-          tittel: {
-            default: faggruppe.fag.beskrivelse || faggruppe.fag.navn,
-            nob: faggruppe.fag.beskrivelse || faggruppe.fag.navn,
-            nno: faggruppe.fag.beskrivelse || faggruppe.fag.navn, // I am so sorry nynorsk
-            eng: faggruppe.fag.beskrivelse || faggruppe.fag.navn, // I am so sorry english
-            sme: faggruppe.fag.beskrivelse || faggruppe.fag.navn // I am so sorry samisk - i do not have the time... next version :(
-          },
-          kortform: {
-            default: faggruppe.fag.navn,
-            nob: faggruppe.fag.navn,
-            nno: faggruppe.fag.navn, // I am so sorry nynorsk
-            eng: faggruppe.fag.navn, // I am so sorry english
-            sme: faggruppe.fag.navn // I am so sorry samisk - i do not have the time... next version :(
-          },
-          data_url: Array.isArray(faggruppe.fag.grepreferanse) && faggruppe.fag.grepreferanse.length > 0 ? faggruppe.fag.grepreferanse[0] : 'https://erdetnoensombrukerdettefeltet.no/Whattttttdoyoumean' // Will there be errors here - time will show
-        }
-        return {
-          ...faggruppeInfo,
-          grep
-        }
-      })
-      faggrupper = [...faggrupper, ...repackedFag] // In case of several schools - we iterate elevforhold
-    }
-    return faggrupper
-  } catch (error) {
-    throw { response: { status: 500, data: error.stack || error.toString() } }
+  let faggrupper = []
+  for (const elevforhold of fintData.elev.elevforhold) {
+    const repackedFag = elevforhold.faggruppemedlemskap.map(medlemskap => {
+      // repack to match pifu data
+      const kortnavn = elevforhold.skole.organisasjon.kortnavn.startsWith('OF-') ? elevforhold.skole.organisasjon.kortnavn.replace('OF-', '') : elevforhold.skole.organisasjon.kortnavn // Make it match pifu data
+      const faggruppe = medlemskap.faggruppe
+      const faggruppeInfo = {
+        id: faggruppe.systemId.identifikatorverdi,
+        type: 'faggruppe',
+        name: faggruppe.navn,
+        groupId: `${kortnavn}:${faggruppe.navn}`,
+        schoolId: elevforhold.skole.skolenummer.identifikatorverdi,
+        schoolName: elevforhold.skole.navn
+      }
+      const grep = {
+        kode: faggruppe.fag.systemId.identifikatorverdi,
+        tittel: {
+          default: faggruppe.fag.beskrivelse || faggruppe.fag.navn,
+          nob: faggruppe.fag.beskrivelse || faggruppe.fag.navn,
+          nno: faggruppe.fag.beskrivelse || faggruppe.fag.navn, // I am so sorry nynorsk
+          eng: faggruppe.fag.beskrivelse || faggruppe.fag.navn, // I am so sorry english
+          sme: faggruppe.fag.beskrivelse || faggruppe.fag.navn // I am so sorry samisk - i do not have the time... next version :(
+        },
+        kortform: {
+          default: faggruppe.fag.navn,
+          nob: faggruppe.fag.navn,
+          nno: faggruppe.fag.navn, // I am so sorry nynorsk
+          eng: faggruppe.fag.navn, // I am so sorry english
+          sme: faggruppe.fag.navn // I am so sorry samisk - i do not have the time... next version :(
+        },
+        data_url: Array.isArray(faggruppe.fag.grepreferanse) && faggruppe.fag.grepreferanse.length > 0 ? faggruppe.fag.grepreferanse[0] : 'https://erdetnoensombrukerdettefeltet.no/Whattttttdoyoumean' // Will there be errors here - time will show
+      }
+      return {
+        ...faggruppeInfo,
+        grep
+      }
+    })
+    faggrupper = [...faggrupper, ...repackedFag] // In case of several schools - we iterate elevforhold
   }
+  return faggrupper
 }
 
 module.exports = { getFaggrupper }
diff --git a/lib/get-pifu-data.js b/lib/get-pifu-data.js
index a5e182c..ce52504 100644
--- a/lib/get-pifu-data.js
+++ b/lib/get-pifu-data.js
@@ -54,7 +54,7 @@ exports.getStudent = async (caller, id) => {
     const faggrupper = await getFaggrupper(`${id}@vtfk.no`)
     return student.map(student => { return { ...student, groups: [...classes, ...faggrupper] } })
   } catch (error) {
-    const { status, data } = error.response
+    const { status, data } = error.response || { status: 500, data: error.stack || error.toString() }
     throw new HTTPError(status || 500, data || 'Unknown error occured')
   }
 }