Issue CollaboraOnline#43: Normalize spaces.

donquixote · Nov 8, 2024 · 7d2bdfe · 7d2bdfe
1 parent e20aa29
commit 7d2bdfe
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/src/Controller/WopiController.php b/src/Controller/WopiController.php
@@ -184,7 +184,7 @@ function wopiPutFile(string $id, Request $request) {
                 \Drupal::logger('cool')->error('Conflict saving file ' . $id . ' wopi: ' . $wopi_stamp->format('c') . ' differs from file: ' . $file_stamp->format('c'));
 
                 return new Response(
-                    json_encode([ 'COOLStatusCode' => 1010 ]),
+                    json_encode(['COOLStatusCode' => 1010]),
                     Response::HTTP_CONFLICT,
                     ['content-type' => 'application/json'],
                 );
@@ -224,13 +224,13 @@ function wopiPutFile(string $id, Request $request) {
             $reasons[] = 'Save on Exit';
         }
         if (count($reasons) > 0) {
-            $save_reason .= ' (' . implode(', ', $reasons)  . ')';
+            $save_reason .= ' (' . implode(', ', $reasons) . ')';
         }
         \Drupal::logger('cool')->error('Save reason: ' . $save_reason);
         $media->setRevisionLogMessage($save_reason);
         $media->save();
 
-        $payload =  json_encode([
+        $payload = json_encode([
             'LastModifiedTime' => $mtime->format('c'),
         ]);
 

diff --git a/src/Cool/CoolRequest.php b/src/Cool/CoolRequest.php
@@ -22,13 +22,13 @@
  *   The full contents of discovery.xml, or FALSE on failure.
  */
 function getDiscovery($server) {
-    $discovery_url = $server.'/hosting/discovery';
+    $discovery_url = $server . '/hosting/discovery';
 
     $default_config = \Drupal::config('collabora_online.settings');
     if ($default_config === null) {
         return false;
     }
-    $disable_checks = (bool)$default_config->get('cool')['disable_cert_check'];
+    $disable_checks = (bool) $default_config->get('cool')['disable_cert_check'];
 
     $stream_context = stream_context_create([
         'ssl' => [

diff --git a/tests/src/ExistingSiteJavascript/CollaboraIntegrationTest.php b/tests/src/ExistingSiteJavascript/CollaboraIntegrationTest.php
@@ -40,14 +40,14 @@ public function testCollaboraPreview(): void {
 
         $document_field = $assert_session->waitForElement('css', 'input#document-name-input');
         $this->assertNotNull($document_field, 'The document name input was not found after 10 seconds.');
-        $this->getCurrentPage()->waitFor(10, function() use ($document_field) {
+        $this->getCurrentPage()->waitFor(10, function () use ($document_field) {
           return $document_field->getValue() === 'shopping-list.txt';
         });
         $this->assertEquals('shopping-list.txt', $document_field->getValue(), 'The document name input did not contain the correct value after 10 seconds.');
 
         $word_count_element = $assert_session->waitForElement('css', 'div#StateWordCount');
         $this->assertNotNull($word_count_element, 'The word count element was not found after 10 seconds.');
-        $this->getCurrentPage()->waitFor(10, function() use ($word_count_element) {
+        $this->getCurrentPage()->waitFor(10, function () use ($word_count_element) {
           return $word_count_element->getText() === '2 words, 18 characters';
         });
         $this->assertEquals('2 words, 18 characters', $word_count_element->getText(), 'The word count element did not contain the correct text after 10 seconds.');