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

OmniKassa 2 bug OrderItems.name AN..max 50 #1

Closed
remcotolsma opened this issue Jan 18, 2019 · 6 comments
Closed

OmniKassa 2 bug OrderItems.name AN..max 50 #1

remcotolsma opened this issue Jan 18, 2019 · 6 comments
Assignees

Comments

@remcotolsma
Copy link
Member

Question Pronamic on 11 jan. 2019 10:40:

We hebben via een aantal van jullie OmniKassa 2 klanten een probleem ontdekt. Volgens de documentatie mag de OrderItems.name maximaal 50 tekens zijn (AN..max 50) Verder op in de documentatie staat hierover nog het volgende:

A.. Max nn
A field that consists of letters and other characters, such as ".", "@", etc. This field contains a maximum of nn characters.

We kwamen bijvoorbeeld een klant tegen met volgende product naam met 48 tekens:

W&N Artists Aquarel 692 Viridian (s3) - tube 5ml

OmniKassa 2 antwoord echter met de volgende foutmelding:

the item name is too long, maximum length is [50]

Na veel onderzoek zijn we tot de ontdekking gekomen dat het probleem hem in tekens zoals &, < en > zit. De volgende namen met 50 tekens werken namelijk niet:

  • 1234567890123456789012345678901234567890123456789&
  • 1234567890123456789012345678901234567890123456789<
  • 1234567890123456789012345678901234567890123456789>

En de volgende namen wel:

  • 123456789012345678901234567890123456789012345&
  • 1234567890123456789012345678901234567890123456<
  • 1234567890123456789012345678901234567890123456>

We vermoeden dat jullie verkeerd om gaan met de &, < en > tekens. Waarschijnlijk worden deze tekens omgezet naar de HTML-entities en daarna de lengte berekend:

  • & » &amp;
  • < » &lt;
  • > » &gt;

Het probleem is overigens niet beperkt tot bovenstaande tekens ook bij tekens die op bovenstaande tekens lijken gaat het fout. Het gaat dan bijvoorbeeld om de volgende tekens:

Het zou fijn zijn als dit snel opgelost kan worden, omdat we nu niet op een juiste manier de namen kunnen inkorten. We ontvangen graag even een bevestiging van dit probleem en horen graag wanneer dit opgelost is. Alvast bedankt.

@remcotolsma
Copy link
Member Author

Wesley Overhorst van Rabobank zegt op 17 jan. 2019 11:30:

Beste Remco,

Ik heb een antwoord vanuit ons development team gekregen en dat is onderstaande. Ik houd je op de hoogte van toekomstige wijzigingen.

Bedankt voor uw bericht. Uw analyse is correct. Wij kunnen helaas nog geen timeframe opgeven wanneer dit opgelost is.

Met vriendelijke groet / Kind regards,

Wesley Overhorst
Productspecialist
Operations

@remcotolsma
Copy link
Member Author

Remco Tolsma van Pronamic zegt op 17 jan. 2019 15:29:

Hoi Wesley,

Bedankt voor de update. We hadden gehoopt dat jullie hier inmiddels wel wat verder mee zouden zijn. Weet je ook waarom er nog geen timeframe gegeven kan worden? Of is het misschien wel duidelijk wanneer er wel een timeframe gegeven kan worden? Het zou wel fijn zijn als jullie dit iets concreter kunnen maken. Dat communiceert ook wat fijner richting onze en jullie klanten. We horen het graag!

Met vriendelijke groet,

Remco Tolsma

@remcotolsma remcotolsma self-assigned this Jan 18, 2019
@remcotolsma
Copy link
Member Author

Remco Tolsma van Pronamic zegt op 23 jan. 2019 14:47:

Hoi Wesley,

Inmiddels zijn we weer 6 dagen verder en hebben we geen reactie gekregen op voorgaand bericht. Ondertussen blijven er bij onze support afdeling klachten binnen komen over dit probleem. Kunnen jullie ook iets actiever zijn in het oplossen van dit probleem? Wij hebben hier inmiddels al aardig wat energie in gestoken. Kan het ook zijn dat er in de tussentijd al wel iets is gewijzigd aan jullie kant? De quickfix die wij hadden toegevoegd aan onze plugin lijkt namelijk ook niet meer te werken. In jullie documentatie spreken jullie over UTF-8 en daar rekenen wij dus ook mee qua lengte van de product namen e.d. De AN..max 50 limiet lijkt nu echter niet een limiet op aantal UTF-8 tekens, maar op het aantal bytes. De volgende tekst bestaat bijvoorbeeld uit 50 tekens, maar doordat het speciale tekens zijn uit 101 bytes:

ÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖ×ØÙÚÛÜÝÞßàáâãäåæçèéêëìíîïð…

Volgens mij gaan jullie hier nu ook niet (meer) goed mee om. Zou je ons vandaag in ieder geval kunnen informeren over wanneer we iets kunnen verwachten? We horen het graag!

Met vriendelijke groet,

Remco Tolsma

@remcotolsma
Copy link
Member Author

Wesley Overhorst van Rabobank zegt op 23 jan. 2019 16:03:

Hoi Remco,

Ik heb jouw vraag rechtsreeks bij het development team uitgezet en gevraagd of ze jou z.s.m. willen bellen.

Met vriendelijke groet / Kind regards,

Wesley Overhorst

@remcotolsma
Copy link
Member Author

Zojuist telefonisch contact gehad met de Rabobank, er wordt momenteel gewerkt aan een oplossing. In de loop van volgende week moet het probleem opgelost zijn binnen de productie omgeving. We worden op de hoogte gehouden van ontwikkelingen en zullen volgende week vrijdag testen of het probleem inderdaad is opgelost.

@remcotolsma
Copy link
Member Author

reuel [08:41]

Had je nog iets van OmniKassa vernomen of zal ik vandaag maar met een update aan de slag?

Remco [08:56]

Nee, geen update meer gekregen…
Ik krijg nu Invalid or missing signature

Remco [09:10]

Code weer even herschreven
probleem lijkt opgelost te zijn
producten met volgende titels lijken nu goed te werken
1234567890123456789012345678901234567890123456789⅋
❶❷❸❹❺❻❼❽❾❿❶❷❸❹❺❻❼❽❾❿❶❷❸❹❺❻❼❽❾❿❶❷❸❹❺❻❼❽❾❿❶❷❸❹❺❻❼❽❾❿
1234567890123456789012345678901234567890123456789⋗
1234567890123456789012345678901234567890123456789﹥
1234567890123456789012345678901234567890123456789&
&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&&
ÀÁÂÃÄÅÆÇÈÉÊËÌÍÎÏÐÑÒÓÔÕÖ×ØÙÚÛÜÝÞßàáâãäåæçèéêëìíîïðñ

cc77e80

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

No branches or pull requests

1 participant