Issue #56: Add strict_types declaration, don't rely on implicit cast.

CollaboraOnline · Dec 3, 2024 · d9ce6af · d9ce6af
1 parent adc2b6d
commit d9ce6af
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/Controller/ViewerController.php b/src/Controller/ViewerController.php
@@ -10,6 +10,8 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
+declare(strict_types=1);
+
 namespace Drupal\collabora_online\Controller;
 
 use Drupal\collabora_online\Cool\CollaboraDiscovery;
@@ -96,7 +98,7 @@ public function editor(MediaInterface $media, Request $request, $edit = FALSE) {
     $render_array['#attached']['library'][] = 'collabora_online/cool.frame';
 
     $response = new Response();
-    $response->setContent($this->renderer->renderRoot($render_array));
+    $response->setContent((string) $this->renderer->renderRoot($render_array));
 
     return $response;
   }

diff --git a/src/Cool/CollaboraDiscovery.php b/src/Cool/CollaboraDiscovery.php
@@ -10,6 +10,8 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/.
  */
 
+declare(strict_types=1);
+
 namespace Drupal\collabora_online\Cool;
 
 use Drupal\collabora_online\Exception\CollaboraNotAvailableException;