Replace deprecated methods and interfaces of react/promise

library/Pdfexport/HeadlessChrome.php

@@ -15,7 +15,7 @@
 use React\EventLoop\Loop;
 use React\EventLoop\TimerInterface;
 use React\Promise;
-use React\Promise\ExtendedPromiseInterface;
+use React\Promise\PromiseInterface;
 use Throwable;
 use WebSocket\Client;
 
@@ -190,7 +190,7 @@
 
         foreach ($arguments as $name => $value) {
             if ($value !== null) {
                 $value = escapeshellarg($value);
 
                 if (! is_int($name)) {
                     if (substr($name, -1) === '=') {
@@ -244,18 +244,18 @@
     /**
      * Generate a PDF raw string asynchronously.
      *
-     * @return ExtendedPromiseInterface
+     * @return PromiseInterface
      */
-    public function asyncToPdf(): ExtendedPromiseInterface
+    public function asyncToPdf(): PromiseInterface
     {
         $deferred = new Promise\Deferred();
         Loop::futureTick(function () use ($deferred) {
             switch (true) {
                 case $this->remote !== null:
                     try {
                         $result = $this->jsonVersion($this->remote[0], $this->remote[1]);
                         if (is_array($result)) {
                             $parts = explode('/', $result['webSocketDebuggerUrl']);
                             $pdf = $this->printToPDF(
                                 join(':', $this->remote),
                                 end($parts),
@@ -329,10 +329,10 @@
 
                     $chrome->start();
 
                     $chrome->stderr->on('data', function ($chunk) use ($chrome, $deferred, $killer) {
                         Logger::debug('Caught browser output: %s', $chunk);
 
                         if (preg_match(self::DEBUG_ADDR_PATTERN, trim($chunk), $matches)) {
                             Loop::cancelTimer($killer);
 
                             try {
@@ -397,9 +397,8 @@
     public function toPdf()
     {
         $pdf = '';
-        // We don't intend to register any then/otherwise handlers, so call done on that promise
-        // to properly propagate unhandled exceptions to the caller.
-        $this->asyncToPdf()->done(function (string $newPdf) use (&$pdf) {
+
+        $this->asyncToPdf()->then(function (string $newPdf) use (&$pdf) {
             $pdf = $newPdf;
         });
 
@@ -428,7 +427,7 @@
 
     private function printToPDF($socket, $browserId, array $parameters)
     {
         $browser = new Client(sprintf('ws://%s/devtools/browser/%s', $socket, $browserId));
 
         // Open new tab, get its id
         $result = $this->communicate($browser, 'Target.createTarget', [

library/Pdfexport/ProvidedHook/Pdfexport.php

@@ -14,7 +14,7 @@
 use Icinga\Module\Pdfexport\HeadlessChrome;
 use Icinga\Module\Pdfexport\PrintableHtmlDocument;
 use Karriere\PdfMerge\PdfMerge;
-use React\Promise\ExtendedPromiseInterface;
+use React\Promise\PromiseInterface;
 
 class Pdfexport extends PdfexportHook
 {
@@ -98,9 +98,9 @@ public function htmlToPdf($html)
      *
      * @param PrintableHtmlDocument|string $html
      *
-     * @return ExtendedPromiseInterface
+     * @return PromiseInterface
      */
-    public function asyncHtmlToPdf($html): ExtendedPromiseInterface
+    public function asyncHtmlToPdf($html): PromiseInterface
     {
         // Keep reference to the chrome object because it is using temp files which are automatically removed when
         // the object is destructed
@@ -109,7 +109,7 @@ public function asyncHtmlToPdf($html): ExtendedPromiseInterface
         $pdfPromise = $chrome->fromHtml($html, static::getForceTempStorage())->asyncToPdf();
 
         if ($html instanceof PrintableHtmlDocument && ($coverPage = $html->getCoverPage()) !== null) {
-            /** @var ExtendedPromiseInterface $pdfPromise */
+            /** @var PromiseInterface $pdfPromise */
             $pdfPromise = $pdfPromise->then(function (string $pdf) use ($chrome, $html, $coverPage) {
                 return $chrome->fromHtml(
                     (new PrintableHtmlDocument())