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