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

Update NfcPlugin.java #156

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

valemunoz
Copy link

modified to read Mifare Classic cards
example call

nfc.readMifare(success,error);
nfc.readMifareSecBloc(sector,block,success,error);

modified to read Mifare Classic cards
example call

nfc.readMifare(success,error);
nfc.readMifareSecBloc(sector,block,success,error);
@@ -70,7 +72,13 @@

private CallbackContext shareTagCallback;
private CallbackContext handoverCallback;

private static final String READMIFARE ="readMf";
private static final String READMIFARE_SB="readMf_SB";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix your indentation so it matches please!

@JohnMcLear
Copy link
Contributor

Yea you got this wrong, yous hould only have one pull erquest with your javascript and your java in.. Read about using git, it's worth understanding...

Read this too: http://nvie.com/posts/a-successful-git-branching-model/

@valemunoz
Copy link
Author

ups

@don
Copy link
Member

don commented Nov 18, 2014

I think returning the data as Uint8Array would be better than a String since the NFC tag can contain binary data. An array of int or byte would be OK too since that's more consistent with the other API calls in this plugin.

Can you make the catch blocks call the callback failure method where appropriate?

What if the user wants to authenticate with another key? Should the key be an optional parameter?

@valemunoz
Copy link
Author

the code only work for default key, but is very simple define a key. I code it and upload it if it works

@don
Copy link
Member

don commented Apr 2, 2015

@valemunoz sorry I haven't merged this yet, I'll try to test and merge in the next few weeks.

@valemunoz
Copy link
Author

OMG I completely forget

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.

None yet

3 participants