Skip to content

Conversation

@alinnert
Copy link

Checking the variable featString for null value.
If third parameter of window.open() is not used, the method fails at featString.Split(',') because featString is null.
This was the reason why window.open() with target _system did not work on Windows Phone.

Checking the variable "featString" for null value.
If third parameter of window.open() is not used, the method fails at featString.Split(',') because featString is null.
This was the reason why window.open() with target '_system' did not work on Windows Phone.
@purplecabbage
Copy link
Contributor

This has been addressed in JS code [1] ,
and is a native patch is also part of pull-req #38

Expect these changes to land in the next published version of IAB or plugman install from source.
This pull request should be closed.

Thanks,
Jesse

[1] bddf86c

@alinnert
Copy link
Author

Ah, I see. Then I use the mentioned solution. Thank you.

@alinnert alinnert closed this Apr 15, 2014
@alinnert alinnert deleted the patch-1 branch April 15, 2014 05:42
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