Dispositif / Wikibot

src/Application/AbstractRefBotWorker.php

Spacing +5 added lines, -5 removed lines

@@ -24,7 +24,7 @@ 
 
     public function hasWarning(): bool
     {
-        return (bool)$this->warning;
+        return (bool) $this->warning;
     }
 
     /**
@@ -55,7 +55,7 @@ 
         $refs = array_slice($refs, 0, self::MAX_REFS_PROCESSED_IN_ARTICLE, true);
 
         foreach ($refs as $ref) {
-            $refContent = WikiTextUtil::stripFinalPoint(trim((string)$ref[1]));
+            $refContent = WikiTextUtil::stripFinalPoint(trim((string) $ref[1]));
 
             $newRefContent = $this->processRefContent($refContent);
 
@@ -70,11 +70,11 @@ 
     protected function replaceRefInText(array $ref, string $replace, string $text)
     {
         // Pas de changement
-        if (WikiTextUtil::stripFinalPoint(trim($replace)) === WikiTextUtil::stripFinalPoint(trim((string)$ref[1]))) {
+        if (WikiTextUtil::stripFinalPoint(trim($replace)) === WikiTextUtil::stripFinalPoint(trim((string) $ref[1]))) {
             return $text;
         }
         $replace = $this->addFinalPeriod($ref[0], $replace);
-        $result = str_replace($ref[1], $replace, (string)$ref[0]);
+        $result = str_replace($ref[1], $replace, (string) $ref[0]);
         $this->printDiff($ref[0], $result);
 
         return str_replace($ref[0], $result, $text);
@@ -85,7 +85,7 @@ 
      */
     protected function addFinalPeriod($ref, string $replace): string
     {
-        if (preg_match('#</ref>#', (string)$ref)) {
+        if (preg_match('#</ref>#', (string) $ref)) {
             $replace .= '.';
         }
         return $replace;

src/Application/CLI/lastExternRefProcess.php

Spacing +2 added lines, -2 removed lines

@@ -56,10 +56,10 @@
 $titles = $list->getPageTitles();
 unset($list);
 //echo count($titles)." titles\n";
-$edited = file(__DIR__ . '/../resources/article_externRef_edited.txt', FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);
+$edited = file(__DIR__.'/../resources/article_externRef_edited.txt', FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);
 $filtered = array_diff($titles, $edited);
 $list = new PageList($filtered);
-echo ">" . $list->count() . " dans liste\n";
+echo ">".$list->count()." dans liste\n";
 
 
 $httpClient = ServiceFactory::getHttpClient();

src/Application/CLI/testExternLink.php

Spacing +3 added lines, -3 removed lines

@@ -37,7 +37,7 @@ 
 
 // todo command --tor --wikiwix --internetarchive
 $torEnabled = false;
-echo "TOR enabled : ".($torEnabled ? "oui" : "non"). "\n";
+echo "TOR enabled : ".($torEnabled ? "oui" : "non")."\n";
 
 $client = ServiceFactory::getHttpClient();
 $wikiwix = new WikiwixAdapter($client, $logger);
@@ -56,10 +56,10 @@ 
     // Attention : pas de post-processing (sanitize title, etc.)
     $result = $trans->process($url, $summary);
 } catch (Exception $e) {
-    $result = "EXCEPTION ". $e->getMessage().$e->getFile().$e->getLine();
+    $result = "EXCEPTION ".$e->getMessage().$e->getFile().$e->getLine();
 }
 
-echo '>>> '. $result."\n";
+echo '>>> '.$result."\n";
 
 
 

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/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/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/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,7 +103,7 @@ 
      */
     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(),
@@ -217,7 +217,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);
     }
 
@@ -241,7 +241,7 @@ 
         $commitHashFromFile = @file_get_contents(self::GIT_COMMIT_HASH_PATH);
         if ($commitHash && $commitHashFromFile !== $commitHash) {
             file_put_contents(self::GIT_COMMIT_HASH_PATH, $commitHash);
-            $taskname = sprintf('[%s] %s', substr($commitHash, 0, 6), $taskname, );
+            $taskname = sprintf('[%s] %s', substr($commitHash, 0, 6), $taskname,);
         }
 
         return $taskname;