Dispositif / Wikibot

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/Application/AbstractBotTaskWorker.php

Spacing +6 added lines, -6 removed lines

@@ -32,12 +32,12 @@ 
     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;
     public const THROTTLE_DELAY_AFTER_EACH_TITLE = 1; //secs
-    protected const GIT_COMMIT_HASH_PATH = __DIR__ . '/resources/commithash.txt';
+    protected const GIT_COMMIT_HASH_PATH = __DIR__.'/resources/commithash.txt';
 
     /**
      * @var PageListInterface
@@ -93,7 +93,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
     }
 
     /**
@@ -103,13 +103,13 @@ 
      */
     final public function run(): void
     {
-        $this->log->notice('*** New BotTaskWorker: ' . $this->defaultTaskname, ['stats' => 'bottaskworker.instance']);
+        $this->log->notice('*** New BotTaskWorker: '.$this->defaultTaskname, ['stats' => 'bottaskworker.instance']);
         $this->log->notice(sprintf(
             '*** Bot: %s - commit: %s',
             $this->bot::getBotName(),
             $this->bot->getCurrentGitCommitHash() ?? '??'
         ));
-        $this->log->notice('*** Stats: ' . $this->log->stats::class);
+        $this->log->notice('*** Stats: '.$this->log->stats::class);
 
         foreach ($this->getTitles() as $title) {
             try {
@@ -218,7 +218,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

@@ -59,13 +59,13 @@
     $list = new PageList([trim($options['page'])]);
 
     // 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->setTaskName('

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/DeadLinkTransformer.php

Spacing +4 added lines, -4 removed lines

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

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;
     }