Factor out batch push code for reuse on app side (#18)

* Factor out batch push code for reuse on app side * Add parameter description * Clean up --------- Co-authored-by: Scott Kim <[email protected]>
Symplicity · Mar 11, 2024 · f157b49 · f157b49
1 parent 920af43
commit f157b49
Showing 1 changed file with 13 additions and 1 deletion.
diff --git a/src/Calendar.php b/src/Calendar.php
@@ -235,6 +235,19 @@ public function getInstancesCollection(string $id, ?InstancesRequestBuilderGetQu
      * @throws \Exception|\Throwable
      */
     public function push(array $params = [], array $args = []): void
+    {
+        $eventsToWrite = $this->getLocalEvents();
+        $this->batchPush($eventsToWrite, $params, $args);
+    }
+
+     /**
+     * @param Event[] $eventsToWrite
+     * @param array<string, string> $params
+     * @param array<string, mixed> $args
+     * @throws \JsonException
+     * @throws \Exception|\Throwable
+     */
+    protected function batchPush(array $eventsToWrite, array $params = [], array $args = []): void
     {
         $this->logger?->info('Pushing batch events to outlook ...', [
             'params' => http_build_query($params)
@@ -246,7 +259,6 @@ public function push(array $params = [], array $args = []): void
 
         $batchRequestConfiguration = $this->getEventPostBatchRequestConfiguration();
 
-        $eventsToWrite = $this->getLocalEvents();
         $chunks = array_chunk($eventsToWrite, static::BATCH_BY);
 
         foreach ($chunks as $chunk) {