Dispositif / Wikibot

src/Application/WikiBotConfig.php

Spacing +10 added lines, -11 removed lines

@@ -31,7 +31,7 @@ 
 class WikiBotConfig
 {
     public const VERSION = '2.1';
-    public const WATCHPAGE_FILENAME = __DIR__ . '/resources/watch_pages.json';
+    public const WATCHPAGE_FILENAME = __DIR__.'/resources/watch_pages.json';
     public const EXIT_ON_CHECK_WATCHPAGE = false;
     // do not stop if they play with {stop} on bot talk page
     public const BLACKLIST_EDITOR = ['OrlodrimBot'];
@@ -56,7 +56,7 @@ 
      * @var DateTimeImmutable
      */
     protected $lastCheckStopDate;
-    protected SMSInterface|null $SMSClient;
+    protected SMSInterface | null $SMSClient;
     protected $mediawikiFactory;
     protected ?string $gitCommitHash = null;
 
@@ -90,9 +90,8 @@ 
     {
         $text = WikiTextUtil::removeHTMLcomments($text);
         $botName = $botName ?: self::getBotName();
-        $denyReg = (empty($botName)) ? '' :
-            '|\{\{bots ?\| ?(optout|deny)\=[^\}]*' . preg_quote($botName, '#') . '[^\}]*\}\}';
-        return preg_match('#({{nobots}}|{{bots ?\| ?(optout|deny) ?= ?all ?}}' . $denyReg . ')#i', $text) > 0;
+        $denyReg = (empty($botName)) ? '' : '|\{\{bots ?\| ?(optout|deny)\=[^\}]*'.preg_quote($botName, '#').'[^\}]*\}\}';
+        return preg_match('#({{nobots}}|{{bots ?\| ?(optout|deny) ?= ?all ?}}'.$denyReg.')#i', $text) > 0;
     }
 
     /**
@@ -127,12 +126,12 @@ 
         if ($this->gitCommitHash) {
             return $this->gitCommitHash;
         }
-        $path = __DIR__ . '/../../.git/';
+        $path = __DIR__.'/../../.git/';
         if (!file_exists($path)) {
             return null;
         }
-        $head = trim(substr(file_get_contents($path . 'HEAD'), 4));
-        $hash = trim(file_get_contents(sprintf($path . $head)));
+        $head = trim(substr(file_get_contents($path.'HEAD'), 4));
+        $hash = trim(file_get_contents(sprintf($path.$head)));
         $this->gitCommitHash = $hash; // cached
 
         return $hash;
@@ -190,7 +189,7 @@ 
 
     protected function getBotTalkPageTitle(): string
     {
-        return self::TALK_PAGE_PREFIX . $this::getBotName();
+        return self::TALK_PAGE_PREFIX.$this::getBotName();
     }
 
     protected function sendSMSandFunnyTalk(string $lastEditor, ?bool $botTalk): void
@@ -290,8 +289,8 @@ 
      */
     public function minutesSinceLastEdit(string $title): int
     {
-        $time = $this->getTimestamp($title);  // 2011-09-02T16:31:13Z
+        $time = $this->getTimestamp($title); // 2011-09-02T16:31:13Z
 
-        return (int)round((time() - strtotime($time)) / 60);
+        return (int) round((time() - strtotime($time)) / 60);
     }
 }

