-
-
src/Lunr/Vortex/GCM/GCMBatchResponse.php (Diff revision 1) I'd rather have a default statement then having it set before the switch
-
src/Lunr/Vortex/GCM/GCMDispatcher.php (Diff revision 1) pass by reference?
I suppose it would make sense here as we'd potentially have a lot of endpoints.
Change Summary:
fix small stuff
add unit tests
-
-
src/Lunr/Vortex/GCM/GCMDispatcher.php (Diff revisions 1 - 2) I see you made $this->push_batch() receive the parameter by reference, which is fine, and actually something I missed ;)
What I actually meant before was the foreach
-
src/Lunr/Vortex/GCM/GCMResponse.php (Diff revisions 1 - 2) We generally don't do this in Lunr. Type hinting provides contracts, and Lunr mostly provides contract free functionality.
Change Summary:
use references in foreach loop with endpoints
remove GCMBatchResponse type hinting
Diff: |
Revision 3 (+779 -339)
|
---|