Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed Double scan issue Cudi sale tool #926

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

wiskiv1
Copy link
Contributor

@wiskiv1 wiskiv1 commented Dec 4, 2024

Test deze oplossing in een Incognito Tabblad!!!
De js-code wordt ergens in de browser gecached dus wanneer je de sale tool open doet in een normaal tabblad nadat je iets hebt veranderd duurt het heel lang voordat deze aanpassingen worden weergegeven in de browser (bij mij was het pas aangepast de volgende dag!) (Cookies clearen helpt bij mij niet!) Beste is dus om gwn de sale tool open te doen in een incognito tabblad dan lijkt alles gwn te updaten zodra je de pagina herlaad

De code die wordt meegegeven aan de browser zit in "Litus/public/_assetic/sale_js.js" (dus niet in de CudiBundle) => Dit is een geminimaliseerde versie van verschillende sale tool scriptjes , momenteel is dit bestand READ-ONLY je moet dus best ook met chmod de lees en schrijf rechten van dit bestand aanpassen, anders worden je veranderingen in de andere scriptjes niet meegegeven aan de browser
sudo chmod 666 public/_assetic/sale_js.js
(standaard rechten zijn 644, voor moest je ze daarna willen terugbrengen naar hun originele staat)

Bij elke refresh van de browser wordt sale_js.js opnieuw gegenereerd vanuit de "bron scriptjes" wanneer je dus de schrijfrechten hebt aangepast kan je dus aanpassingen doen in de "bron scriptjes", je aanpassingen maken rechtstreeks in sale_js.js is dus nutteloos, want deze worden meteen overschreden wanneer sale_js.js opnieuw wordt gegenereerd

Het probleem zat em in saleInterface.js => _gotBarcode() => der is 2 keer dezelfde logica die de "add Article" knop zoekt en dan daarop klikt. waardoor er 2 keer een article wordt toegevoegd.

Op eerste zicht doen beide implementaties exact hetzelfde, maar ze doen het allebei op een ietsjes andere manier. ik gok dat dit is omdat één implementatie op zichzelf niet alle edge-cases vangt en het dus 2 keer wordt gedaan, maar ben niet een JS-expert dus kan het niet met zekerheid zeggen.

Nu is het zo dat als de eerste methode de "add Article" knop vindt en erop klikt de 2de methode niet meer wordt uitgevoerd. Dan heb je nog steeds de 2de methode als "safety" in het geval van edge cases.

@Tiddons Tiddons merged commit 2b3b760 into LitusProject:master Dec 4, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants