standard fix

vtfk · Sep 26, 2023 · 5584e65 · 5584e65
1 parent c56023e
commit 5584e65
Show file tree

Hide file tree

Showing 2 changed files with 41 additions and 45 deletions.
diff --git 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
@@ -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')
   }
 }