[jira] [Commented] (CB-13647) Incomplete plugin restoration

2020-11-26 Thread Yourspace (Jira)


[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17239241#comment-17239241
 ] 

Yourspace commented on CB-13647:


any update for Admob SDK v20?

> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joey Robert Bowser
>Priority: Major
>  Labels: android-next
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13647) Incomplete plugin restoration

2019-02-18 Thread Jose M (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16771445#comment-16771445
 ] 

Jose M commented on CB-13647:
-

Hi there!

With new cordova versions unwanted plugins are systematically added to the 
project while building android apps and cant find a way to get rid of them. 
Result is creating apps with unnecessary user permissions. I just created one 
app which does not need any permission at all but cordova forced 10 permissions 
cos it inherited them from the former app i built that required all those 
permissions.

I tried some hooks, removed permissions from manifest.xml, android.json etc. 
but it did not work

Not sure if this issue has to do with this reported problem but i did not find 
another way to place this issue. Sorry if I bother, I am new here.

Any solution will be appreciated.

Cheers,

J

> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joe Bowser
>Priority: Major
>  Labels: android-next
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13647) Incomplete plugin restoration

2018-01-18 Thread Joe Bowser (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16330976#comment-16330976
 ] 

Joe Bowser commented on CB-13647:
-

[~goffioul] Unfortunately I don't have a timeframe as to when the plugin paths 
will be changed.  We've tested the core plugins with mobilespec, and all of 
them currently install, so that's not a pressing issue currently.

> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joe Bowser
>Priority: Major
>  Labels: android-next
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13647) Incomplete plugin restoration

2018-01-17 Thread Michael Goffioul (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16329302#comment-16329302
 ] 

Michael Goffioul commented on CB-13647:
---

When will plugins be ported to use new paths? I've checked a couple of them and 
they're still using legacy paths?

 

> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joe Bowser
>Priority: Major
>  Labels: android-next
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13647) Incomplete plugin restoration

2017-12-19 Thread Joe Bowser (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16297127#comment-16297127
 ] 

Joe Bowser commented on CB-13647:
-

[~goffioul] Everything will be using new paths coming forward.  The only reason 
that we support legacy paths is to not break plugins, and I'm wondering if the 
InAppBrowser's icons are missing post-install.  We're definitely not patching 
edit-config, since it's a hack anyway, and xml and java source files can both 
use new or legacy paths, since I'm not aware of any Koltin plugins out in the 
wild yet (and they should use new paths anyway)



> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joe Bowser
>  Labels: android-next
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
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



[jira] [Commented] (CB-13647) Incomplete plugin restoration

2017-12-08 Thread Michael Goffioul (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16283474#comment-16283474
 ] 

Michael Goffioul commented on CB-13647:
---

[~bowserj] I admit that 2  entries with different path types is 
more theoretical. The more practical problem I faced was with my own plugin, 
using both  and . The cordova-android@7.0.0 release 
notes only states to modify , it doesn't say anything about 
, and more importantly it doesn't state that if you use both, they 
MUST refer to the same path type.

I'm using cordova-common@2.2.0

On a side but related note, I also faced inconsistencies when using 
, because path translation is enforced, meaning you can't (and 
must not) use the prefix "app/src/main" in the target attribute. For instance, 
with the following plugin.xml, you end up with a file in 
platforms/android/app/src/main/app/src/main/res/drawable/image.png and that's 
not what you want:
{code}

http://apache.org/cordova/ns/plugins/1.0; 
id="com.example.testplugin1" version="1.0.0">
Test Plugin 1




{code}

So at the moment, the rules I've figured out are (new == with prefix 
"app/src/main", legacy == without prefix):
- : you may use new or legacy paths
- : you may only use new paths
- : you may use new or legacy paths for *.java files; for other 
files you may only use new paths
- : you may only use legacy paths

It would be easier to have a consistent behavior for all of them.


> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joe Bowser
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
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



[jira] [Commented] (CB-13647) Incomplete plugin restoration

2017-12-07 Thread Joe Bowser (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16283046#comment-16283046
 ] 

Joe Bowser commented on CB-13647:
-

[~goffioul] Actually, first of all, what version of cordova-common is currently 
installed?  This might be a duplicate.

> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joe Bowser
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
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



