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

Bugs while creating QRCode #1

Closed
lorimmar opened this issue Mar 6, 2014 · 5 comments
Closed

Bugs while creating QRCode #1

lorimmar opened this issue Mar 6, 2014 · 5 comments
Labels

Comments

@lorimmar
Copy link

lorimmar commented Mar 6, 2014

Hi Raffael
I use your library for printing QRCode on documents. Some of the codes are generated incorrectly. I give a few examples, and the parameters and values ​​with which they were noticed.
Encoded string: PA 9486/AMG/2014
ECC Level: M
Decoded as: PA 94870AMG/2014

Encoded string: PA 9490/AMG/2014
ECC Level: M
Decoded as: PA 949CFAMG/2014

Encoded string: PA 9491/AMG/2014
ECC Level: M
Decoded as: PA 949DFAMG/2014

@radbonk
Copy link

radbonk commented Mar 6, 2014

I have ran into a similar issue when generating codes with ECC Level: Q. To give an example, if I tried generating a URL, it would mix up the letter E with U.

@dcr26
Copy link

dcr26 commented Mar 8, 2014

Hi , I am having the same issues

The following urls are incoded incorrectly:
http://locahost/7A43CE -> scans to http://locahost/7a4#ce
http://locahost/TN7GCH -> scans to http://locahost/T^7GCH
http://locahost/TIN3PQ-> scans to http://locahost/TYN3PQ

I have a few more, and also one that just wont scan at all.

QRCodeGenerator.ECCLevel.M

@TheSeanLavery
Copy link
Contributor

@dcr26 I am unable to reproduce those results. Are you still having issues with the latest version of this library?

@obelixA
Copy link

obelixA commented Jul 5, 2014

HI

same here. Generating URLs containing GUIDs mixes up letters (d and t) which is a show stopper.
I.e.
9d566132-f75d-4665-8449-0570c838530e is decoded as
9t566132-f75d-4665-8449-0570c838530e
second letter t instead of d.

@codebude
Copy link
Owner

codebude commented Apr 6, 2015

There was an error in the CalculateECCWords-method. I forgot to remove the leading coefficients from the message polynoms if they are zero. It's fixed with Commit 8103f03

@codebude codebude closed this as completed Apr 6, 2015
@codebude codebude added the bug label Apr 7, 2015
codebude pushed a commit that referenced this issue Mar 11, 2016
Add exemple of multiple genration of Qrcode with safe memory manegement
codebude pushed a commit that referenced this issue Jan 27, 2019
Sync with codebude/QRCoder
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants