-
Notifications
You must be signed in to change notification settings - Fork 46
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
Implement AS 16 Find command to fix e-mail search on iOS 16+ #57 #60
base: develop
Are you sure you want to change the base?
Conversation
…ck) of type string is deprecated
After my last PR merge this now has conflicts can you review and update? |
It looks like your merge makes my pull request obsolete doesn't it? It implements the same thing it seems. |
I fixed the conflicts. the file find.php was difficult to merge. I took my version and used the SYNC_* constants like you did. |
Sorry I haven't responded earlier. Really appreciate your work on this.
I don't have an iPhone so no way of knowing if my changes will work in general or for AS14 backends.
I'm not sure if you are in a position to test this with the Imap backend, but I'll shout out in the issue if someone is able to test. |
Will be able to test this soon, I've got an iPhone XS with the latest version of iOS on the way. |
@mschering I've tested this and it does work, but so does the existing code :) your find.php is more in the style of Z-Push, but there are a lot of references to Search instead of Find. Not sure if you want to put the work in to fix these up and merge the rest, but if you do, I would be happy to review it with the intent on merging. |
I can only find references in the comment block on top? |
What does this implement/fix? Explain your changes.
iOS 16 and up wrongfully sends a Find command when searching mail using Z-Push. I implemented the Find command so that searching works on iOS 16.
Does this close any currently open issues?
Fixes Error when searching mail on iphone #23
Any relevant logs, error output, etc?
No
Where has this been tested?
Server (please complete the following information):
Smartphone (please complete the following information):