<div dir="ltr">fixed</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 7, 2015 at 11:43 PM, Kelvin Lunsford <span dir="ltr"><<a href="mailto:kelvin@qantel.com" target="_blank">kelvin@qantel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks Erik,<br>
<br>
The latest commit should fix the permission problem I was seeing.<br>
<br>
The plugin.xml file still has unneeded xmlns:android namespace attributes in elements on lines 35, 38, and 39.<br>
<br>
- Kelvin<br>
<br>
------------------------------------------------------------------------------------<br>
<br>
From: <a href="mailto:aerogear-users-bounces@lists.jboss.org">aerogear-users-bounces@lists.jboss.org</a> [mailto:<a href="mailto:aerogear-users-bounces@lists.jboss.org">aerogear-users-bounces@lists.jboss.org</a>] On Behalf Of Erik Jan de Wit<br>
Sent: Tuesday, July 07, 2015 2:17 AM<br>
To: <a href="mailto:aerogear-users@lists.jboss.org">aerogear-users@lists.jboss.org</a><br>
Subject: Re: [Aerogear-users] cordova push plugin.xml namespace<br>
<div class="HOEnZb"><div class="h5"><br>
Right, good find! I've created a PR for this <a href="https://github.com/aerogear/aerogear-cordova-push/pull/72" rel="noreferrer" target="_blank">https://github.com/aerogear/aerogear-cordova-push/pull/72</a> maybe you can try it out?<br>
<br>
cordova plugin add <a href="https://github.com/edewit/aerogear-pushplugin-cordova\#namespace-fix" rel="noreferrer" target="_blank">https://github.com/edewit/aerogear-pushplugin-cordova\#namespace-fix</a><br>
<br>
On Tue, Jul 7, 2015 at 2:59 AM, Kelvin Lunsford <<a href="mailto:kelvin@qantel.com">kelvin@qantel.com</a>> wrote:<br>
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 (<a href="https://cordova.apache.org/docs/en/5.1.1/plugin_ref_spec.md.html" rel="noreferrer" target="_blank">https://cordova.apache.org/docs/en/5.1.1/plugin_ref_spec.md.html</a>), and take the xmlns:android attributes out of the lower elements.<br>
<br>
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).<br>
<br>
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.<br>
<br>
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.<br>
<br>
- Kelvin<br>
<br>
<br>
_______________________________________________<br>
Aerogear-users mailing list<br>
<a href="mailto:Aerogear-users@lists.jboss.org">Aerogear-users@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-users" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-users</a><br>
<br>
<br>
<br>
<br>
--<br>
Cheers,<br>
Erik Jan<br>
<br>
_______________________________________________<br>
Aerogear-users mailing list<br>
<a href="mailto:Aerogear-users@lists.jboss.org">Aerogear-users@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/aerogear-users" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/aerogear-users</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Cheers,<div> Erik Jan</div></div></div>
</div>