From 23084ab302752826f43f4f58576fe7a3921a4795 Mon Sep 17 00:00:00 2001
From: Carlos Nihelton <carlos.santanadeoliveira@canonical.com>
Date: Mon, 18 Sep 2023 13:34:15 -0300
Subject: [PATCH] Fix MSBuild PreprocessorDefinitions

DefineConstants is a C# property.
For C++. it must be PreprocessorDefinitions under a ClCompile item.
Also, the logic was twisted
---
 msix/storeapi/storeapi.vcxproj | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/msix/storeapi/storeapi.vcxproj b/msix/storeapi/storeapi.vcxproj
index c98886514..8c063c59d 100644
--- a/msix/storeapi/storeapi.vcxproj
+++ b/msix/storeapi/storeapi.vcxproj
@@ -43,9 +43,11 @@
     <Import Project="$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props" Condition="exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props')" Label="LocalAppDataPlatform" />
   </ImportGroup>
   <PropertyGroup Label="UserMacros" />
-  <PropertyGroup Label="Mock API" Condition="'$(UP4W_TEST_WITH_MS_STORE_MOCK)' == ''">
-    <DefineConstants>UP4W_TEST_WITH_MS_STORE_MOCK</DefineConstants>
-  </PropertyGroup>
+  <ItemDefinitionGroup Label="Mock API" Condition="'$(UP4W_TEST_WITH_MS_STORE_MOCK)' != ''">
+    <ClCompile>
+      <PreprocessorDefinitions>UP4W_TEST_WITH_MS_STORE_MOCK;%(PreProcessorDefinitions)</PreprocessorDefinitions>
+    </ClCompile>
+  </ItemDefinitionGroup>
   <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">
     <ClCompile>
       <WarningLevel>Level3</WarningLevel>