src/Infrastructure/Monitor/StatsSqlite3.php

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
  */
 class StatsSqlite3 implements StatsInterface
 {
-    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;
@@ -75,7 +75,7 @@ 
         // `num` default value is 1 so insert is enough to set num=1
         try {
             return $this->sqliteExecWriteOrWait(
-                '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;
@@ -107,7 +107,7 @@ 
         $tag = $this->formatTag($tag);
         try {
             return $this->sqliteExecWriteOrWait(
-                'INSERT OR REPLACE INTO tagnum (tag,num) VALUES("' . $tag . '", ' . $num . ')'
+                'INSERT OR REPLACE INTO tagnum (tag,num) VALUES("'.$tag.'", '.$num.')'
             );
         } catch (Exception $e) {
             return false;
@@ -119,7 +119,7 @@ 
         $tag = $this->formatTag($tag);
         try {
             return $this->sqliteExecWriteOrWait(
-                '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;

src/Infrastructure/Monitor/ConsoleLogger.php

Spacing +10 added lines, -10 removed lines

@@ -20,7 +20,7 @@ 
 {
     //    use LoggerTrait;
 
-    protected const CRITICAL_LOG_PATH = __DIR__ . '/../../../log/critical.log';
+    protected const CRITICAL_LOG_PATH = __DIR__.'/../../../log/critical.log';
     public bool $verbose = false;
     public bool $debug = false;
     public bool $colorMode = false;
@@ -36,7 +36,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.'()');
     }
 
     /**
@@ -62,7 +62,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);
                 }
@@ -70,20 +70,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);
                     }
@@ -91,14 +91,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;
         }
     }
@@ -121,7 +121,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;
         }
@@ -132,7 +132,7 @@ 
     {
         file_put_contents(
             self::CRITICAL_LOG_PATH,
-            date('d-m-Y H:i:s') . " : $level : " . $message . PHP_EOL,
+            date('d-m-Y H:i:s')." : $level : ".$message.PHP_EOL,
             FILE_APPEND
         );
     }

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 : registrableDomain not a valid URL : ' . $this->url,
+            $this->log->warning('Skip : registrableDomain 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->registrableDomain && !$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;
@@ -159,7 +159,7 @@ 
     protected function isSiteBlackListed(): bool
     {
         if ($this->registrableDomain && $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;
@@ -177,7 +177,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;
         }
@@ -188,9 +188,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é");
         }
     }
 
@@ -216,7 +216,7 @@ 
         if ($pageData === []
             || (empty($pageData['JSON-LD']) && empty($pageData['meta']))
         ) {
-            $this->log->notice('No metadata : ' . $url);
+            $this->log->notice('No metadata : '.$url);
 
             return true;
         }
@@ -231,7 +231,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;
         }
@@ -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;
     }
 
@@ -354,7 +354,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;
     }

src/Infrastructure/WikiBlameAdapter.php

Spacing +5 added lines, -5 removed lines

@@ -23,7 +23,7 @@ 
 {
     protected HttpClientInterface $client;
 
-    public function __construct(private readonly LoggerInterface|NullLogger $log = new NullLogger())
+    public function __construct(private readonly LoggerInterface | NullLogger $log = new NullLogger())
     {
         $this->client = ServiceFactory::getHttpClient();
     }
@@ -38,16 +38,16 @@ 
     {
         $url = 'http://wikipedia.ramselehof.de/wikiblame.php?project=wikipedia&article='
             . str_replace(' ', '+', $article)
-            . '&needle=' . urlencode($string)
+            . '&needle='.urlencode($string)
             . '&lang=fr&limit=2000&offjahr=2024&offmon=1&offtag=1&offhour=23&offmin=55'
             . '&searchmethod=int&order=desc'
-            . '&force_wikitags=' . ($hasWikicode ? 'on' : 'off')
+            . '&force_wikitags='.($hasWikicode ? 'on' : 'off')
             . '&user_lang=fr&ignorefirst=0';
 
         $response = $this->client->get($url);
 
         if ($response->getStatusCode() !== 200) {
-            $this->log->warning('WikiBlame: ' . $response->getStatusCode() . ' ' . $response->getReasonPhrase());
+            $this->log->warning('WikiBlame: '.$response->getStatusCode().' '.$response->getReasonPhrase());
             return null;
         }
 
@@ -68,7 +68,7 @@ 
         }
 
         return [
-            'versionid' => (int)$matches[1],
+            'versionid' => (int) $matches[1],
             'dateraw' => $matches[2],
             'datetime' => DateUtil::simpleFrench2object($matches[2]),
         ];

src/Domain/ExternLink/ExternPage.php

Spacing +4 added lines, -4 removed lines

@@ -54,7 +54,7 @@ 
     )
     {
         if (!HttpUtil::isHttpURL($url)) {
-            throw new Exception('string is not an URL ' . $url);
+            throw new Exception('string is not an URL '.$url);
         }
         $this->url = $url;
     }
@@ -94,7 +94,7 @@ 
                 '//script[@type="application/ld+json"]'
             );
         } catch (Exception $e) {
-            $this->log->warning('TagParser->xpathResults NULL ' . $this->url);
+            $this->log->warning('TagParser->xpathResults NULL '.$this->url);
 
             return [];
         }
@@ -176,7 +176,7 @@ 
     {
         try {
             if (!HttpUtil::isHttpURL($this->url)) {
-                throw new Exception('string is not an URL ' . $this->url);
+                throw new Exception('string is not an URL '.$this->url);
             }
             if (!$this->domainParser instanceof InternetDomainParserInterface) {
                 $this->log->notice('InternetDomainParser is not set');
@@ -187,7 +187,7 @@ 
             return $this->domainParser->getRegistrableDomainFromURL($this->url);
         } catch (Exception $e) {
             if ($this->log !== null) {
-                $this->log->warning('InternetDomainParser->getRegistrableDomainFromURL NULL ' . $this->url);
+                $this->log->warning('InternetDomainParser->getRegistrableDomainFromURL NULL '.$this->url);
             }
             throw new Exception('InternetDomainParser->getRegistrableDomainFromURL NULL', $e->getCode(), $e);
         }

src/Domain/ExternLink/DeadLinkTransformer.php

Spacing +4 added lines, -4 removed lines

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

src/Domain/ExternLink/SummaryExternTrait.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
             $this->summary->memo['sites'][] = $this->externalPage->getPrettyDomainName(); // ???
         }
         if (isset($mapData['accès url'])) {
-            $this->log->debug('accès