Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
MIME-Version: 1.0
From: julio cesar sanchez <jcesarmobile@gmail.com>
Date: Sat, 16 Jul 2016 18:02:22 +0200
Message-ID: <CAG5ZNwX+Ls2x+2q1e1JwdYho8YF5uaLa+OGeBR6V2e6WzDF9sQ@mail.gmail.com>
Subject: opinion about a bug fix
To: "dev@cordova.apache.org" <dev@cordova.apache.org>
Content-Type: multipart/alternative; boundary=001a113d67da488a730537c2dc11
archived-at: Sat, 16 Jul 2016 16:02:27 -0000

--001a113d67da488a730537c2dc11
Content-Type: text/plain; charset=UTF-8

I'm looking into this issue https://issues.apache.org/jira/browse/CB-11178

The problem is on iOS 9 you can no longer query for schemes unless you add
them to the info.plist under the LSApplicationQueriesSchemes key.

When the inAppBrowser plugin is called with _system option this code is
called:

*- (**void**)openInSystem:(**NSURL***)url*

*{*

    *if** ([[**UIApplication* *sharedApplication**] **canOpenURL**:url]) {*

*        [[**UIApplication* *sharedApplication**] **openURL**:url];*

*    } **else** { **// handle any custom schemes to plugins*

*        [[**NSNotificationCenter* *defaultCenter**] **postNotification**:[*
*NSNotification* *notificationWithName**:*
*CDVPluginHandleOpenURLNotification* *object**:url]];*

*    }*

*}*


So, it fails on iOS 9 as fb scheme is not in the info.plist


I have 3 options here:


1. Document that on iOS 9 you have to add the urls to the info.plist

2. Document that you don't need inAppBrowser plugin to open a 3rd party
app, you just need to use an a tag element with the url scheme on the href

3. Change the code to

*- (**void**)openInSystem:(**NSURL***)url*

*{*

  *     [[**UIApplication* *sharedApplication**] **openURL**:url];*

*}*

This will open the app without querying, but we miss the else. I'm not sure
why that code is there or if it's used anywhere, it's been there since the
beginning of the github repo, it sends a notification with the url scheme
in case some other plugin can do something with the url scheme, but I don't
think it makes much sense as you are trying to open an external app and
that code is to handle the url inside the app.

So, opinions?

--001a113d67da488a730537c2dc11--