-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Don’t pass extension items to system activities that can’t consume them #38
Don’t pass extension items to system activities that can’t consume them #38
Conversation
This reverts commit 3b00421.
I’ve ticketed the fact that I have no idea how to test the system Weibo activities: #39 |
Set your simulator’s region and language to China / Chinese. |
Thanks – I’m still going to leave this as a separate ticket until I have some chance to play with it. It’s a bit more involved since it involves creating two separate Weibo accounts – Sina and Tencent – the latter of which may require a Chinese telephone number? |
This should be ready to go. |
@@ -4,6 +4,14 @@ | |||
#import "CustomParameters.h" | |||
#import "XExtensionItem.h" | |||
|
|||
#define XCTAssertEqualExtensionItems(item1, item2) \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be odd to use a prefix that’s not ours because theoretically Apple could add this same function.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
…to-certain-system-activities Don’t pass extension items to system activities that can’t consume them
😡😡😡😡😡😡😡😡😡😡😡
Thanks to @prendio2 for catching this one. While some system activities do know how to consume
NSExtensionItem
input, others like Mail and Messages do not. Passing anNSExtensionItem
to these doesn’t result in any data making its way through.Unfortunately I don’t know a better way to handle this other than to deduce (via trial and error) which activities can support extension item input, and pass the
placeholder
object to the ones that can’t.This PR to add more
UIActivityItemSource
hooks makes these changes a lot easier to swallow, in my opinion, even though this still sucks. I definitely need to radar this.CC: @mattbischoff @AriX