Minor: Format code

Author: AngelFQC

public/plugin/xapi/admin.php

@@ -1,16 +1,19 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 use Chamilo\PluginBundle\Entity\XApi\LrsAuth;
+use Symfony\Component\HttpFoundation\Request;
 
 $cidReset = true;
 
 require_once __DIR__.'/../../main/inc/global.inc.php';
 
 api_protect_admin_script();
 
-$request = \Symfony\Component\HttpFoundation\Request::createFromGlobals();
+$request = Request::createFromGlobals();
 $plugin = XApiPlugin::create();
 $em = Database::getManager();
 
@@ -19,11 +22,11 @@
 $pageContent = '';
 
 /**
- * @throws \Exception
+ * @return FormValidator
  *
- * @return \FormValidator
+ * @throws Exception
  */
-function createForm(LrsAuth $auth = null)
+function createForm(?LrsAuth $auth = null)
 {
     $pageBaseUrl = api_get_self();
 
@@ -67,7 +70,8 @@ function createForm(LrsAuth $auth = null)
                 ->setEnabled(isset($values['enabled']))
                 ->setCreatedAt(
                     api_get_utc_datetime(null, false, true)
-                );
+                )
+            ;
 
             $em->persist($auth);
             $em->flush();
@@ -77,6 +81,7 @@ function createForm(LrsAuth $auth = null)
             );
 
             header('Location: '.$pageBaseUrl);
+
             exit;
         }
 
@@ -85,7 +90,9 @@ function createForm(LrsAuth $auth = null)
             $pageBaseUrl
         );
         $pageContent = $form->returnForm();
+
         break;
+
     case 'edit':
         $auth = $em->find(LrsAuth::class, $request->query->getInt('id'));
 
@@ -104,7 +111,8 @@ function createForm(LrsAuth $auth = null)
                 ->setEnabled(isset($values['enabled']))
                 ->setCreatedAt(
                     api_get_utc_datetime(null, false, true)
-                );
+                )
+            ;
 
             $em->persist($auth);
             $em->flush();
@@ -114,6 +122,7 @@ function createForm(LrsAuth $auth = null)
             );
 
             header('Location: '.$pageBaseUrl);
+
             exit;
         }
 
@@ -122,7 +131,9 @@ function createForm(LrsAuth $auth = null)
             $pageBaseUrl
         );
         $pageContent = $form->returnForm();
+
         break;
+
     case 'delete':
         $auth = $em->find(LrsAuth::class, $request->query->getInt('id'));
 
@@ -138,7 +149,9 @@ function createForm(LrsAuth $auth = null)
         );
 
         header('Location: '.$pageBaseUrl);
+
         exit;
+
     case 'list':
     default:
         $pageActions = Display::url(
@@ -177,11 +190,12 @@ function createForm(LrsAuth $auth = null)
                 $table->setCellContents($row, 1, $auth->getPassword());
                 $table->setCellContents($row, 2, $auth->isEnabled() ? get_lang('Yes') : get_lang('No'));
                 $table->setCellContents($row, 3, api_convert_and_format_date($auth->getCreatedAt()));
-                $table->setCellContents($row, 4, implode(PHP_EOL, $actions));
+                $table->setCellContents($row, 4, implode(\PHP_EOL, $actions));
             }
 
             $pageContent = $table->toHtml();
         }
+
         break;
 }
 

public/plugin/xapi/cmi5/launch.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 use Chamilo\PluginBundle\Entity\XApi\Cmi5Item;
@@ -74,7 +76,8 @@
         api_get_setting('Institution').' - '.api_get_setting('siteName')
     )
     ->withLanguage(api_get_language_isocode())
