diff options
author | Rob Mensching <rob@firegiant.com> | 2024-12-29 18:01:20 -0800 |
---|---|---|
committer | Rob Mensching <rob@firegiant.com> | 2024-12-30 06:45:49 -0800 |
commit | 523c66a62a619e6aa9f30070173ea33edfb5e328 (patch) | |
tree | 04fa3146250b7eeaa6864b0f71e37905a2d77be1 /src/burn/engine/detect.cpp | |
parent | 6edc5d1e2a289eac50c6d59a29e195353bb023cb (diff) | |
download | wix-523c66a62a619e6aa9f30070173ea33edfb5e328.tar.gz wix-523c66a62a619e6aa9f30070173ea33edfb5e328.tar.bz2 wix-523c66a62a619e6aa9f30070173ea33edfb5e328.zip |
Rename "bundle id" concept to "bundle code"
The "bundle id" always had more in common with the PackageCode from the Windows
Installer. With the introduction of an actual Id attribute on the Bundle
element, there is potential for confusion, so there is finally real motivation
to rename "bundle id" to "bundle code".
Diffstat (limited to 'src/burn/engine/detect.cpp')
-rw-r--r-- | src/burn/engine/detect.cpp | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/src/burn/engine/detect.cpp b/src/burn/engine/detect.cpp index 18820c5d..08f6b57c 100644 --- a/src/burn/engine/detect.cpp +++ b/src/burn/engine/detect.cpp | |||
@@ -18,13 +18,13 @@ static HRESULT WINAPI AuthenticationRequired( | |||
18 | ); | 18 | ); |
19 | 19 | ||
20 | static HRESULT DetectAtomFeedUpdate( | 20 | static HRESULT DetectAtomFeedUpdate( |
21 | __in_z LPCWSTR wzBundleId, | 21 | __in_z LPCWSTR wzBundleCode, |
22 | __in BURN_USER_EXPERIENCE* pUX, | 22 | __in BURN_USER_EXPERIENCE* pUX, |
23 | __in BURN_UPDATE* pUpdate | 23 | __in BURN_UPDATE* pUpdate |
24 | ); | 24 | ); |
25 | 25 | ||
26 | static HRESULT DownloadUpdateFeed( | 26 | static HRESULT DownloadUpdateFeed( |
27 | __in_z LPCWSTR wzBundleId, | 27 | __in_z LPCWSTR wzBundleCode, |
28 | __in BURN_USER_EXPERIENCE* pUX, | 28 | __in BURN_USER_EXPERIENCE* pUX, |
29 | __in BURN_UPDATE* pUpdate, | 29 | __in BURN_UPDATE* pUpdate, |
30 | __deref_inout_z LPWSTR* psczTempFile | 30 | __deref_inout_z LPWSTR* psczTempFile |
@@ -38,8 +38,8 @@ extern "C" void DetectReset( | |||
38 | ) | 38 | ) |
39 | { | 39 | { |
40 | RelatedBundlesUninitialize(&pRegistration->relatedBundles); | 40 | RelatedBundlesUninitialize(&pRegistration->relatedBundles); |
41 | ReleaseNullStr(pRegistration->sczDetectedProviderKeyBundleId); | 41 | ReleaseNullStr(pRegistration->sczDetectedProviderKeyBundleCode); |
42 | pRegistration->fDetectedForeignProviderKeyBundleId = FALSE; | 42 | pRegistration->fDetectedForeignProviderKeyBundleCode = FALSE; |
43 | pRegistration->fSelfRegisteredAsDependent = FALSE; | 43 | pRegistration->fSelfRegisteredAsDependent = FALSE; |
44 | pRegistration->fParentRegisteredAsDependent = FALSE; | 44 | pRegistration->fParentRegisteredAsDependent = FALSE; |
45 | pRegistration->fForwardCompatibleBundleExists = FALSE; | 45 | pRegistration->fForwardCompatibleBundleExists = FALSE; |
@@ -123,14 +123,14 @@ extern "C" HRESULT DetectForwardCompatibleBundles( | |||
123 | HRESULT hr = S_OK; | 123 | HRESULT hr = S_OK; |
124 | int nCompareResult = 0; | 124 | int nCompareResult = 0; |
125 | 125 | ||
126 | if (pRegistration->fDetectedForeignProviderKeyBundleId) | 126 | if (pRegistration->fDetectedForeignProviderKeyBundleCode) |
127 | { | 127 | { |
128 | for (DWORD iRelatedBundle = 0; iRelatedBundle < pRegistration->relatedBundles.cRelatedBundles; ++iRelatedBundle) | 128 | for (DWORD iRelatedBundle = 0; iRelatedBundle < pRegistration->relatedBundles.cRelatedBundles; ++iRelatedBundle) |
129 | { | 129 | { |
130 | BURN_RELATED_BUNDLE* pRelatedBundle = pRegistration->relatedBundles.rgRelatedBundles + iRelatedBundle; | 130 | BURN_RELATED_BUNDLE* pRelatedBundle = pRegistration->relatedBundles.rgRelatedBundles + iRelatedBundle; |
131 | 131 | ||
132 | if (BOOTSTRAPPER_RELATION_UPGRADE == pRelatedBundle->detectRelationType && | 132 | if (BOOTSTRAPPER_RELATION_UPGRADE == pRelatedBundle->detectRelationType && |
133 | CSTR_EQUAL == ::CompareStringW(LOCALE_NEUTRAL, NORM_IGNORECASE, pRegistration->sczDetectedProviderKeyBundleId, -1, pRelatedBundle->package.sczId, -1)) | 133 | CSTR_EQUAL == ::CompareStringW(LOCALE_NEUTRAL, NORM_IGNORECASE, pRegistration->sczDetectedProviderKeyBundleCode, -1, pRelatedBundle->package.sczId, -1)) |
134 | { | 134 | { |
135 | hr = VerCompareParsedVersions(pRegistration->pVersion, pRelatedBundle->pVersion, &nCompareResult); | 135 | hr = VerCompareParsedVersions(pRegistration->pVersion, pRelatedBundle->pVersion, &nCompareResult); |
136 | ExitOnFailure(hr, "Failed to compare bundle version '%ls' to related bundle version '%ls'", pRegistration->pVersion->sczVersion, pRelatedBundle->pVersion->sczVersion); | 136 | ExitOnFailure(hr, "Failed to compare bundle version '%ls' to related bundle version '%ls'", pRegistration->pVersion->sczVersion, pRelatedBundle->pVersion->sczVersion); |
@@ -201,7 +201,7 @@ LExit: | |||
201 | } | 201 | } |
202 | 202 | ||
203 | extern "C" HRESULT DetectUpdate( | 203 | extern "C" HRESULT DetectUpdate( |
204 | __in_z LPCWSTR wzBundleId, | 204 | __in_z LPCWSTR wzBundleCode, |
205 | __in BURN_USER_EXPERIENCE* pUX, | 205 | __in BURN_USER_EXPERIENCE* pUX, |
206 | __in BURN_UPDATE* pUpdate | 206 | __in BURN_UPDATE* pUpdate |
207 | ) | 207 | ) |
@@ -228,7 +228,7 @@ extern "C" HRESULT DetectUpdate( | |||
228 | 228 | ||
229 | if (!fSkip) | 229 | if (!fSkip) |
230 | { | 230 | { |
231 | hr = DetectAtomFeedUpdate(wzBundleId, pUX, pUpdate); | 231 | hr = DetectAtomFeedUpdate(wzBundleCode, pUX, pUpdate); |
232 | ExitOnFailure(hr, "Failed to detect atom feed update."); | 232 | ExitOnFailure(hr, "Failed to detect atom feed update."); |
233 | } | 233 | } |
234 | 234 | ||
@@ -306,7 +306,7 @@ LExit: | |||
306 | } | 306 | } |
307 | 307 | ||
308 | static HRESULT DownloadUpdateFeed( | 308 | static HRESULT DownloadUpdateFeed( |
309 | __in_z LPCWSTR wzBundleId, | 309 | __in_z LPCWSTR wzBundleCode, |
310 | __in BURN_USER_EXPERIENCE* pUX, | 310 | __in BURN_USER_EXPERIENCE* pUX, |
311 | __in BURN_UPDATE* pUpdate, | 311 | __in BURN_UPDATE* pUpdate, |
312 | __deref_inout_z LPWSTR* psczTempFile | 312 | __deref_inout_z LPWSTR* psczTempFile |
@@ -334,7 +334,7 @@ static HRESULT DownloadUpdateFeed( | |||
334 | cacheCallback.pv = NULL; //pProgress; | 334 | cacheCallback.pv = NULL; //pProgress; |
335 | 335 | ||
336 | authenticationData.pUX = pUX; | 336 | authenticationData.pUX = pUX; |
337 | authenticationData.wzPackageOrContainerId = wzBundleId; | 337 | authenticationData.wzPackageOrContainerId = wzBundleCode; |
338 | 338 | ||
339 | authenticationCallback.pv = static_cast<LPVOID>(&authenticationData); | 339 | authenticationCallback.pv = static_cast<LPVOID>(&authenticationData); |
340 | authenticationCallback.pfnAuthenticate = &AuthenticationRequired; | 340 | authenticationCallback.pfnAuthenticate = &AuthenticationRequired; |
@@ -364,7 +364,7 @@ LExit: | |||
364 | 364 | ||
365 | 365 | ||
366 | static HRESULT DetectAtomFeedUpdate( | 366 | static HRESULT DetectAtomFeedUpdate( |
367 | __in_z LPCWSTR wzBundleId, | 367 | __in_z LPCWSTR wzBundleCode, |
368 | __in BURN_USER_EXPERIENCE* pUX, | 368 | __in BURN_USER_EXPERIENCE* pUX, |
369 | __in BURN_UPDATE* pUpdate | 369 | __in BURN_UPDATE* pUpdate |
370 | ) | 370 | ) |
@@ -385,7 +385,7 @@ static HRESULT DetectAtomFeedUpdate( | |||
385 | hr = AtomInitialize(); | 385 | hr = AtomInitialize(); |
386 | ExitOnFailure(hr, "Failed to initialize Atom."); | 386 | ExitOnFailure(hr, "Failed to initialize Atom."); |
387 | 387 | ||
388 | hr = DownloadUpdateFeed(wzBundleId, pUX, pUpdate, &sczUpdateFeedTempFile); | 388 | hr = DownloadUpdateFeed(wzBundleCode, pUX, pUpdate, &sczUpdateFeedTempFile); |
389 | ExitOnFailure(hr, "Failed to download update feed."); | 389 | ExitOnFailure(hr, "Failed to download update feed."); |
390 | 390 | ||
391 | hr = AtomParseFromFile(sczUpdateFeedTempFile, &pAtomFeed); | 391 | hr = AtomParseFromFile(sczUpdateFeedTempFile, &pAtomFeed); |