[release/9.0.1xx] [UIKit] Make sure to invoke the correct constructor. #23131
Conversation
…23127) There are three constructors involved here: 1. (UIBarButtonSystemItem systemItem, EventHandler handler) 2. (UIBarButtonSystemItem systemItem, NSObject target, Selector action) 3. (UIBarButtonSystemItem systemItem, UIAction primaryAction, UIMenu menu) The first constructor is a manually written convenience constructor, and needs to call another constructor. Either the second or the third constructor works, but the third constructor is only available on iOS 16+, while the second is available on all iOS version. This means that calling the third constructor is a bad idea, because it won't work on anything before iOS 16. Unfortunately that's exactly what we did. So change the code to call the second constructor instead. This is a regression that was introduced in f431538. Fixes #23122.
dalexsoto
changed the title
dalexsoto
changed the title
✅ [PR Build #134a841] Build passed (Detect API changes) ✅Pipeline on Agent |
✅ [CI Build #134a841] Build passed (Build packages) ✅Pipeline on Agent |
✅ API diff for current PR / commit.NET ( No breaking changes )✅ API diff vs stable.NET ( No breaking changes )ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
✅ [CI Build #134a841] Build passed (Build macOS tests) ✅Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
💻 [CI Build #134a841] Tests on macOS M1 - Mac Monterey (12) passed 💻✅ All tests on macOS M1 - Mac Monterey (12) passed. Pipeline on Agent |
💻 [CI Build #134a841] Tests on macOS M1 - Mac Ventura (13) passed 💻✅ All tests on macOS M1 - Mac Ventura (13) passed. Pipeline on Agent |
💻 [CI Build #134a841] Tests on macOS arm64 - Mac Sequoia (15) passed 💻✅ All tests on macOS arm64 - Mac Sequoia (15) passed. Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
💻 [CI Build #134a841] Tests on macOS X64 - Mac Sonoma (14) passed 💻✅ All tests on macOS X64 - Mac Sonoma (14) passed. Pipeline on Agent |
🚀 [CI Build #134a841] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 119 tests passed 🎉 Tests counts✅ cecil: All 1 tests passed. Html Report (VSDrops) Download Pipeline on Agent |
3 participants
Backport of #23127
Fixes #23122
There are three constructors involved here:
The first constructor is a manually written convenience constructor, and needs
to call another constructor. Either the second or the third constructor works,
but the third constructor is only available on iOS 16+, while the second is
available on all iOS version.
This means that calling the third constructor is a bad idea, because it won't
work on anything before iOS 16. Unfortunately that's exactly what we did.
So change the code to call the second constructor instead.
This is a regression that was introduced in
f431538.
Fixes #23122.