-    ->withRegistration($registration);
+    ->withRegistration($registration)
+;
 
 $statementUuid = Uuid::uuid5(
     $plugin->get(XApiPlugin::SETTING_UUID_NAMESPACE),
@@ -95,17 +98,17 @@
 
 $statementClient = XApiPlugin::create()->getXApiStatementClient();
 
-//try {
+// try {
 //    $statementClient->storeStatement($statement);
-//} catch (ConflictException $e) {
+// } catch (ConflictException $e) {
 //    echo Display::return_message($e->getMessage(), 'error');
 //
 //    exit;
-//} catch (XApiException $e) {
+// } catch (XApiException $e) {
 //    echo Display::return_message($e->getMessage(), 'error');
 //
 //    exit;
-//}
+// }
 
 $viewSessionId = (string) Uuid::uuid4();
 
@@ -146,7 +149,8 @@
         ->getXApiStateClient()
         ->createOrReplaceDocument(
             new StateDocument($state, $documentData)
-        );
+        )
+    ;
 } catch (Exception $exception) {
     echo Display::return_message($exception->getMessage(), 'error');
 

public/plugin/xapi/cmi5/token.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 use Symfony\Component\HttpFoundation\JsonResponse;

public/plugin/xapi/cmi5/view.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 use Chamilo\PluginBundle\Entity\XApi\Cmi5Item;
@@ -25,6 +27,7 @@
     || 'cmi5' !== $toolLaunch->getActivityType()
 ) {
     header('Location: '.api_get_course_url());
+
     exit;
 }
 
@@ -42,7 +45,8 @@
     ->from(Cmi5Item::class, 'item')
     ->where('item.tool = :tool')
     ->setParameter('tool', $toolLaunch->getId())
-    ->getQuery();
+    ->getQuery()
+;
 
 $tocHtml = $itemsRepo->buildTree(
     $query->getArrayResult(),

public/plugin/xapi/cron/send_statements.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 use Chamilo\PluginBundle\Entity\XApi\SharedStatement;
@@ -12,11 +14,11 @@
 
 require_once __DIR__.'/../../../main/inc/global.inc.php';
 
-if (php_sapi_name() !== 'cli') {
+if (\PHP_SAPI !== 'cli') {
     exit;
 }
 
-echo 'XAPI: Cron to send statements.'.PHP_EOL;
+echo 'XAPI: Cron to send statements.'.\PHP_EOL;
 
 $em = Database::getManager();
 $serializer = Serializer::createSerializer();
@@ -28,11 +30,12 @@
         ['uuid' => null, 'sent' => false],
         null,
         100
-    );
+    )
+;
 $countNotSent = count($notSentSharedStatements);
 
 if ($countNotSent > 0) {
-    echo '['.time().'] Trying to send '.$countNotSent.' statements to LRS'.PHP_EOL;
+    echo '['.time().'] Trying to send '.$countNotSent.' statements to LRS'.\PHP_EOL;
 
     $client = XApiPlugin::create()->getXapiStatementCronClient();
 
@@ -55,25 +58,26 @@
 
             echo "\t\tStatement ID received: \"{$sentStatement->getId()->getValue()}\"";
         } catch (ConflictException $e) {
-            echo $e->getMessage().PHP_EOL;
+            echo $e->getMessage().\PHP_EOL;
 
             continue;
         } catch (XApiException $e) {
-            echo $e->getMessage().PHP_EOL;
+            echo $e->getMessage().\PHP_EOL;
 
             continue;
         }
 
         $notSentSharedStatement
             ->setUuid($sentStatement->getId()->getValue())
-            ->setSent(true);
+            ->setSent(true)
+        ;
 
         $em->persist($notSentSharedStatement);
 
-        echo "\t\tShared statement updated".PHP_EOL;
+        echo "\t\tShared statement updated".\PHP_EOL;
     }
 
     $em->flush();
 } else {
-    echo 'No statements to process.'.PHP_EOL;
+    echo 'No statements to process.'.\PHP_EOL;
 }

public/plugin/xapi/install.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 XApiPlugin::create()->install();

public/plugin/xapi/lang/english.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 $strings['plugin_title'] = 'Experience API (xAPI)';

public/plugin/xapi/lang/french.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 $strings['plugin_title'] = 'Experience API (xAPI)';

public/plugin/xapi/lang/spanish.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 $strings['plugin_title'] = 'Experience API (xAPI)';

public/plugin/xapi/lrs.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 /* For licensing terms, see /license.txt */
 
 use Chamilo\PluginBundle\XApi\Lrs\LrsRequest;