From b0c6dd791f47b2f1cb919fdfa6a7594e95a26ff0 Mon Sep 17 00:00:00 2001
From: kapusch <BAILLATJean-Emmanuel@hotmail.com>
Date: Wed, 15 Jul 2026 23:51:26 +0200
Subject: [PATCH] fix(packaging): evaluate Facebook features after project
 properties

---
 .github/workflows/ci.yml                      |  3 +++
 .github/workflows/publish.yml                 |  3 +++
 .../Kapusch.Facebook.iOS.props                |  3 ---
 .../Kapusch.Facebook.iOS.targets              |  5 +++++
 tests/FeatureSelection.proj                   | 21 +++++++++++++++++++
 tests/validate-feature-selection.sh           | 11 ++++++++++
 6 files changed, 43 insertions(+), 3 deletions(-)
 create mode 100644 tests/FeatureSelection.proj
 create mode 100644 tests/validate-feature-selection.sh

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 67d00af..42a355b 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -17,6 +17,9 @@ jobs:
         with:
           dotnet-version: "10.0.x"
 
+      - name: Validate feature selection
+        run: bash tests/validate-feature-selection.sh
+
       - name: Install iOS workload
         run: |
           dotnet workload install ios
diff --git a/.github/workflows/publish.yml b/.github/workflows/publish.yml
index 67c1c1b..fa519bb 100644
--- a/.github/workflows/publish.yml
+++ b/.github/workflows/publish.yml
@@ -71,6 +71,9 @@ jobs:
         with:
           dotnet-version: "10.0.x"
 
+      - name: Validate feature selection
+        run: bash tests/validate-feature-selection.sh
+
       - name: Install iOS workload
         run: |
           dotnet workload install ios
diff --git a/src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.props b/src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.props
index 2dd050c..27f7324 100644
--- a/src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.props
+++ b/src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.props
@@ -1,9 +1,6 @@
 <Project>
   <PropertyGroup>
 	<KapuschFacebookFeatures Condition="'$(KapuschFacebookFeatures)' == ''">Login</KapuschFacebookFeatures>
-	<_KapuschFacebookFeaturesDelimited>;$([System.String]::Copy('$(KapuschFacebookFeatures)').Replace(' ', ''));</_KapuschFacebookFeaturesDelimited>
-	<_KapuschFacebookLoginEnabled>$([System.String]::Copy('$(_KapuschFacebookFeaturesDelimited)').Contains(';Login;'))</_KapuschFacebookLoginEnabled>
-	<_KapuschFacebookShareEnabled>$([System.String]::Copy('$(_KapuschFacebookFeaturesDelimited)').Contains(';Share;'))</_KapuschFacebookShareEnabled>
     <UseKapuschFacebookInteropFromSource Condition="'$(UseKapuschFacebookInteropFromSource)' == ''"
       >false</UseKapuschFacebookInteropFromSource
     >
diff --git a/src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.targets b/src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.targets
index 3b64a5e..d76447a 100644
--- a/src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.targets
+++ b/src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.targets
@@ -1,5 +1,10 @@
 <Project>
   <PropertyGroup>
+    <!-- Package .props are imported before the consuming project body. Resolve the
+         selected features here, after the consumer had a chance to override them. -->
+    <_KapuschFacebookFeaturesDelimited>;$([System.String]::Copy('$(KapuschFacebookFeatures)').Replace(' ', ''));</_KapuschFacebookFeaturesDelimited>
+    <_KapuschFacebookLoginEnabled>$([System.String]::Copy('$(_KapuschFacebookFeaturesDelimited)').Contains(';Login;'))</_KapuschFacebookLoginEnabled>
+    <_KapuschFacebookShareEnabled>$([System.String]::Copy('$(_KapuschFacebookFeaturesDelimited)').Contains(';Share;'))</_KapuschFacebookShareEnabled>
     <_KapuschFacebookInteropPackageRoot>$([System.IO.Path]::GetFullPath('$(MSBuildThisFileDirectory)..'))</_KapuschFacebookInteropPackageRoot>
   </PropertyGroup>
   <PropertyGroup Condition="'$(UseKapuschFacebookInteropFromSource)' == 'true'">
diff --git a/tests/FeatureSelection.proj b/tests/FeatureSelection.proj
new file mode 100644
index 0000000..ca4e472
--- /dev/null
+++ b/tests/FeatureSelection.proj
@@ -0,0 +1,21 @@
+<Project>
+  <!-- Reproduce NuGet's import order: .props, consumer project body, .targets. -->
+  <Import Project="../src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.props" />
+
+  <PropertyGroup Condition="'$(RequestedFacebookFeatures)' != ''">
+    <KapuschFacebookFeatures>$(RequestedFacebookFeatures)</KapuschFacebookFeatures>
+  </PropertyGroup>
+
+  <Import Project="../src/Kapusch.FacebookApisForiOSComponents/buildTransitive/Kapusch.Facebook.iOS.targets" />
+
+  <Target Name="ValidateFeatureSelection">
+    <Error
+      Condition="'$(_KapuschFacebookLoginEnabled)' != '$(ExpectedLoginEnabled)'"
+      Text="Expected Login=$(ExpectedLoginEnabled), got $(_KapuschFacebookLoginEnabled) for '$(KapuschFacebookFeatures)'."
+    />
+    <Error
+      Condition="'$(_KapuschFacebookShareEnabled)' != '$(ExpectedShareEnabled)'"
+      Text="Expected Share=$(ExpectedShareEnabled), got $(_KapuschFacebookShareEnabled) for '$(KapuschFacebookFeatures)'."
+    />
+  </Target>
+</Project>
diff --git a/tests/validate-feature-selection.sh b/tests/validate-feature-selection.sh
new file mode 100644
index 0000000..6f3bc49
--- /dev/null
+++ b/tests/validate-feature-selection.sh
@@ -0,0 +1,11 @@
+#!/usr/bin/env bash
+
+set -euo pipefail
+
+project="tests/FeatureSelection.proj"
+
+dotnet msbuild "$project" -t:ValidateFeatureSelection -p:ExpectedLoginEnabled=True -p:ExpectedShareEnabled=False
+
+dotnet msbuild "$project" -t:ValidateFeatureSelection -p:RequestedFacebookFeatures=Share -p:ExpectedLoginEnabled=False -p:ExpectedShareEnabled=True
+
+dotnet msbuild "$project" -t:ValidateFeatureSelection -p:RequestedFacebookFeatures=Login%3BShare -p:ExpectedLoginEnabled=True -p:ExpectedShareEnabled=True