Kan niet abonneren op eerste vermelding artikel

[robertwebbe]

Als ik bij onderstaand artikel 'Prijs volgen' aan klik, dan krijg ik de melding 'De CSRF-token is ongeldig. Probeer het formulier opnieuw te versturen.'. Ook na refresh van de pagina.

pricewatch: Ubiquiti UniFi Protect G3 Instant Camera

[Glaanie]

@robertwebbe Hmm hier hetzelfde. Bij de eerste klik op de button Prijsalert instellen gebeurt er niets, bij een tweede klik krijg je de CSRF foutmelding. Bij een willekeurig ander product zonder prijs, gebeurt hetzelfde. Ik verplaats dit topic naar Stoute Bugs

[DaFeliX]

Glaanie schreef op woensdag 17 maart 2021 @ 11:20:
[...] Ik verplaats dit topic naar Stoute Bugs

lekker makkelijk om het maar bij ons neer te gooien

Maar, het lijkt er inderdaad op dat de eerste klik een foutmelding geeft; waardoor er 'niets' lijkt te gebeuren. Dat ie dan de tweede keer een CSRF-foutmelding geeft is dan op zich wel goed.

We gaan het uitzoeken!

[Glaanie]

DaFeliX schreef op woensdag 17 maart 2021 @ 11:29:
[...]


lekker makkelijk om het maar bij ons neer te gooien
[...]

Wie de bug maakt, plette hem met de schoen. Of zo iets

[DaFeliX]

robertwebbe schreef op woensdag 17 maart 2021 @ 11:17:
Als ik bij onderstaand artikel 'Prijs volgen' aan klik, dan krijg ik de melding 'De CSRF-token is ongeldig. Probeer het formulier opnieuw te versturen.'. Ook na refresh van de pagina.

pricewatch: Ubiquiti UniFi Protect G3 Instant Camera

Deze bug heeft een schoenzool gevonden en is geplet; het werkt nu weer

We hebben onlangs de code hier aangepast, en o.a. typehints toegevoegd. Hierdoor dwingen we in de code af dat waardes die je meegeeft in een functie van een bepaalde type moet zijn. In dit geval is er een methode voor het aanmaken van een prijsalert dat de huidige prijs nodig heeft. We hebben hier de typehint voor ingesteld dat de huidige prijs een integer of float moet zijn. Maar, voor dit product was er nog geen prijs bekend, en is de huidige prijs dus null. null is geen integer of float, dus crashte het geheel.
Wat we hebben aangepast is dat we nu de huidige prijs op 0 zetten als er nog geen prijs bekend is, en het probleem is verholpen.

Thanks voor je melding

[Glaanie]

DaFeliX schreef op donderdag 18 maart 2021 @ 09:27:
[...]


Deze bug heeft een schoenzool gevonden en is geplet; het werkt nu weer

We hebben onlangs de code hier aangepast, en o.a. typehints toegevoegd. Hierdoor dwingen we in de code af dat waardes die je meegeeft in een functie van een bepaalde type moet zijn. In dit geval is er een methode voor het aanmaken van een prijsalert dat de huidige prijs nodig heeft. We hebben hier de typehint voor ingesteld dat de huidige prijs een integer of float moet zijn. Maar, voor dit product was er nog geen prijs bekend, en is de huidige prijs dus null. null is geen integer of float, dus crashte het geheel.
Wat we hebben aangepast is dat we nu de huidige prijs op 0 zetten als er nog geen prijs bekend is, en het probleem is verholpen.

Thanks voor je melding

Ik mis onwijs veel context, maar zoals het er nu staat lijk ik te lezen dat bij een "vanaf eerste prijsmelding" prijsalert voor een prijsloos product, de prijs van het betreffende product vanaf nu op 0 wordt gezet in de alert. Maar nu er technisch gezien een prijs is (immers niet null als in niets), gaat dat wel goed voor als er wel een echte prijs komt?

[DaFeliX]

Glaanie schreef op donderdag 18 maart 2021 @ 09:49:
[...]

Ik mis onwijs veel context, maar zoals het er nu staat lijk ik te lezen dat bij een "vanaf eerste prijsmelding" prijsalert voor een prijsloos product, de prijs van het betreffende product vanaf nu op 0 wordt gezet in de alert. Maar nu er technisch gezien een prijs is (immers niet null als in niets), gaat dat wel goed voor als er wel een echte prijs komt?

Voorheen werd er impliciet ook 0 ipv null doorgegeven, alleen gebeurde dat een stap later. Wat dat betreft is daar niets veranderd.

Op het moment dat er een prijs update plaatsvind, wordt er eerst gekeken of er een prijs bekend was; was er geen prijs bekend stuurt ie sowieso een prijsalert, ongeacht wat er bij de alert is ingesteld. Het maakt dus niet uit of deze 0, 1337, √-1 of 4294967296 is

[Glaanie]

DaFeliX schreef op donderdag 18 maart 2021 @ 09:58:
[...]


Voorheen werd er impliciet ook 0 ipv null doorgegeven, alleen gebeurde dat een stap later. Wat dat betreft is daar niets veranderd.

Op het moment dat er een prijs update plaatsvind, wordt er eerst gekeken of er een prijs bekend was; was er geen prijs bekend stuurt ie sowieso een prijsalert, ongeacht wat er bij de alert is ingesteld. Het maakt dus niet uit of deze 0, 1337, √-1 of 4294967296 is

Succes met het invullen van √-1 als er een int of float wordt verwacht

[robertwebbe]

Ik ben dan weer zo'n type die denkt: even proberen....

Deze waarde is niet geldig.

[DaFeliX]

robertwebbe schreef op donderdag 18 maart 2021 @ 15:51:
Ik ben dan weer zo'n type die denkt: even proberen....

Deze waarde is niet geldig.

heb je wel een getal ingevuld bij het bedrag? Mag geen tekst zijn (of kies "Vanaf de eerste prijsmelding" als optie)