Consider separating broadcast from a push with no recipients

klaussilveira / urbanairship-php

PHP library that provides a simple way to communicate with Urban Airship services

BSD 3-Clause "New" or "Revised" License

6 stars 5 forks source link

Consider separating broadcast from a push with no recipients #1

Open robotadam opened 11 years ago

robotadam commented 11 years ago

Right now you send a broadcast by sending a notification without any devices assigned to it. I recommend changing that to using a separate method or class. I've seen similar systems have unintentional sends when a coding error specifies no recipients.