Slim4

application/BasePathMiddleware.php

@@ -0,0 +1,54 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Shaarli;
+
+use Selective\BasePath\BasePathDetector;
+use Psr\Http\Message\ResponseInterface;
+use Psr\Http\Message\ServerRequestInterface;
+use Psr\Http\Server\MiddlewareInterface;
+use Psr\Http\Server\RequestHandlerInterface;
+use Slim\App;
+
+/*
+ * This is a hack that to make BasePathMiddleware Selective's library without a public subdirectory,
+ * and without re-implementing the dectector.
+ */
+class BasePathMiddleware implements MiddlewareInterface
+{
+    /**
+     * @var App The slim app
+     */
+    private $app;
+
+    /**
+     * @var string|null
+     */
+    private $phpSapi;
+
+    /**
+     * The constructor.
+     *
+     * @param App $app The slim app
+     * @param string|null $phpSapi The PHP_SAPI value
+     */
+    public function __construct(App $app, string $phpSapi = null)
+    {
+        $this->app = $app;
+        $this->phpSapi = $phpSapi;
+    }
+
+    public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
+    {
+        $params = $request->getServerParams();
+        $pathParts = pathinfo($params['SCRIPT_NAME']);
+        $params['SCRIPT_NAME'] = $pathParts['dirname'] . '/public/' . $pathParts['basename'];
+
+        $detector = new BasePathDetector($params, $this->phpSapi);
+
+        $this->app->setBasePath($detector->getBasePath());
+
+        return $handler->handle($request);
+    }
+}

application/Languages.php

@@ -96,17 +96,19 @@ protected function initGettextTranslator()
     {
         $this->translator = new GettextTranslator();
         $this->translator->setLanguage($this->language);
-        $this->translator->loadDomain(self::DEFAULT_DOMAIN, 'inc/languages');
+        $this->translator->loadDomain(self::DEFAULT_DOMAIN, __DIR__ . '/../inc/languages');
 
         // Default extension translation from the current theme
-        $themeTransFolder = rtrim($this->conf->get('raintpl_tpl'), '/') . '/' . $this->conf->get('theme') . '/language';
+        $themeTransFolder = __DIR__ . '/../' . rtrim($this->conf->get('raintpl_tpl'), '/') . '/'
+            . $this->conf->get('theme') . '/language';
         if (is_dir($themeTransFolder)) {
             $this->translator->loadDomain($this->conf->get('theme'), $themeTransFolder, false);
         }
 
         foreach ($this->conf->get('translation.extensions', []) as $domain => $translationPath) {
+            $path = __DIR__ . '/../' . $translationPath;
             if ($domain !== self::DEFAULT_DOMAIN) {
-                $this->translator->loadDomain($domain, $translationPath, false);
+                $this->translator->loadDomain($domain, $path, false);
             }
         }
     }
