Re: [Aerogear-users] cordova push plugin.xml namespace
Thanks Erik,
The latest commit should fix the permission problem I was seeing.
The plugin.xml file still has unneeded xmlns:android namespace attributes in elements on
lines 35, 38, and 39.
- Kelvin
------------------------------------------------------------------------------------
From: aerogear-users-bounces(a)lists.jboss.org
[mailto:aerogear-users-bounces@lists.jboss.org] On Behalf Of Erik Jan de Wit
Sent: Tuesday, July 07, 2015 2:17 AM
To: aerogear-users(a)lists.jboss.org
Subject: Re: [Aerogear-users] cordova push plugin.xml namespace
Right, good find! I've created a PR for
this https://github.com/aerogear/aerogear-cordova-push/pull/72 maybe you can try it out?
cordova plugin add https://github.com/edewit/aerogear-pushplugin-cordova\#namespace-fix
On Tue, Jul 7, 2015 at 2:59 AM, Kelvin Lunsford <kelvin(a)qantel.com> wrote:
Can someone please modify plugin.xml so the xmlns:android namespace attribute is in the
top-level element as specified in the plugin.xml documentation
(https://cordova.apache.org/docs/en/5.1.1/plugin_ref_spec.md.html), and take the
xmlns:android attributes out of the lower elements.
If the namespace is not changed problems will occur when installing the push plugin along
with any other plugin containing one of the push plugins android permissions (e.g.
cordova-plugin-contacts uses GET_ACCOUNTS, cordova-plugin-wakeuptimer uses WAKE_LOCK).
Cordova CLI can deal with the same android permission used by multiple plugins but will
install an invalid extra copy of a permission if it includes a xmlns:android attribute.
The android ACCESS_NETWORK_STATE permission was removed by aerogear in the latest release
because of AGCORDOVA-72. The cause of the AGCORDOVA-72 problem was most likely the
namespace problem. The permission should not have been removed.
- Kelvin
_______________________________________________
Aerogear-users mailing list
Aerogear-users(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/aerogear-users
--
Cheers,
Erik Jan