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

Add new barcode type C128RAW #1124

Merged
merged 1 commit into from
Nov 21, 2019
Merged

Add new barcode type C128RAW #1124

merged 1 commit into from
Nov 21, 2019

Conversation

dezsi-istvan
Copy link

Added new barcode type C128RAW.
Needed to generate advanced barcodes.
For example: compressed alfa-numeric barcodes with subtype change in middle of barcode
"A1234567890" is possible to code only in 128A or 128B
<barcode code="A1234567890" type="C128A"/>
with RAW type we can change the subtype after letter A
<barcode code="103 33 99 12 34 56 78 90" type="C128RAW"/>
103 - start 128A
33 - "A"
99 - change to 128C
12 - "12" ...
check digit and stop are added automaticaly by library
same result with ~30% shorter barcode
example
more info https://en.wikipedia.org/wiki/Code_128

@finwe
Copy link
Member

finwe commented Nov 20, 2019

Hi, thanks for the PR. Please fix lint errors fot it to be merged. Also, add a unit test covering the edge cases of the new code branch.

@dezsi-istvan
Copy link
Author

Tests added, and correct some discovered problems in subtype A and C

@finwe
Copy link
Member

finwe commented Nov 21, 2019

Still some lint errors. When fixing, feel free to add your handle and the pull number to the changelog and then squash the commits to one. I'll be happy to merge this when done.

@dezsi-istvan
Copy link
Author

If everything is fine please merge.

@finwe finwe merged commit 53f254b into mpdf:development Nov 21, 2019
@finwe
Copy link
Member

finwe commented Nov 21, 2019

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants