Remove all parts for sending to multiple E-Mails.

After a little Chat with the OneSignal Team, it isn't possible to use the filter with more than one "email" filter.

Author: LKaemmerling

README.md

@@ -104,7 +104,7 @@ public function routeNotificationForOneSignal()
 }
 ```
 
-If you want to send the notification based on the OneSignal "syncHashedEmail" feature just return an array with the index "email" and the E-Mail or an Array with E-Mails.
+If you want to send the notification based on the OneSignal "syncHashedEmail" feature just return an array with the index "email". **It isn't possible to use multiple E-Mails on one filter because of a limitation of the OneSignal API.**
 
 ```php
 public function routeNotificationForOneSignal()

src/OneSignalChannel.php

@@ -34,38 +34,15 @@ public function send($notifiable, Notification $notification)
 
         $payload = $notification->toOneSignal($notifiable)->toArray();
         if (is_array($userIds) && array_key_exists('email', $userIds)) {
-            $payload['filters'] = collect($this->parseEmailsToFilters($userIds['email']));
+            $payload['filters'] = collect([["field" => "email", "value" => $userIds['email']]]);
         } else {
             $payload['include_player_ids'] = collect($userIds);
         }
+
         /** @var ResponseInterface $response */
         $response = $this->oneSignal->sendNotificationCustom($payload);
-
         if ($response->getStatusCode() !== 200) {
             throw CouldNotSendNotification::serviceRespondedWithAnError($response);
         }
     }
-
-    /**
-     * Parse the given E-Mails to the array-format that OneSignal needs.
-     *
-     * @param mixed $userEmails
-     *
-     * @return array
-     */
-    public function parseEmailsToFilters($userEmails)
-    {
-        if (is_array($userEmails)) {
-            return collect($userEmails)->map(function ($email, $key) use ($userEmails) {
-                if ($key < (count($userEmails) - 1)) {
-                    return [["field" => "email", "relation" => "=", "value" => $email], ['operator' => 'OR']];
-                } else {
-                    return [["field" => "email", "relation" => "=", "value" => $email]];
-                }
-            })->flatten(1)->toArray();
-        }
-
-        return [["field" => "email", "relation" => "=", "value" => $userEmails]];
-
-    }
 }

tests/ChannelTest.php

@@ -101,36 +101,11 @@ public function it_can_send_a_notification_with_email()
                 'chrome_icon' => 'Icon',
                 'adm_small_icon' => 'Icon',
                 'small_icon' => 'Icon',
-                'filters' => collect([["field" => "email", "relation" => "=", "value" => '[email protected]']]),
+                'filters' => collect([["field" => "email", "value" => '[email protected]']]),
             ])
             ->andReturn($response);
 
         $this->channel->send(new NotifiableEMail(), new TestNotification());
     }
 
-    /**
-     * @test
-     */
-    public function it_can_send_a_notification_with_multiple_email()
-    {
-        $response = new Response(200);
-
-        $this->oneSignal->shouldReceive('sendNotificationCustom')
-            ->once()
-            ->with([
-                'contents' => ['en' => 'Body'],
-                'headings' => ['en' => 'Subject'],
-                'url' => 'URL',
-                'buttons' => [],
-                'web_buttons' => [],
-                'chrome_web_icon' => 'Icon',
-                'chrome_icon' => 'Icon',
-                'adm_small_icon' => 'Icon',
-                'small_icon' => 'Icon',
-                'filters' => collect([["field" => "email", "relation" => "=", "value" => '[email protected]'], ['operator' => 'OR'], ["field" => "email", "relation" => "=", "value" => '[email protected]']]),
-            ])
-            ->andReturn($response);
-
-        $this->channel->send(new NotifiableMultipleEMail(), new TestNotification());
-    }
 }