Add new XML operations to ios. Closes #79

Author: mlynch

packages/common/test/fixtures/ios-and-android/ios/App/xml-file.xml

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+<dict>
+    <key>com.apple.developer.parent-application-identifiers</key>
+    <array>
+        <string>$(AppIdentifierPrefix)io.ionic.wowzaStarter</string>
+    </array>
+</dict>
+</plist>

packages/configure/src/operations/ios/xml.ts

@@ -12,6 +12,8 @@ export default async function execute(ctx: Context, op: Operation) {
       throw new Error(`No such XML file for xml operation: ${entry.file}`);
     }
 
+    await xmlFile.load();
+
     if (entry.attrs) {
       await xmlFile.setAttrs(entry.target, entry.merge);
     } else if (entry.inject) {
@@ -20,6 +22,10 @@ export default async function execute(ctx: Context, op: Operation) {
       await xmlFile.mergeFragment(entry.target, entry.merge);
     } else if (entry.replace) {
       await xmlFile.replaceFragment(entry.target, entry.replace);
+    } else if (entry.delete) {
+      await xmlFile.deleteNodes(entry.delete);
+    } else if (entry.deleteAttributes) {
+      await xmlFile.deleteAttributes(entry.target, entry.deleteAttributes);
     }
   }
 }

packages/configure/test/ops/ios.xml.test.ts

@@ -0,0 +1,79 @@
+import { copy, readFile } from '@ionic/utils-fs';
+import { join } from 'path';
+import tempy from 'tempy';
+
+import { Context, loadContext } from '../../src/ctx';
+import { IosXmlOperation, Operation } from '../../src/definitions';
+import Op from '../../src/operations/ios/xml';
+import { makeOp } from '../utils';
+
+describe('op: ios.xml', () => {
+  let dir: string;
+  let ctx: Context;
+
+  beforeEach(async () => {
+    dir = tempy.directory();
+
+    await copy('../common/test/fixtures/ios-and-android', dir);
+
+    ctx = await loadContext(dir);
+    ctx.args.quiet = true;
+  });
+
+  it('should delete attributes', async () => {
+    const op: IosXmlOperation = makeOp('ios', 'xml', [
+      {
+        file: 'xml-file.xml',
+        target: '//plist',
+        deleteAttributes: [
+          'version'
+        ]
+      },
+    ]);
+
+    await Op(ctx, op as Operation);
+
+    await ctx.project.commit();
+
+    const file = await readFile(join(dir, 'ios', 'App', 'xml-file.xml'), { encoding: 'utf-8' });
+    //console.log(file);
+    expect(file.trim()).toBe(`
+<?xml version="1.0" encoding="UTF-8" ?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist>
+    <dict>
+        <key>com.apple.developer.parent-application-identifiers</key>
+        <array>
+            <string>$(AppIdentifierPrefix)io.ionic.wowzaStarter</string>
+        </array>
+    </dict>
+</plist>
+    `.trim());
+  });
+
+  it('should delete nodes', async () => {
+    const op: IosXmlOperation = makeOp('ios', 'xml', [
+      {
+        file: 'xml-file.xml',
+        delete: '//array'
+      },
+    ]);
+
+    await Op(ctx, op as Operation);
+
+    await ctx.project.commit();
+
+    const file = await readFile(join(dir, 'ios', 'App', 'xml-file.xml'), { encoding: 'utf-8' });
+    //console.log(file);
+    expect(file.trim()).toBe(`
+<?xml version="1.0" encoding="UTF-8" ?>
+<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
+<plist version="1.0">
+    <dict>
+        <key>com.apple.developer.parent-application-identifiers</key>
+    </dict>
+</plist>
+    `.trim());
+  });
+
+});

packages/website/docs/configuration-tool.md

@@ -491,3 +491,23 @@ platforms:
               data:
                 field: "MY_FIELD"
 ```
+
+### `xml`
+
+Modifies XML files relative to the root of the iOS project. This operation supports the following XML file modifications:
+
+- `attrs` updates the attributes of the given `target` node.
+- `merge` merges the given XML tree supplied to `merge` with the given `target`
+- `inject` injects the given XML tree supplied to `inject` inside of the given `target`
+- `delete` deletes nodes specified by `delete` in XPath format.
+- `deleteAttributes` deletes the given attributes in `deleteAttributes` inside of the given `target`
+
+```yaml
+platforms:
+  android:
+    xml:
+      - file: file.xml
+        target: entries/field
+        merge: |
+          <string>Value</string>
+```