Skip to content

Fix invalid UPC-A and EAN-13 barcodes#9

Merged
lfoust merged 1 commit into
TheDiscDb:mainfrom
ociaw:main
Sep 23, 2022
Merged

Fix invalid UPC-A and EAN-13 barcodes#9
lfoust merged 1 commit into
TheDiscDb:mainfrom
ociaw:main

Conversation

@ociaw
Copy link
Copy Markdown
Contributor

@ociaw ociaw commented Sep 17, 2022

These barcodes failed to pass validation as valid EAN-13 or UPC-A, and have been corrected. I verified each correction by either finding pictures of the release that show the barcode, or checking information available from online retailers.

Two releases contain questionable barcodes: "I, Robot (2004)" and "Rain Man (1988)". The barcodes on the packing of both releases have a suffix of "80" appended to the end of the UPC barcode. I cannot find a reason for this, as no relevant 14-digit barcode format exists and the original 12 digit UPC-A code identifies the releases correctly. Therefore I have simply removed the "80" suffix from each barcode.

These barcodes failed to pass validation, and have been corrected. Each correction
has been verified by either finding pictures of the release that show the barcode,
or by verifying the barcode with information available from online retailers.

Two releases contain questionable barcodes: "I, Robot (2004)" and "Rain Man (1988)".
The barcodes on the packing of both releases have a suffix of "80" appended to the
end of the UPC barcode. I cannot find a reason for this, as no relevant 14-digit
barcode format exists and the original 12 digit UPC-A code identifies the releases
correctly. Therefore I have simply removed the "80" suffix from each barcode.
@lfoust lfoust merged commit a48dd10 into TheDiscDb:main Sep 23, 2022
@lfoust
Copy link
Copy Markdown
Collaborator

lfoust commented Sep 23, 2022

Thanks for fixing all this! These corrections will go out with the next data deployment (in the next week or so) although I don't think the UPCs show up anywhere on the website currently so there may be no visible UI changes

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