Dispositif / Wikibot

src/Domain/ExternLink/CheckURL.php

Spacing +4 added lines, -4 removed lines

@@ -49,12 +49,12 @@ 
         $this->url = $url;
         $this->registrableDomain = null;
         if (!HttpUtil::isHttpURL($url)) {
-            $this->log->debug('Skip : not Http URL : ' . $url, ['stats' => 'externref.skip.notRawURL']);
+            $this->log->debug('Skip : not Http URL : '.$url, ['stats' => 'externref.skip.notRawURL']);
             return false;
         }
 
         if ($this->hasForbiddenFilenameExtension()) {
-            $this->log->debug('Skip : ForbiddenFilenameExtension : ' . $url, ['stats' => 'externref.skip.forbiddenFilenameExtension']);
+            $this->log->debug('Skip : ForbiddenFilenameExtension : '.$url, ['stats' => 'externref.skip.forbiddenFilenameExtension']);
             return false;
         }
 
@@ -70,7 +70,7 @@ 
      */
     protected function hasForbiddenFilenameExtension(): bool
     {
-        return (bool)preg_match(
+        return (bool) preg_match(
             '#\.(pdf|jpg|jpeg|gif|png|webp|xls|xlsx|xlr|xml|xlt|txt|csv|js|docx|exe|gz|zip|ini|movie|mp3|mp4|ogg|raw|rss|tar|tgz|wma)$#i',
             $this->url
         );
@@ -81,7 +81,7 @@ 
         try {
             $this->registrableDomain = $this->internetDomainParser->getRegistrableDomainFromURL($this->url);
         } catch (Exception) {
-            $this->log->warning('Skip : not a valid URL : ' . $this->url,
+            $this->log->warning('Skip : not a valid URL : '.$this->url,
                 ['stats' => 'externref.skip.URLAuthorized.exception2']
             );
             return null;

src/Domain/ExternLink/ExternRefTransformer.php

Spacing +16 added lines, -16 removed lines

@@ -35,13 +35,13 @@ 
     use SummaryExternTrait, PublisherLogicTrait;
 
     final public const HTTP_REQUEST_LOOP_DELAY = 10;
-    final public const SKIP_DOMAIN_FILENAME = __DIR__ . '/../resources/config_skip_domain.txt';
+    final public const SKIP_DOMAIN_FILENAME = __DIR__.'/../resources/config_skip_domain.txt';
     final public const REPLACE_404 = true;
     final public const REPLACE_410 = true;
-    final public const CONFIG_PRESSE = __DIR__ . '/../resources/config_presse.yaml';
-    final public const CONFIG_NEWSPAPER_JSON = __DIR__ . '/../resources/data_newspapers.json';
-    final public const CONFIG_SCIENTIFIC_JSON = __DIR__ . '/../resources/data_scientific_domain.json';
-    final public const CONFIG_SCIENTIFIC_WIKI_JSON = __DIR__ . '/../resources/data_scientific_wiki.json';
+    final public const CONFIG_PRESSE = __DIR__.'/../resources/config_presse.yaml';
+    final public const CONFIG_NEWSPAPER_JSON = __DIR__.'/../resources/data_newspapers.json';
+    final public const CONFIG_SCIENTIFIC_JSON = __DIR__.'/../resources/data_scientific_domain.json';
+    final public const CONFIG_SCIENTIFIC_WIKI_JSON = __DIR__.'/../resources/data_scientific_wiki.json';
 
     public bool $skipSiteBlacklisted = true;
     public bool $skipRobotNoIndex = true;
@@ -96,13 +96,13 @@ 
         }
         $this->registrableDomain = $this->urlChecker->getRegistrableDomain($url); // hack
         if ($this->isSiteBlackListed()) {
-            $this->log->debug('Site blacklisted : ' . $this->registrableDomain, ['stats' => 'externref.skip.blacklisted']);
+            $this->log->debug('Site blacklisted : '.$this->registrableDomain, ['stats' => 'externref.skip.blacklisted']);
             return $url;
         }
 
         if (!$this->validateConfigWebDomain($this->registrableDomain)) {
             $this->log->debug(
-                'Domain blocked by config : ' . $this->registrableDomain,
+                'Domain blocked by config : '.$this->registrableDomain,
                 ['stats' => 'externref.skip.domainDisabledByConfig']
             );
             return $url;
@@ -157,7 +157,7 @@ 
     protected function isSiteBlackListed(): bool
     {
         if ($this->skipSiteBlacklisted && in_array($this->registrableDomain, $this->skip_domain)) {
-            $this->log->notice("Skip web site " . $this->registrableDomain);
+            $this->log->notice("Skip web site ".$this->registrableDomain);
             return true;
         }
         return false;
@@ -175,7 +175,7 @@ 
         $this->config[$domain] = is_array($this->config[$domain]) ? $this->config[$domain] : [];
 
         if ($this->config[$domain] === 'deactivated' || isset($this->config[$domain]['deactivated'])) {
-            $this->log->info("Domain " . $domain . " disabled\n");
+            $this->log->info("Domain ".$domain." disabled\n");
 
             return false;
         }
@@ -186,9 +186,9 @@ 
     protected function logDebugConfigWebDomain(string $domain): void
     {
         if (!isset($this->config[$domain])) {
-            $this->log->debug("Domain " . $domain . " non configuré");
+            $this->log->debug("Domain ".$domain." non configuré");
         } else {
-            $this->log->debug("Domain " . $domain . " configuré");
+            $this->log->debug("Domain ".$domain." configuré");
         }
     }
 
@@ -214,7 +214,7 @@ 
         if ($pageData === []
             || (empty($pageData['JSON-LD']) && empty($pageData['meta']))
         ) {
-            $this->log->notice('No metadata : ' . $url);
+            $this->log->notice('No metadata : '.$url);
 
             return true;
         }
@@ -229,7 +229,7 @@ 
     protected function emptyMapData(array $mapData, string $url): bool
     {
         if ($mapData === [] || empty($mapData['url']) || empty($mapData['titre'])) {
-            $this->log->info('Mapping incomplet : ' . $url);
+            $this->log->info('Mapping incomplet : '.$url);
 
             return true;
         }
@@ -282,7 +282,7 @@ 
 
         $template = WikiTemplateFactory::create($templateName);
         $template->userSeparator = " |";
-        $this->summary->memo['count ' . $templateName] = 1 + ($this->summary->memo['count ' . $templateName] ?? 0);
+        $this->summary->memo['count '.$templateName] = 1 + ($this->summary->memo['count '.$templateName] ?? 0);
 
         return $template;
     }
@@ -322,7 +322,7 @@ 
     protected function correctSiteViaWebarchiver(array $mapData): array
     {
         if (!empty($this->options['originalRegistrableDomain']) && $mapData['site']) {
-            $mapData['site'] = $this->options['originalRegistrableDomain'] . ' via ' . $mapData['site'];
+            $mapData['site'] = $this->options['originalRegistrableDomain'].' via '.$mapData['site'];
         }
 
         return $mapData;
@@ -346,7 +346,7 @@ 
         $templateOptimized = $optimizer->getOptiTemplate();
 
         $serialized = $templateOptimized->serialize(true);
-        $this->log->info('Serialized 444: ' . $serialized . "\n");
+        $this->log->info('Serialized 444: '.$serialized."\n");
         return $serialized;
     }
 }

src/Domain/ExternLink/ExternHttpErrorLogic.php

Spacing +5 added lines, -5 removed lines

@@ -18,7 +18,7 @@ 
  */
 class ExternHttpErrorLogic
 {
-    final public const LOG_REQUEST_ERROR = __DIR__ . '/../../Application/resources/external_request_error.log';
+    final public const LOG_REQUEST_ERROR = __DIR__.'/../../Application/resources/external_request_error.log';
     protected const LOOSE = true;
 
     public function __construct(
@@ -40,12 +40,12 @@ 
             return $url;
         }
         if (preg_match('#400 Bad Request#i', $errorMessage)) {
-            $this->log->warning('400 Bad Request : ' . $url, ['stats' => 'externHttpErrorLogic.400']);
+            $this->log->warning('400 Bad Request : '.$url, ['stats' => 'externHttpErrorLogic.400']);
 
             return $url;
         }
         if (preg_match('#(403 Forbidden|403 Access Forbidden)#i', $errorMessage)) {
-            $this->log->warning('403 Forbidden : ' . $url, ['stats' => 'externHttpErrorLogic.403']);
+            $this->log->warning('403 Forbidden : '.$url, ['stats' => 'externHttpErrorLogic.403']);
             // TODO return blankLienWeb without consulté le=...
 
             return $url;
@@ -59,7 +59,7 @@ 
             return $url;
         }
         if (preg_match('#401 (Unauthorized|Authorization Required)#i', $errorMessage)) {
-            $this->log->notice('401 Unauthorized : skip ' . $url, ['stats' => 'externHttpErrorLogic.401']);
+            $this->log->notice('401 Unauthorized : skip '.$url, ['stats' => 'externHttpErrorLogic.401']);
 
             return $url;
         }
@@ -112,7 +112,7 @@ 
         // "cURL error 28: Connection timed out after 20005 milliseconds (see https://curl.haxx.se/libcurl/c/libcurl-errors.html)
         //"cURL error 28: Connection timed out after 20005 milliseconds (see https://curl.haxx.se/libcurl/c/libcurl-errors.html)
         $this->log->notice(
-            'erreur non gérée sur extractWebData: "' . $errorMessage . "\" URL: " . $url,
+            'erreur non gérée sur extractWebData: "'.$errorMessage."\" URL: ".$url,
             ['stats' => 'externHttpErrorLogic.defaultSkip']
         );
         //file_put_contents(self::LOG_REQUEST_ERROR, $this->domain."\n", FILE_APPEND);

src/Domain/ExternLink/DeadLinkTransformer.php

Spacing +4 added lines, -4 removed lines

@@ -56,7 +56,7 @@
                 if ($webarchiveDTO->getArchiver() === '[[Internet Archive]]') {
                     $this->log->notice('

src/Infrastructure/Monitor/Stats.php

Spacing +7 added lines, -7 removed lines

@@ -18,7 +18,7 @@ 
  */
 class Stats
 {
-    protected const DEFAULT_FILEPATH = __DIR__ . '/../../../log/stats.db';
+    protected const DEFAULT_FILEPATH = __DIR__.'/../../../log/stats.db';
     protected const MAX_TAG_LENGTH = 100;
 
     protected SQLite3 $db;
@@ -51,7 +51,7 @@ 
     {
         $tag = $this->formatTag($tag);
         try {
-            return $this->db->exec('INSERT OR REPLACE INTO tagnum (tag,num) VALUES("' . $tag . '", ' . $num . ')');
+            return $this->db->exec('INSERT OR REPLACE INTO tagnum (tag,num) VALUES("'.$tag.'", '.$num.')');
         } catch (Exception $e) {
             return false;
         }
@@ -62,7 +62,7 @@ 
         $tag = $this->formatTag($tag);
         try {
             return $this->db->exec(
-                'INSERT INTO tagnum (tag) VALUES("' . $tag . '") ON CONFLICT(tag) DO UPDATE SET num=num-1'
+                'INSERT INTO tagnum (tag) VALUES("'.$tag.'") ON CONFLICT(tag) DO UPDATE SET num=num-1'
             );
         } catch (Exception $e) {
             return false;
@@ -87,17 +87,17 @@ 
     {
         try {
             $this->db->exec('CREATE TABLE if not exists tagnum_monthly (
-                tag VARCHAR(' . self::MAX_TAG_LENGTH . ') NOT NULL PRIMARY KEY,
+                tag VARCHAR(' . self::MAX_TAG_LENGTH.') NOT NULL PRIMARY KEY,
                 num int(11) NOT NULL DEFAULT 1
             )');
 
             $this->db->exec('CREATE TABLE if not exists tagnum_daily (
-                tag VARCHAR(' . self::MAX_TAG_LENGTH . ') NOT NULL PRIMARY KEY,
+                tag VARCHAR(' . self::MAX_TAG_LENGTH.') NOT NULL PRIMARY KEY,
                 num int(11) NOT NULL DEFAULT 1
             )');
 
             return $this->db->exec('CREATE TABLE if not exists tagnum (
-                tag VARCHAR(' . self::MAX_TAG_LENGTH . ') NOT NULL PRIMARY KEY,
+                tag VARCHAR(' . self::MAX_TAG_LENGTH.') NOT NULL PRIMARY KEY,
                 num int(11) NOT NULL DEFAULT 1
             )');
         } catch (Exception $e) {
@@ -111,7 +111,7 @@ 
         try {
             // upsert :)
             return $this->db->exec(
-                'INSERT INTO '.$table.' (tag) VALUES("' . $tag . '") ON CONFLICT(tag) DO UPDATE SET num=num+1'
+                'INSERT INTO '.$table.' (tag) VALUES("'.$tag.'") ON CONFLICT(tag) DO UPDATE SET num=num+1'
             );
         } catch (Exception $e) {
             return false;

src/Infrastructure/Monitor/ConsoleLogger.php

Spacing +11 added lines, -11 removed lines

@@ -24,7 +24,7 @@ 
     public bool $debug = false;
     public bool $colorMode = false;
 
-    public function __construct(public Stats|NullStats $stats = new Stats())
+    public function __construct(public Stats | NullStats $stats = new Stats())
     {
         try {
             $this->stats->increment('test.consolelogger');
@@ -35,7 +35,7 @@ 
 
     public function __call(string $method, array $args): void
     {
-        $this->notice('Call to undefined method ConsoleLogger:' . $method . '()');
+        $this->notice('Call to undefined method ConsoleLogger:'.$method.'()');
     }
 
     /**
@@ -61,7 +61,7 @@ 
             case 'emergency':
             case 'alert':
             case 'critical':
-                $this->echoColor("[$level] " . $date . ' : ' . $message . "\n", Color::BG_RED . Color::WHITE);
+                $this->echoColor("[$level] ".$date.' : '.$message."\n", Color::BG_RED.Color::WHITE);
                 if ($context !== []) {
                     dump($context);
                 }
@@ -69,20 +69,20 @@ 
                 break;
             case 'error':
             case 'warning':
-                $this->echoColor("[$level] " . $date . ' : ' . $message . "\n", Color::BG_YELLOW . Color::BLACK);
+                $this->echoColor("[$level] ".$date.' : '.$message."\n", Color::BG_YELLOW.Color::BLACK);
                 if ($context !== []) {
                     dump($context);
                 }
                 break;
             case 'notice':
-                $this->echoColor("[$level] " . $message . "\n");
+                $this->echoColor("[$level] ".$message."\n");
                 if ($context !== []) {
                     dump($context);
                 }
                 break;
             case 'info':
                 if ($this->verbose || $this->debug) {
-                    $this->echoColor("[$level] " . $message . "\n", Color::GRAY);
+                    $this->echoColor("[$level] ".$message."\n", Color::GRAY);
                     if ($context !== []) {
                         dump($context);
                     }
@@ -90,14 +90,14 @@ 
                 break;
             case 'debug':
                 if ($this->debug) {
-                    $this->echoColor("[$level] " . $message . "\n", Color::GRAY);
+                    $this->echoColor("[$level] ".$message."\n", Color::GRAY);
                     if ($context !== []) {
                         dump($context);
                     }
                 }
                 break;
             case 'echo':
-                $this->echoColor($message . "\n");
+                $this->echoColor($message."\n");
                 break;
         }
     }
@@ -120,7 +120,7 @@ 
     private function echoColor(string $text, ?string $color = null): void
     {
         if ($this->colorMode && !empty($color)) {
-            echo $color . $text . Color::NORMAL;
+            echo $color.$text.Color::NORMAL;
 
             return;
         }
@@ -130,8 +130,8 @@ 
     private function logInFile($level, string $message): void
     {
         file_put_contents(
-            __DIR__ . '/resources/critical.log',
-            date('d-m-Y H:i:s') . " : $level : " . $message . PHP_EOL,
+            __DIR__.'/resources/critical.log',
+            date('d-m-Y H:i:s')." : $level : ".$message.PHP_EOL,
             FILE_APPEND
         );
     }

src/Application/AbstractBotTaskWorker.php

Spacing +4 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
     public const SLEEP_AFTER_EDITION = 60;
     public const MINUTES_DELAY_AFTER_LAST_HUMAN_EDIT = 15;
     public const CHECK_EDIT_CONFLICT = true;
-    public const ARTICLE_ANALYZED_FILENAME = __DIR__ . '/resources/article_edited.txt';
+    public const ARTICLE_ANALYZED_FILENAME = __DIR__.'/resources/article_edited.txt';
     public const SKIP_LASTEDIT_BY_BOT = true;
     public const SKIP_NOT_IN_MAIN_WIKISPACE = true;
     public const SKIP_ADQ = true;
@@ -95,7 +95,7 @@ 
         $this->initializePastAnalyzedTitles();
 
         // @throw exception on "Invalid CSRF token"
-        $this->run();//todo delete that and use (Worker)->run($duration) or process management
+        $this->run(); //todo delete that and use (Worker)->run($duration) or process management
     }
 
     protected function setUpInConstructor(): void
@@ -110,7 +110,7 @@ 
      */
     final public function run(): void
     {
-        $this->log->notice('*** New worker ' . $this->defaultTaskname, ['stats' => 'bottaskworker.instance']);
+        $this->log->notice('*** New worker '.$this->defaultTaskname, ['stats' => 'bottaskworker.instance']);
         foreach ($this->getTitles() as $title) {
             try {
                 $this->titleProcess($title);
@@ -219,7 +219,7 @@ 
         }
 
         $this->log->notice($result ? '>> OK' : '>>  NOCHANGE');
-        $this->log->debug("Sleep " . static::SLEEP_AFTER_EDITION);
+        $this->log->debug("Sleep ".static::SLEEP_AFTER_EDITION);
         sleep(static::SLEEP_AFTER_EDITION);
     }
 

src/Application/CLI/externRefProcess.php

Spacing +5 added lines, -5 removed lines

@@ -54,23 +54,23 @@
     $list = new PageList([trim($argv[1])]);
 
     // delete Title from edited.txt
-    $file = __DIR__ . '/../resources/article_externRef_edited.txt';
+    $file = __DIR__.'/../resources/article_externRef_edited.txt';
     $text = file_get_contents($file);
-    $newText = str_replace(trim($argv[1]) . "\n", '', $text);
+    $newText = str_replace(trim($argv[1])."\n", '', $text);
     if (!empty($text) && $text !== $newText) {
         @file_put_contents($file, $newText);
     }
-    $botConfig->taskName = '

src/Application/ExternLink/ExternRefWorker.php

Spacing +8 added lines, -8 removed lines

@@ -28,7 +28,7 @@ 
     public const SLEEP_AFTER_EDITION = 5; // sec
     public const MINUTES_DELAY_AFTER_LAST_HUMAN_EDIT = 10; // minutes
     public const CHECK_EDIT_CONFLICT = true;
-    public const ARTICLE_ANALYZED_FILENAME = __DIR__ . '/../resources/article_externRef_edited.txt';
+    public const ARTICLE_ANALYZED_FILENAME = __DIR__.'/../resources/article_externRef_edited.txt';
     public const SKIP_ADQ = false;
     public const SKIP_LASTEDIT_BY_BOT = false;
     public const CITATION_NUMBER_ON_FIRE = 15;
@@ -77,7 +77,7 @@ 
             $result = $this->transformer->process($refContent, $this->summary);
         } catch (Throwable $e) {
             $this->log->critical(
-                'Error patate34 ' . $e->getMessage() . " " . $e->getFile() . ":" . $e->getLine(),
+                'Error patate34 '.$e->getMessage()." ".$e->getFile().":".$e->getLine(),
                 ['stats' => 'externref.exception.patate34']
             );
             // TODO : parse $e->message -> variable process, taskName, botflag...
@@ -144,11 +144,11 @@ 
         $suffix = '';
         if (isset($this->summary->memo['count article'])) {
             $this->log->stats->increment('externref.count.article');
-            $suffix .= ' ' . $this->summary->memo['count article'] . 'x {article}';
+            $suffix .= ' '.$this->summary->memo['count article'].'x {article}';
         }
         if (isset($this->summary->memo['count lien web'])) {
             $this->log->stats->increment('externref.count.lienweb');
-            $suffix .= ' ' . $this->summary->memo['count lien web'] . 'x {lien web}';
+            $suffix .= ' '.$this->summary->memo['count lien web'].'x {lien web}';
         }
         if (isset($this->summary->memo['presse'])) {
             $this->log->stats->increment('externref.count.presse');
@@ -162,20 +162,20 @@ 
             $this->log->stats->increment('externref.count.lienbrisé');
             $suffix .= ' ⚠️️️lien brisé'; //⚠️