From 5ea7e78c9b36d2d280ee5fe9e723792cd567e3e5 Mon Sep 17 00:00:00 2001 From: Glenn Goossens <48956933+glgoose@users.noreply.github.com> Date: Mon, 14 Sep 2026 09:04:30 +0000 Subject: [PATCH] bipp-budgetcode: haal de optielijst via de postback als de GET faalt AnalyticalFieldSelector.aspx gooit een NullReferenceException in Page_Load zolang de sessie nog geen huidige regel heeft, dus de kale GET van loadOptions gaf HTTP 500 en alle dropdowns stonden disabled. Dat viel pas op toen de cache van zeven dagen verliep. loadOptions probeert nog steeds eerst die GET en valt bij een fout terug op de keten van saveCode zonder het opslaan: de SpecifyAF-postback, de lijst uit het antwoord, en Terug zodat de server weer op het winkelmandje staat en syncHidden de verse viewstate kan overzetten. Loopt door enqueue, want er mag er maar een tegelijk aan de huidige regel van de sessie zitten. Daarbij de foutmeldingen concreter: ze noemen nu de link bij naam en zeggen wat je moet doen, in plaats van "gebruik de link ernaast". De rijen staan tijdens het laden op bezig, en setState zet het grijs voortaan op zowel de select als de select-search-wrapper. Co-Authored-By: Claude Opus 5 --- scripts/bipp-budgetcode/README.md | 8 +++ .../bipp-budgetcode/bipp-budgetcode.user.js | 58 ++++++++++++++++--- 2 files changed, 58 insertions(+), 8 deletions(-) diff --git a/scripts/bipp-budgetcode/README.md b/scripts/bipp-budgetcode/README.md index 4506735..c8503a8 100644 --- a/scripts/bipp-budgetcode/README.md +++ b/scripts/bipp-budgetcode/README.md @@ -53,6 +53,9 @@ naar de selectorpagina, *Opslaan*, en *Terug*. Het script controleert in het ant server de waarde echt geselecteerd heeft en verwisselt daarna `__VIEWSTATE`, `__EVENTVALIDATION` en `__VIEWSTATEGENERATOR` in de live pagina, zodat een gewone postback erna blijft werken. +Dezelfde keten haalt ook de optielijst op als de selectorpagina niet rechtstreeks opvraagbaar +is, dan zonder *Opslaan*: de postback, meteen de lijst uit het antwoord, en *Terug*. + Omdat het live formulier meegestuurd wordt, gaat niet-opgeslagen typwerk (aantal, commentaar) niet verloren: dat wordt door de eerste stap zelfs meteen mee gecommit. Het verkeer per keuze is hetzelfde als dat van de drie handmatige stappen. @@ -67,6 +70,11 @@ hetzelfde als dat van de drie handmatige stappen. commentaarregel nog vasthouden: de eerste stap van de keten commit het formulier, de latere stap is degene die faalde. De eerstvolgende geslaagde keuze zet dat recht, want die stuurt het formulier opnieuw mee. +- **In een verse sessie kost het eerste laden drie verzoeken.** `AnalyticalFieldSelector.aspx` + geeft HTTP 500 (`NullReferenceException` in `Page_Load`) zolang de sessie nog geen huidige + regel heeft. Het script vangt dat zelf op met de keten hieronder, dus je merkt er enkel een + moment grijze dropdowns van. Lukt ook dat niet, dan zegt de melding wat je met de hand moet + doen: één keer op *Creëer* klikken en daarna op *Terug*. - **Meer dan één analytisch veld** wordt niet inline aangeboden. De selectorpagina is een repeater; verschijnt er ooit een tweede veld, dan valt het script voor die rij terug op de link. diff --git a/scripts/bipp-budgetcode/bipp-budgetcode.user.js b/scripts/bipp-budgetcode/bipp-budgetcode.user.js index e61f1d6..346b84c 100644 --- a/scripts/bipp-budgetcode/bipp-budgetcode.user.js +++ b/scripts/bipp-budgetcode/bipp-budgetcode.user.js @@ -1,7 +1,7 @@ // ==UserScript== // @name BIPP: budgetcode in het winkelmandje // @namespace https://github.com/glgoose/ua-userscripts -// @version 1.6.0 +// @version 1.7.0 // @description Zet de budgetcode als dropdown in de winkelmandjerij zelf, onder de grootboekrekening, zodat je niet meer via de aparte pagina Selectie analytische velden moet en de gekozen code ook gewoon ziet staan. Slaat op via een achtergrond-postback en zet de code ook in Interne commentaar. // @author Glenn Goossens // @license GPL-3.0-or-later @@ -96,8 +96,33 @@ return data; } + // De selectorpagina crasht serverzijdig zolang de sessie nog geen huidige regel heeft, en + // een kale GET geeft dan HTTP 500. De postback van SpecifyAF zet die regel wel, waarna + // dezelfde pagina gewoon de lijst geeft. Dit is dus de keten van saveCode zonder het + // opslaan, met Terug op het eind zodat de server weer op het winkelmandje staat en de + // viewstate in de live pagina klopt. + function loadViaSelector(target) { + return enqueue(function () { + var form = document.forms[0]; + return postForm(form.getAttribute('action') || location.pathname, + serialize(form, { '__EVENTTARGET': target, '__EVENTARGUMENT': '' }) + ).then(function (doc) { + var data = harvest(doc); + if (!data) throw new Error('optielijst niet gevonden'); + // De lijst is binnen. Lukt Terug niet, dan is dat spijtig voor de viewstate maar geen + // reden om de dropdown leeg te laten: de eerstvolgende keuze post het formulier toch + // opnieuw mee. + return postForm(SELECTOR_URL, serialize(doc.forms[0], { 'BackBtn': 'Terug' })) + .then(function (cart) { + if (cart.getElementById('resultTable')) syncHidden(cart); + return data; + })['catch'](function () { return data; }); + }); + }); + } + var loading = null; - function loadOptions() { + function loadOptions(target) { var cached = readCache(); if (cached) return Promise.resolve(cached); if (loading) return loading; @@ -110,6 +135,9 @@ var data = harvest(new DOMParser().parseFromString(t, 'text/html')); if (!data) throw new Error('optielijst niet gevonden'); return data; + })['catch'](function (err) { + if (!target) throw err; + return loadViaSelector(target); }); loading['catch'](function () { loading = null; }); return loading; @@ -354,6 +382,10 @@ return u.select; } + // De uitweg heet in de rij zelf Creëer of Bekijk, afhankelijk van wat er al gekozen is. + // Noem die tekst letterlijk, anders moet je zelf uitzoeken welke link bedoeld is. + function linkName(row) { return text(row.link) || 'Creëer'; } + function setState(row, state, message) { var u = ui(row); var target = skin(row); @@ -369,6 +401,12 @@ // legt een eigen invoerveld over de dropdown dat daar niet in meegaat. target.style.opacity = state === 'bezig' ? '0.5' : '1'; target.style.pointerEvents = state === 'bezig' ? 'none' : ''; + // select-search kan de wrapper tussen twee toestanden door aanmaken. Zet het grijs dus + // op allebei, anders blijft de select eronder half doorzichtig staan. + if (u.select && u.select !== target) { + u.select.style.opacity = target.style.opacity; + u.select.style.pointerEvents = target.style.pointerEvents; + } if (u.select) { u.select.style.outline = state === 'fout' ? '1.5px solid #c33' : ''; u.select.style.outlineOffset = state === 'fout' ? '-1px' : ''; @@ -489,7 +527,8 @@ setValue(row, before); row.savedCode = before; setState(row, 'fout', 'niet opgeslagen: ' + - (err && err.message ? err.message : 'fout') + '. Gebruik de link ernaast.'); + (err && err.message ? err.message : 'fout') + + '. Kies de code via ' + linkName(row) + ' hiernaast.'); }); }); } @@ -505,20 +544,22 @@ list.forEach(function (row) { row.savedCode = currentCode(row); renderSelect(row); - setState(row, 'rust', ''); + setState(row, 'bezig', 'budgetcodes laden…'); }); - loadOptions().then(function (data) { + loadOptions(list[0].target).then(function (data) { list.forEach(function (row) { var u = ui(row); // Meerdere analytische velden kunnen we niet met een enkele dropdown afhandelen, dan // laten we de vendor-flow staan. if (data.fields && data.fields.length > 1) { u.select.disabled = true; - setState(row, 'fout', 'meerdere analytische velden, gebruik de link ernaast'); + setState(row, 'fout', 'meerdere analytische velden: kies de code via ' + + linkName(row) + ' hiernaast.'); return; } fillOptions(u.select, data, row.savedCode); + setState(row, 'rust', ''); // Staat de code er al serverzijdig en nog niet in de commentaar, dan aanvullen. Dit kan // pas hier: shortCode heeft het label nodig en dat komt met de optielijst mee. if (row.savedCode) updateComment(row, row.savedCode); @@ -527,8 +568,9 @@ })['catch'](function (err) { list.forEach(function (row) { ui(row).select.disabled = true; - setState(row, 'fout', 'lijst niet geladen: ' + - (err && err.message ? err.message : 'fout') + '. Gebruik de link ernaast.'); + setState(row, 'fout', 'Lijst niet geladen (' + + (err && err.message ? err.message : 'fout') + '). Klik één keer op ' + + linkName(row) + ' hiernaast en daarna op Terug, dan laadt de lijst wel.'); }); }); }