Optional minor/major parameter about cordova-plugin-ibeacon HOT 7 CLOSED

Not at all. It should work smoothly if you pass in null for major and minor.

Are you sure you are on the latest version of the code?
Can you please elaborate on the error that is being thrown? Is it a Javascript error on an Objective-C one?

Peter

from cordova-plugin-ibeacon.

Well, to be honest, I didn't try to pass null as an argument. I just didn't pass the minor or major parameter at all.

from cordova-plugin-ibeacon.

I just tried to pass null with the following code:

var beacon = new window.IBeacon.CLBeaconRegion(uuid, null, null, identifier);

I now get the following error:

PS: When I omit the minor / major parameter a JS error gets thrown.

from cordova-plugin-ibeacon.

It was a bug indeed in the code. Huge thanks for the detailed explanation of the problem!
The issue should be fixed by 0e98d01 . Please let me know if it resolved the issue for you as well.

Regarding the PS:
If you are omitting it like this:
var beacon = new window.IBeacon.CLBeaconRegion(uuid);
Then it fails because the identifier is still a mandatory parameter, so the error being thrown is intended.
If that's not how you omitting the major/minor parameters, let's talk more about that case as well.

Peter

from cordova-plugin-ibeacon.

Well I think it would be worth a breaking API change and alter the order of the arguments to be really able to omit the minor/major params. Would be more convenient I guess.

from cordova-plugin-ibeacon.

Thanks for the suggestion. I reconsidered it and you are right.

The breaking change will be part of version 2.

from cordova-plugin-ibeacon.

Awesome! 👍

from cordova-plugin-ibeacon.