[
https://issues.apache.org/jira/browse/CB-10845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15214644#comment-15214644
]
ASF GitHub Bot commented on CB-10845:
-------------------------------------
Github user omefire commented on a diff in the pull request:
https://github.com/apache/cordova-windows/pull/164#discussion_r57611536
--- Diff: template/cordova/lib/prepare.js ---
@@ -420,6 +420,11 @@ module.exports.prepare = function (cordovaProject) {
this._config = updateConfigFilesFrom(cordovaProject.projectConfig,
this._munger, this.locations);
+ // CB-10845 avoid using cached appxmanifests since they could be
+ // previously modififed outside of AppxManifest class
+ // TODO: invalidate only entries that were affected by config munge
+ AppxManifest.purgeCache();
--- End diff --
In this instance, who modifies the file outside the 'AppxManifest' class ?
> Location permission lost for Windows Phone 8.1
> ----------------------------------------------
>
> Key: CB-10845
> URL: https://issues.apache.org/jira/browse/CB-10845
> Project: Apache Cordova
> Issue Type: Bug
> Components: Plugin Geolocation, Windows
> Environment: Windows Phone 8.1
> Reporter: Steffen Schaffert
> Assignee: Vladimir Kotikov
>
> After updating to the latest Cordova versions (cordova-cli 6.0.0,
> cordova-windows 4.3.1, cordova-plugin-geolocation 2.1.0), the location
> capability is missing from the appxmanifest file.
> In the platforms/windows folder, the file "package.windows.appxmanifest"
> contains the following region:
> <Capabilities>
> <Capability Name="internetClient" />
> <DeviceCapability Name="location" />
> </Capabilities>
> The file "package.phone.appxmanifest", which seems to be relevant when
> building for the ARM platform, is missing the location entry:
> <Capabilities>
> <Capability Name="internetClientServer" />
> </Capabilities>
> I don't know whether this is a bug in the geolocation plugin (which should
> add the permission) or in the windows platform. I was able to fix the problem
> for me by extending the plugin.xml of the geolocation plugin and adding:
> <config-file target="package.phone.appxmanifest"
> parent="/Package/Capabilities">
> <DeviceCapability Name="location" />
> </config-file>
> This seems to be a bug because the existing extra with
> target="package.appxmanifest" works for the file package.windows.appxmanifest
> but not for package.phone.appxmanifest.
> Edit: The build target was "Windows Phone (Universal)" from Visual Studio
> 2015, which results in creating an app package with the following name:
> "CordovaApp.Phone_<version>_arm.appxupload
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
