GiovanniBattista commented on issue #201:
URL: https://github.com/apache/cordova-common/issues/201#issuecomment-1681899632

   Sorry for the late response, I was on a short vacation.
   
   Unfortunately, my proposed fix didn't work correctly and led to a subsequent 
error. 
   
   Yeah, it also seams to me, that the cleanup upon plugin removal doesn't work 
properly. 
   In my case, the resulting data point looks like this:
   ```
   "/manifest/application": [
   {
     "xml": "<application 
android:networkSecurityConfig=\"@xml/uq_network_security_config\" />",
     "count": 1,
     "mode": "merge",
     "plugin": "uq-cordova-plugin-settings",
     "oldAttrib": {
       "android:hardwareAccelerated": "true",
       "android:icon": "@mipmap/ic_launcher",
       "android:label": "@string/app_name",
       "android:networkSecurityConfig": "@xml/uq_network_security_config",
       "android:supportsRtl": "true"
     }
   },
   ```
   
   Upon removal, it actually looks like this which produces the error when the 
plugin is re-added again.
   ```
   "/manifest/application": []
   ```
   If I remove this line, then the re-adding works.
   
   It also works, if at least one other plugin adds something to 
"/manifest/application", i.e. if the array isn't empty when the plugin is 
re-added again.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to