Skip to content
This repository has been archived by the owner on Aug 11, 2020. It is now read-only.

PDF 417 not work with binary encoded at iphone #519

Closed
jasj opened this issue Aug 2, 2017 · 8 comments
Closed

PDF 417 not work with binary encoded at iphone #519

jasj opened this issue Aug 2, 2017 · 8 comments
Labels

Comments

@jasj
Copy link

jasj commented Aug 2, 2017

Expected Behaviour

At iphone must to decode the binary code as is do it on android

Actual Behaviour

Android decode a lot of data but ios just the first few chars, when is binnary data(probably found some ends chars)

Reproduce Scenario (including but not limited to)

try to can costarican cardId as it:
cedulashort

Steps to Reproduce

scan code on android, then on iphone, first characters are equal in both operative systems but iphone has less length, in the picture bellow just 8 chats vs more then 100 with others card id even just 1 char

Platform and Version (eg. Android 5.0 or iOS 9.2.1)

iOS 10

(Android) What device vendor (e.g. Samsung, HTC, Sony...)

no problem on androin

Cordova CLI version and cordova platform version

PhoneGap (iOS / Android / Windows)
cli-6.5.0 (4.3.1 / 6.1.2 / 4.4.3)

Plugin version

af3494b commit

Sample Code that illustrates the problem

cordova.plugins.barcodeScanner.scan(
function (result) {
alert("We got a barcode\n" +
"Result: " + binaryAgent(result.text) + "\n" +
"Format: " + result.format + "\n" +
"Cancelled: " + result.cancelled);

			console.log((result.text.length))
  },
  function (error) {
      
  },
  {
      preferFrontCamera : false, // iOS and Android
      showFlipCameraButton : true, // iOS and Android
      showTorchButton : true, // iOS and Android
      torchOn: true, // Android, launch with the torch switched on (if available)
      saveHistory: false ,// Android, save scan history (default false)
      prompt : "Ubique la parte detraz de la cédula", // Android
      resultDisplayDuration: 3500, // Android, display scanned text for X ms. 0 suppresses it entirely, default 1500
      formats : "PDF_417", // default: all but PDF_417 and RSS_EXPANDED
      orientation : "landscape", // Android only (portrait|landscape), default unset so it rotates with the device

  }

);

Logs taken while reproducing problem

@macdonst macdonst added the ios label Aug 9, 2017
@andersborgabiro
Copy link

andersborgabiro commented Aug 26, 2017

It might be due to this #532.

The image seems to be based on a low-quality, low-resolution and low-contrast JPEG, and it's not complete: It's cut at the bottom. This image shouldn't be readable.

Can you submit a better and complete image?

@jasj
Copy link
Author

jasj commented Oct 10, 2017

i fond this implementation but its is not an cordova pluggin

https://github.com/TheLevelUp/ZXingObjC

@jasj
Copy link
Author

jasj commented Oct 10, 2017

pdfluiout

its works greate in android but not in iphone

@andersborgabiro
Copy link

That's extremely detailed. I can't read it with a new Android phone either. This is no doubt close to or even beyond the resolution limit of video preview.

@jasj
Copy link
Author

jasj commented Oct 30, 2017

I think that the
20171030_132322
problem is with binary encode on pdf417 at iphone

@andersborgabiro
Copy link

Check the Zxing documentation for whether it supports binary data for iOS.

https://github.com/zxing/zxing

@jasj
Copy link
Author

jasj commented Nov 3, 2017 via email

@lock
Copy link

lock bot commented Jun 7, 2018

This thread has been automatically locked.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests

3 participants