fix(arstechnica): plus a few unrelated tweaks

dvikan · Dec 13, 2023 · 9a86141 · 9a86141
1 parent 0b67544
commit 9a86141
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 17 deletions.
diff --git a/actions/FrontpageAction.php b/actions/FrontpageAction.php
@@ -31,6 +31,7 @@ public function execute(array $request)
             }
         }
 
+        // todo: cache this renderered template
         return render(__DIR__ . '/../templates/frontpage.html.php', [
             'messages' => $messages,
             'admin_email' => Configuration::getConfig('admin', 'email'),

diff --git a/bridges/ArsTechnicaBridge.php b/bridges/ArsTechnicaBridge.php
@@ -37,7 +37,14 @@ protected function parseItem(array $item)
     {
         $item_html = getSimpleHTMLDOMCached($item['uri'] . '&amp');
         $item_html = defaultLinkTo($item_html, self::URI);
-        $item['content'] = $item_html->find('.amp-wp-article-content', 0);
+
+        $item_content = $item_html->find('.article-content.post-page', 0);
+        if (!$item_content) {
+            // The dom selector probably broke. Let's just return the item as-is
+            return $item;
+        }
+
+        $item['content'] = $item_content;
 
         // remove various ars advertising
         $item['content']->find('#social-left', 0)->remove();

diff --git a/bridges/VkBridge.php b/bridges/VkBridge.php
@@ -523,7 +523,7 @@ private function getContents()
             }
 
             if (!preg_match('#^https?://vk.com/#', $uri)) {
-                returnServerError('Unexpected redirect location');
+                returnServerError('Unexpected redirect location: ' . $uri);
             }
 
             $redirects++;

diff --git a/caches/FileCache.php b/caches/FileCache.php
@@ -49,8 +49,8 @@ public function set($key, $value, int $ttl = null): void
     {
         $item = [
             'key'           => $key,
-            'value'         => $value,
             'expiration'    => $ttl === null ? 0 : time() + $ttl,
+            'value'         => $value,
         ];
         $cacheFile = $this->createCacheFile($key);
         $bytes = file_put_contents($cacheFile, serialize($item), LOCK_EX);

diff --git a/index.php b/index.php
@@ -6,7 +6,11 @@
 
 require_once __DIR__ . '/lib/bootstrap.php';
 
-Configuration::verifyInstallation();
+$errors = Configuration::checkInstallation();
+if ($errors) {
+    die('<pre>' . implode("\n", $errors) . '</pre>');
+}
+
 $customConfig = [];
 if (file_exists(__DIR__ . '/config.ini.php')) {
     $customConfig = parse_ini_file(__DIR__ . '/config.ini.php', true, INI_SCANNER_TYPED);

diff --git a/lib/Configuration.php b/lib/Configuration.php
@@ -15,15 +15,7 @@ private function __construct()
     {
     }
 
-    /**
-     * Verifies the current installation of RSS-Bridge and PHP.
-     *
-     * Returns an error message and aborts execution if the installation does
-     * not satisfy the requirements of RSS-Bridge.
-     *
-     * @return void
-     */
-    public static function verifyInstallation()
+    public static function checkInstallation(): array
     {
         $errors = [];
 
@@ -57,10 +49,7 @@ public static function verifyInstallation()
         if (!extension_loaded('json')) {
             $errors[] = 'json extension not loaded';
         }
-
-        if ($errors) {
-            throw new \Exception(sprintf('Configuration error: %s', implode(', ', $errors)));
-        }
+        return $errors;
     }
 
     public static function loadConfiguration(array $customConfig = [], array $env = [])