You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "ASF subversion and git services (Jira)" <ji...@apache.org> on 2019/08/21 03:24:00 UTC
[jira] [Commented] (CB-14108) in config.xml increases
config_munge count in ios.json and android.json.
[ https://issues.apache.org/jira/browse/CB-14108?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16911910#comment-16911910 ]
ASF subversion and git services commented on CB-14108:
------------------------------------------------------
Commit d1a10a9774ee8d0f5505c2aa746d4f2733060481 in cordova-common's branch refs/heads/master from エリス
[ https://gitbox.apache.org/repos/asf?p=cordova-common.git;h=d1a10a9 ]
revert: #24 CB-14108: fix incorrect count in config_munge (#89)
* Revert "CB-14108: fix incorrect count in config_munge in ios.json and android.json"
This reverts commit ce3801a184d34cc4c3ea08aeecd159227c6ae9e9.
* chore: eslint object-curly-spacing fix
* add CordovaError spec
* chore: fix eslint warnings for CordovaError spec
* fix CordovaError spec test
* chore: added additional AS project path check
> <config-file> in config.xml increases config_munge count in ios.json and android.json.
> --------------------------------------------------------------------------------------
>
> Key: CB-14108
> URL: https://issues.apache.org/jira/browse/CB-14108
> Project: Apache Cordova
> Issue Type: Bug
> Components: cordova-common, cordova-lib
> Affects Versions: 8.0.0
> Reporter: Ken Naito
> Priority: Major
> Fix For: cordova-common@3.0.0
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> On both iOS and Android platforms, using <config-file> causes duplication in the `ios.json` and `android.json` files, which could lead to extremely large file sizes (and possibly unexpected behaviour?).
> For a Cordova 7.1/8.0 application, add the following to config.xml:
>
> {code:java}
> <platform name="ios">
> <allow-intent href="itms:*" />
> <allow-intent href="itms-apps:*" />
> <config-file parent="NSCameraUsageDescription" target="*-Info.plist">
> <string>Please permit Camera!</string>
> </config-file>
> </platform>
> {code}
> Now run `cordova prepare`. The file `platforms/ios/ios.json` now contains the below. Note the count is 1.
>
> {code:java}
> "config_munge": {
> "files": {
> "*-Info.plist": {
> "parents": {
> "NSCameraUsageDescription": [
> {
> "xml": "<string>Please permit Camera!</string>",
> "count": 1
> }
> ]
> }
> }
> }
> },
> {code}
> Run `cordova prepare` again. The platforms/ios/ios.json changes, increasing the count to 2.
>
> {code:java}
> "config_munge": {
> "files": {
> "*-Info.plist": {
> "parents": {
> "NSCameraUsageDescription": [
> {
> "xml": "<string>Please permit Camera!</string>",
> "count": 2
> }
> ]
> }
> }
> }
> },
> {code}
> Furthermore, if we change the text inside the `<string>` in the `config.xml`:
> {code:java}
> <platform name="ios">
> <allow-intent href="itms:*" />
> <allow-intent href="itms-apps:*" />
> <config-file parent="NSCameraUsageDescription" target="*-Info.plist">
> <string>This app uses Camera!</string>
> </config-file>
> </platform>
> {code}
> Then after running `cordova prepare`, there is now a second entry in `platforms/ios/ios.json`, rather than replacing the existing entry.
>
> {code:java}
> "config_munge": {
> "files": {
> "*-Info.plist": {
> "parents": {
> "NSCameraUsageDescription": [
> {
> "xml": "<string>Please permit Camera!</string>",
> "count": 2
> },
> {
> "xml": "<string>This app uses Camera!</string>",
> "count": 1
> }
> ]
> }
> }
> }
> },
> {code}
> This behaviour is the same for Android, and can be reproduced in the same way, by adding something like the below to `config.xml`:
> {code:java}
> <platform name="android">
> <allow-intent href="market:*" />
> <config-file parent="/manifest" target="AndroidManifest.xml">
> <uses-permission android:name="android.permission.VIBRATE" />
> </config-file>
> </platform>
> {code}
--
This message was sent by Atlassian Jira
(v8.3.2#803003)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org