Fix application ID name

There's a lot of terminological/conceptual confusion in the current API.

An .appx file should have an `AppXManifest.xml` which looks like the following,
omitting unimportant elements:

    <?xml version="1.0" encoding="utf-8"?>
    <Package ...>
        <Identity Name="12345MyCompany.MyApp" ... />
        ...
        <Applications>
            <Application Id="MyApp" ...>
                ...
            </Application>
        </Applications>
    </Package>

Notice that there are two separate identities: the Package Name, and the Application Id.

When you run `electron-windows-store`,
it generates an `AppXManifest.xml` from the config you supply.
[The template is here](https://github.com/felixrieseberg/electron-windows-store/blob/master/template/appxmanifest.xml)
and it looks like:

<?xml version="1.0" encoding="utf-8"?>
<Package ...>
  <Identity Name="${identityName}" ... />
  ...
  <Applications>
    <Application Id="${packageName}" ...>
      ...
    </Application>
  </Applications>
</Package>

Notice in the variable names:
it calls the package name `identityName`,
and calls the application id `packageName`.
These template variables are subtituted by the user-supplied config like so:

    lib/convert.js:      result = result.replace(/\${identityName}/g, program.identityName || program.packageName)
    lib/convert.js:      result = result.replace(/\${packageName}/g, program.packageName)

This means, to create a correct `AppXManifest.xml`,
you need to call it like so:

    const convertToWindowsStore = require('electron-windows-store');
    convertToWindowsStore({
        identityName: '19463Vidrio.Vidrio',  // This is actually the package name!
        packageName: 'Vidrio',  // This is actually the application id!!
        // ...
    });

This is very confusing, and has led to many tickets:

    electron-userland#82
    electron-userland#114
    electron-userland#103
    electron-userland#120

The best way forward would be to introduce a separate `program.applicationId` config,
which is used in preference to the `packageName`.

README.md

@@ -66,15 +66,15 @@ These are all options for the CLI:
   -i, --input-directory <path>               Directory containing your application
   -o, --output-directory <path>              Output directory for the appx
   -p, --package-version <version>            Version of the app package
-  -n, --package-name <name>                  Name of the app package
+  -n, --package-name <name>                  Name of the app package (example: 12345MyCompany.Ghost)
       --package-display-name <displayName>   Display name of the package
       --package-description <description>    Description of the package
       --package-background-color <color>     Background color for the app icon (example: #464646)
   -e, --package-executable <executablePath>  Path to the package executable
+  --application-id <applicationId>           Application id (example: Ghost)
   -a, --assets <assetsPath>                  Path to the visual assets for the appx
   -m, --manifest <manifestPath>              Path to a manifest, if you want to be overwritten
   -d, --deploy <true|false>                  Should the app be deployed after creation?
-  --identity-name <name>                     Name for identity
   --publisher <publisher>                    Publisher to use (example: CN=developmentca)
   --publisher-display-name <publisherDisplayName> Publisher display name to use
   --make-pri <true|false>                    Use makepri.exe (you don't need to unless you know you do)
@@ -101,10 +101,11 @@ convertToWindowsStore({
    inputDirectory: 'C:\\input\\',
    outputDirectory: 'C:\\output\\',
    packageVersion: '1.0.0.0',
-   packageName: 'Ghost',
+   packageName: '12345MyCompany.Ghost',
    packageDisplayName: 'Ghost Desktop',
    packageDescription: 'Ghost for Desktops',
    packageExecutable: 'app/Ghost.exe',
+   applicationId: 'Ghost',
    assets: 'C:\\assets\\',
    manifest: 'C:\\AppXManifest.xml',
    deploy: false,

bin/windowsstore.js

@@ -26,11 +26,12 @@ program
   .option('-i, --input-directory <path>', 'Directory containing your application')
   .option('-o, --output-directory <path>', 'Output directory for the appx')
   .option('-p, --package-version <version>', 'Version of the app package')
-  .option('-n, --package-name <name>', 'Name of the app package')
-  .option('--identity-name <name>', 'Name for identity')
+  .option('-n, --package-name <name>', 'Name of the app package (example: 12345MyCompany.Ghost)')
+  .option('--identity-name <name>', 'Name for identity (deprecated; use --package-name)')
   .option('--package-display-name <displayName>', 'Dispay name of the package')
   .option('--package-description <description>', 'Description of the package')
   .option('--package-background-color <color>', 'Background color for the app icon (example: #464646)')
+  .option('--application-id <applicationId>', 'Application id (example: Ghost)')
   .option('-e, --package-executable <executablePath>', 'Path to the package executable')
   .option('-a, --assets <assetsPath>', 'Path to the visual assets for the appx')
   .option('-m, --manifest <manifestPath>', 'Path to a manifest, if you want to overwrite the default one')

lib/convert.js

@@ -135,7 +135,7 @@ function convertWithFileCopy (program) {
       result = result.replace(/\${publisherDisplayName}/g, program.publisherDisplayName || 'Reserved')
       result = result.replace(/\${identityName}/g, program.identityName || program.packageName)
       result = result.replace(/\${packageVersion}/g, program.packageVersion)
-      result = result.replace(/\${packageName}/g, program.packageName)
+      result = result.replace(/\${applicationId}/g, program.applicationId || program.packageName)
       result = result.replace(/\${packageExecutable}/g, executable)
       result = result.replace(/\${packageDisplayName}/g, program.packageDisplayName || program.packageName)
       result = result.replace(/\${packageDescription}/g, program.packageDescription || program.packageName)

lib/params.js

@@ -53,10 +53,16 @@ module.exports = function (program) {
         },
         when: () => (!program.outputDirectory)
       },
+      {
+        name: 'applicationId',
+        type: 'input',
+        message: "Please enter your app's application id (Example: Ghost): ",
+        when: () => (!program.applicationId)
+      },
       {
         name: 'packageName',
         type: 'input',
-        message: "Please enter your app's package name (name of your exe - without '.exe'): ",
+        message: "Please enter your app's package name (Example: 12345MyCompany.Ghost): ",
         when: () => (!program.packageName)
       },
       {
@@ -76,7 +82,7 @@ module.exports = function (program) {
     inquirer.prompt(questions)
       .then((answers) => {
         if (!program.packageExecutable && program.containerVirtualization) {
-          program.packageExecutable = `C:\\Users\\ContainerAdministrator\\AppData\\Roaming\\e\\${program.packageName}.exe`
+          program.packageExecutable = `C:\\Users\\ContainerAdministrator\\AppData\\Roaming\\e\\${program.applicationId}.exe`
         }
 
         Object.assign(program, answers)

template/appxmanifest.xml

@@ -23,7 +23,7 @@
     <rescap:Capability Name="runFullTrust"/>
   </Capabilities>
   <Applications>
-    <Application Id="${packageName}" Executable="${packageExecutable}" EntryPoint="Windows.FullTrustApplication">
+    <Application Id="${applicationId}" Executable="${packageExecutable}" EntryPoint="Windows.FullTrustApplication">
       <uap:VisualElements
        BackgroundColor="${packageBackgroundColor}"
        DisplayName="${packageDisplayName}"