[jira] [Commented] (CB-13647) Incomplete plugin restoration

2017-12-07 Thread Joe Bowser (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16282944#comment-16282944
 ] 

Joe Bowser commented on CB-13647:
-

[~goffioul] This seems more like a theoretical problem than a real one.  Does 
this actually occur on real plugins, or only on error data that you're 
generating. Which core plugins does this actually happen with?

> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joe Bowser
>Priority: Critical
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
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



[jira] [Commented] (CB-13647) Incomplete plugin restoration

2017-12-07 Thread Michael Goffioul (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16282828#comment-16282828
 ] 

Michael Goffioul commented on CB-13647:
---

I understand the problem with edit-config is known and documented. The problem 
is when mixing config-file/edit-config with new and legacy paths. And it 
doesn't have to include edit-config. For instance, if you change the plugin.xml 
with the following (notice how the 2 config-file target the same file, but with 
different paths):
{code}

http://apache.org/cordova/ns/plugins/1.0; 
id="com.example.testplugin1" version="1.0.0">
Test Plugin 1









{code}

If you add the plugin with "cordova plugin add localplugins/testplugin1", then 
only 1 of the changes is applied (note that it does not even involve plugin 
restoration anymore).

> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joe Bowser
>Priority: Critical
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
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



[jira] [Commented] (CB-13647) Incomplete plugin restoration

2017-12-07 Thread Joe Bowser (JIRA)

[ 
https://issues.apache.org/jira/browse/CB-13647?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16282790#comment-16282790
 ] 

Joe Bowser commented on CB-13647:
-

The edit-config issue is known and none of the core plugins use edit-config on 
Android.  We haven't experienced any issues when testing plugin addition with 
the existing plugins.

AFAIK: We're not going to be adding a work-around for edit-config and these 
paths will have to be changed for 7.0.0

> Incomplete plugin restoration
> -
>
> Key: CB-13647
> URL: https://issues.apache.org/jira/browse/CB-13647
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: 7.0.0
>Reporter: Michael Goffioul
>Assignee: Joe Bowser
>Priority: Critical
>
> I'm having a problem with plugin restoration since cordova-android 7.0.0, 
> when plugins use a mix of config-file and edit-config on the same file (e.g. 
> AndroidManifest.xml) and only edit-config uses the real path to the file. 
> I've tried both cordova-cli 6.5.0 and 7.1.0 with the same result.
> 1) Create a new corodva project and enter top-level direcotyr
> 2) Create subfolder to contain a local plugin: mkdir -p 
> localplugins/testplugin1
> 3) Create the following files to define the local plugin:
> - plugin.xml (adds an attribute to  and a category to 
> )
> {code}
> 
> http://apache.org/cordova/ns/plugins/1.0; 
> id="com.example.testplugin1" version="1.0.0">
> Test Plugin 1
> 
>  parent="/manifest/application/activity/intent-filter">
> 
> 
>  target="/manifest/application" mode="merge">
> 
> 
> 
> 
> {code}
> - package.json
> {code}
> {
> "name": "com.example.testplugin1",
> "version": "1.0.0"
> }
> {code}
> 4) Add android platform: cordova platform add android@7.0.0 --save
> 5) Add local plugin: cordova plugin add localplugins/testplugin1 --save
> At this point, if you check 
> platforms/android/app/src/main/AndroidManifest.xml, it'll contain the 2 
> changes defined in the plugin.
> Now attempt to use platform/plugin restoration with thw following:
> {code}
> cordova platform rm android
> cordova prepare android
> {code}
> If you check again AndroidManifest.xml, it will only contain one of the 
> changes defined in the plugin (the attribute to ). The other 
> change is missing.
> If you use the real path to AndroidManifest.xml for the definition of 
> , then the problem does not occur. BUT, the problem can also 
> occur when changes to AndroidManifest.xml are coming from multiple plugins, 
> including the standard cordova plugin, most of which do not use the real path 
> to AndroidManifest.xml yet.
> The problem can affect other files too. For instance I experienced incomplete 
> restoration of plugins trying to register themselves in res/xml/config.xml. 
> Only one plugin, out of 7, was using the real path 
> app/src/main/res/xml/config.xml and I ended up with only one plugin defined 
> in res/xml/config.xml, causing the application to fail to start.



--
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