@@ -123,7 +125,7 @@ protected function initPhpTranslator()
         // Core translations
         try {
             $translations = $translations->addFromPoFile(
-                'inc/languages/' . $this->language . '/LC_MESSAGES/shaarli.po'
+                __DIR__ . '/../inc/languages/' . $this->language . '/LC_MESSAGES/shaarli.po'
             );
             $translations->setDomain('shaarli');
             $this->translator->loadTranslations($translations);
@@ -132,7 +134,8 @@ protected function initPhpTranslator()
 
         // Default extension translation from the current theme
         $theme = $this->conf->get('theme');
-        $themeTransFolder = rtrim($this->conf->get('raintpl_tpl'), '/') . '/' . $theme . '/language';
+        $themeTransFolder = __DIR__ . '/../' . rtrim($this->conf->get('raintpl_tpl'), '/') . '/'
+            . $theme . '/language';
         if (is_dir($themeTransFolder)) {
             try {
                 $translations = Translations::fromPoFile(
@@ -152,7 +155,7 @@ protected function initPhpTranslator()
 
             try {
                 $extension = Translations::fromPoFile(
-                    $translationPath . $this->language . '/LC_MESSAGES/' . $domain . '.po'
+                    __DIR__ . '/../' . $translationPath . $this->language . '/LC_MESSAGES/' . $domain . '.po'
                 );
                 $extension->setDomain($domain);
                 $this->translator->loadTranslations($extension);

application/api/ApiMiddleware.php

@@ -3,13 +3,14 @@
 namespace Shaarli\Api;
 
 use malkusch\lock\mutex\FlockMutex;
+use Psr\Container\ContainerInterface as Container;
+use Psr\Http\Message\ResponseInterface as Response;
+use Psr\Http\Message\ServerRequestInterface as Request;
+use Psr\Http\Server\RequestHandlerInterface as RequestHandler;
 use Shaarli\Api\Exceptions\ApiAuthorizationException;
 use Shaarli\Api\Exceptions\ApiException;
 use Shaarli\Bookmark\BookmarkFileService;
 use Shaarli\Config\ConfigManager;
-use Slim\Container;
-use Slim\Http\Request;
-use Slim\Http\Response;
 
 /**
  * Class ApiMiddleware
@@ -43,7 +44,7 @@ class ApiMiddleware
      *
      * @param Container $container instance.
      */
-    public function __construct($container)
+    public function __construct(Container $container)
     {
         $this->container = $container;
         $this->conf = $this->container->get('conf');
@@ -62,13 +63,12 @@ public function __construct($container)
      *
      * @return Response response.
      */
-    public function __invoke($request, $response, $next)
+    public function __invoke(Request $request, RequestHandler $handler): Response
     {
         try {
             $this->checkRequest($request);
-            $response = $next($request, $response);
+            $response = $handler->handle($request);
         } catch (ApiException $e) {
-            $e->setResponse($response);
             $e->setDebug($this->conf->get('dev.debug', false));
             $response = $e->getApiResponse();
         }
@@ -111,7 +111,7 @@ protected function checkToken($request)
     {
         if (
             !$request->hasHeader('Authorization')
-            && !isset($this->container->environment['REDIRECT_HTTP_AUTHORIZATION'])
+            && !isset($request->getServerParams()['REDIRECT_HTTP_AUTHORIZATION'])
         ) {
             throw new ApiAuthorizationException('JWT token not provided');
         }
@@ -120,8 +120,8 @@ protected function checkToken($request)
             throw new ApiAuthorizationException('Token secret must be set in Shaarli\'s administration');
         }
 
-        if (isset($this->container->environment['REDIRECT_HTTP_AUTHORIZATION'])) {
-            $authorization = $this->container->environment['REDIRECT_HTTP_AUTHORIZATION'];
+        if (isset($request->getServerParams()['REDIRECT_HTTP_AUTHORIZATION'])) {
+            $authorization = $request->getServerParams()['REDIRECT_HTTP_AUTHORIZATION'];
         } else {
             $authorization = $request->getHeaderLine('Authorization');
         }
@@ -150,6 +150,6 @@ protected function setLinkDb($conf)
             new FlockMutex(fopen(SHAARLI_MUTEX_FILE, 'r'), 2),
             true
         );
-        $this->container['db'] = $linkDb;
+        $this->container->set('db', $linkDb);
     }
 }

application/api/controllers/ApiController.php

@@ -2,10 +2,11 @@
 
 namespace Shaarli\Api\Controllers;
 
+use DI\Container;
+use Psr\Http\Message\ResponseInterface as Response;
 use Shaarli\Bookmark\BookmarkServiceInterface;
 use Shaarli\Config\ConfigManager;
 use Shaarli\History;
-use Slim\Container;
 
 /**
  * Abstract Class ApiController
@@ -70,4 +71,19 @@ public function getCi()
     {
         return $this->ci;
     }
+
+    /**
+     * Simple helper, which writes data as JSON to the body
+     *
+     * @param Response $response
+     * @param array $data
+     * @param ?int $jsonStyle
+     * @return Response
+     */
+    protected function respondWithJson(Response $response, array $data, ?int $jsonStyle): Response
+    {
+        $jsonStyle = $jsonStyle ?? 0;
+        $response->getBody()->write(json_encode($data, $jsonStyle));
+        return $response->withHeader('Content-Type', 'application/json');
+    }
 }

application/api/controllers/HistoryController.php

@@ -2,9 +2,10 @@
 
 namespace Shaarli\Api\Controllers;
 
+use Psr\Http\Message\ResponseInterface as Response;
+use Psr\Http\Message\ServerRequestInterface as Request;
 use Shaarli\Api\Exceptions\ApiBadParametersException;
-use Slim\Http\Request;
-use Slim\Http\Response;
+use Shaarli\ResponseUtils;
 
 /**
  * Class History
@@ -30,8 +31,8 @@ public function getHistory($request, $response)
         $history = $this->history->getHistory();
 
         // Return history operations from the {offset}th, starting from {since}.
-        $since = \DateTime::createFromFormat(\DateTime::ATOM, $request->getParam('since', ''));
-        $offset = $request->getParam('offset');
+        $since = \DateTime::createFromFormat(\DateTime::ATOM, $request->getQueryParams()['since'] ?? '');
+        $offset = $request->getQueryParams()['offset'] ?? null;
         if (empty($offset)) {
             $offset = 0;
         } elseif (ctype_digit($offset)) {
@@ -41,7 +42,7 @@ public function getHistory($request, $response)
         }
 
         // limit parameter is either a number of bookmarks or 'all' for everything.
-        $limit = $request->getParam('limit');
+        $limit = $request->getQueryParams()['limit'] ?? null;
         if (empty($limit)) {
             $limit = count($history);
         } elseif (ctype_digit($limit)) {
@@ -63,6 +64,6 @@ public function getHistory($request, $response)
         }
         $out = array_values($out);
 
-        return $response->withJson($out, 200, $this->jsonStyle);
+        return $this->respondWithJson($response, $out, $this->jsonStyle)->withStatus(200);
     }
 }

application/api/controllers/Info.php

@@ -2,9 +2,10 @@
 
 namespace Shaarli\Api\Controllers;
 
+use Psr\Http\Message\ResponseInterface as Response;
+use Psr\Http\Message\ServerRequestInterface as Request;
 use Shaarli\Bookmark\BookmarkFilter;
-use Slim\Http\Request;
-use Slim\Http\Response;
+use Shaarli\ResponseUtils;
 
 /**
  * Class Info
@@ -39,6 +40,6 @@ public function getInfo($request, $response)
             ],
         ];
 
-        return $response->withJson($info, 200, $this->jsonStyle);
+        return $this->respondWithJson($response, $info, $this->jsonStyle)->withStatus(200);
     }
 }