[
https://issues.apache.org/jira/browse/CB-12937?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16056328#comment-16056328
]
ASF GitHub Bot commented on CB-12937:
-------------------------------------
Github user shazron commented on a diff in the pull request:
https://github.com/apache/cordova-ios/pull/321#discussion_r123073237
--- Diff: CordovaLib/Classes/Public/CDVAppDelegate.m ---
@@ -78,9 +78,23 @@ - (BOOL)application:(UIApplication*)application
openURL:(NSURL*)url sourceApplic
if (!url) {
return NO;
}
+
+ NSMutableDictionary * openURLData = [[NSMutableDictionary alloc] init];
+
+ [openURLData setValue:url forKey:@"url"];
+
+ if (sourceApplication) {
+ [openURLData setValue:sourceApplication
forKey:@"sourceApplication"];
+ }
+
+ if (annotation) {
+ [openURLData setValue:annotation forKey:@"annotation"];
+ }
--- End diff --
Personally I would have used the dictionary literals here to be more clear:
```
NSMutableDictionary* openURLData = @{
@"url" : url,
@"sourceApplication": sourceApplication,
@"annotation": annotation
};
```
> [iOS] Add new CDVPlugin method to handle incoming urls including
> sourceApplication and annotation
> -------------------------------------------------------------------------------------------------
>
> Key: CB-12937
> URL: https://issues.apache.org/jira/browse/CB-12937
> Project: Apache Cordova
> Issue Type: Improvement
> Components: cordova-ios
> Affects Versions: cordova-ios@4.4.0
> Reporter: Georgios Galatoulas
> Assignee: Shazron Abdullah
> Labels: easy-fix
>
> The App is currently using the
> {code}- (BOOL)application:(UIApplication*)application openURL:(NSURL*)url
> sourceApplication:(NSString*)sourceApplication
> annotation:(id)annotation{code}
> to handle incoming URLs.
> However the notification posted with id CDVPluginHandleOpenURLNotification
> contains only the url.
> I would like to add alongside the current notification a new notification
> which posts a NSDictionary instead that contains the url, sourceApplication
> and annotation and if makes sense I would also like create a new method like
> the current handleOpenURL which will provide the NSDictionary to however
> wants to use.
> I propose this solution because it does not introduce any breaking changes. I
> hope the impact is small and easy to add to the next release. *The work is
> already done in my local fork I will just need to create a branch and ask for
> a pull request if everyone is happy to proceed*.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
