You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by pu...@apache.org on 2014/04/28 20:17:29 UTC

git commit: Windows 8.1 update breaks app manifest parser

Repository: cordova-cli
Updated Branches:
  refs/heads/master eb7f6791a -> df5101d2f


Windows 8.1 update breaks app manifest parser

The powers-that-be at Microsoft decided to add another namespace to the package.appxmanifest when I updated my Windows 8 app to Windows 8.1. The Package/Applications/Application/VisualElements element now has a namespace prefix of "m2:" which causes an error during project preparation. This commit contains a quick, low-risk update to accommodate that change.

Project: http://git-wip-us.apache.org/repos/asf/cordova-cli/repo
Commit: http://git-wip-us.apache.org/repos/asf/cordova-cli/commit/df5101d2
Tree: http://git-wip-us.apache.org/repos/asf/cordova-cli/tree/df5101d2
Diff: http://git-wip-us.apache.org/repos/asf/cordova-cli/diff/df5101d2

Branch: refs/heads/master
Commit: df5101d2fcd2ba7a6ad44578988c946b01434493
Parents: eb7f679
Author: Aaron Newcomer <aa...@evostrat.com>
Authored: Mon Apr 28 13:05:53 2014 -0500
Committer: Aaron Newcomer <aa...@evostrat.com>
Committed: Mon Apr 28 13:05:53 2014 -0500

----------------------------------------------------------------------
 src/metadata/windows8_parser.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cordova-cli/blob/df5101d2/src/metadata/windows8_parser.js
----------------------------------------------------------------------
diff --git a/src/metadata/windows8_parser.js b/src/metadata/windows8_parser.js
index 2245ff4..9502974 100644
--- a/src/metadata/windows8_parser.js
+++ b/src/metadata/windows8_parser.js
@@ -111,7 +111,7 @@ module.exports.prototype = {
                 app['attrib']['Id'] = pkgName;
             }
 
-            var visualElems = manifest.find('.//VisualElements');
+            var visualElems = manifest.find('.//VisualElements') || manifest.find('.//m2:VisualElements');
 
             if(visualElems) {
                 var displayName = visualElems['attrib']['DisplayName'];