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

ionic.Platform.is() unexpected behavior #604

Closed
JonathanHindi opened this issue Feb 14, 2014 · 3 comments
Closed

ionic.Platform.is() unexpected behavior #604

JonathanHindi opened this issue Feb 14, 2014 · 3 comments
Assignees

Comments

@JonathanHindi
Copy link

From forums

I think there is something wrong with ionic.Platform.is(), org.apache.cordova.core.device is installed.
I tried the following
ionic.Platform.platforms => ["cordova", "ios", "ios7", "ios7"]
Then
ionic.Platform.is('cordova'); => false, Also note that ios7 is duplicated in the platform array, only ionic.Platform.is('ios') returns true. P.S: The above was run using weiner's console connected to a real iPhone 4. I am also facing some problems with ionic.Platform.ready() but I will check it separately and report back.

@adamdbradley adamdbradley self-assigned this Feb 17, 2014
@adamdbradley
Copy link
Contributor

Could you describe what you expect the ionic.Platform.is() to do?

These are the tests we currently have setup for the function:
https://github.com/driftyco/ionic/blob/master/js/ext/angular/test/service/ionicPlatform.unit.js

@JonathanHindi
Copy link
Author

I was expecting that it returns true on the cordova too, as it's available in the platforms array. Also note the duplicate in the array for ios7.

@ionitron-bot
Copy link

ionitron-bot bot commented Sep 6, 2018

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants