Dispositif / Wikibot

src/Application/Traits/WorkerCLITrait.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
         if ($this->modeAuto) {
             return true;
         }
-        $ask = readline(Color::LIGHT_MAGENTA . '*** '.$question.' [y/n/auto]' . Color::NORMAL);
+        $ask = readline(Color::LIGHT_MAGENTA.'*** '.$question.' [y/n/auto]'.Color::NORMAL);
         if ('auto' === $ask) {
             $this->modeAuto = true;
 

src/Application/Traits/WorkerAnalyzedTitlesTrait.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
                 FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES
             );
         } catch (Throwable $e) {
-            $this->log->critical("Can't parse ARTICLE_ANALYZED_FILENAME : " . $e->getMessage());
+            $this->log->critical("Can't parse ARTICLE_ANALYZED_FILENAME : ".$e->getMessage());
             $analyzed = [];
         }
         $this->pastAnalyzed = ($analyzed !== false) ? $analyzed : [];
@@ -34,7 +34,7 @@ 
     {
         if (!$this->checkAlreadyAnalyzed($title)) {
             $this->pastAnalyzed[] = $title; // skip doublon title
-            @file_put_contents(static::ARTICLE_ANALYZED_FILENAME, $title . PHP_EOL, FILE_APPEND);
+            @file_put_contents(static::ARTICLE_ANALYZED_FILENAME, $title.PHP_EOL, FILE_APPEND);
         }
     }
 

src/Application/Traits/BotWorkerTrait.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
     protected function printTitle(string $title): void
     {
         echo "---------------------\n";
-        echo date('d-m-Y H:i:s') . ' ' . Color::BG_CYAN . "  $title " . Color::NORMAL . "\n";
+        echo date('d-m-Y H:i:s').' '.Color::BG_CYAN."  $title ".Color::NORMAL."\n";
     }
 
     protected function canProcessTitleArticle(string $title, ?string $text): bool

src/Application/AbstractBotTaskWorker.php

Spacing +5 added lines, -5 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;
@@ -86,7 +86,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
@@ -101,7 +101,7 @@ 
      */
     final public function run(): void
     {
-        echo date('d-m-Y H:i:s') . " *** NEW WORKER ***\n";
+        echo date('d-m-Y H:i:s')." *** NEW WORKER ***\n";
         foreach ($this->getTitles() as $title) {
             try {
                 $this->titleProcess($title);
@@ -202,14 +202,14 @@ 
 
             // If not a critical edition error
             // example : Wiki Conflict : Page has been edited after getText()
-            echo "Error : " . $e->getMessage() . "\n";
+            echo "Error : ".$e->getMessage()."\n";
             $this->log->warning($e->getMessage());
 
             return;
         }
 
         dump($result);
-        echo "Sleep " . static::SLEEP_AFTER_EDITION . "\n";
+        echo "Sleep ".static::SLEEP_AFTER_EDITION."\n";
         sleep(static::SLEEP_AFTER_EDITION);
     }
 

src/Application/TalkBotConfig.php

Spacing +10 added lines, -10 removed lines

@@ -23,8 +23,8 @@ 
 class TalkBotConfig extends WikiBotConfig
 {
     public const BOT_TALK_SUMMARY = 'Réponse artificielle';
-    public const BOT_TALK_FILE = __DIR__ . '/resources/phrases_zizibot.txt';
-    public const TALKCONFIG_FILENAME = __DIR__ . '/resources/botTalk_config.json';
+    public const BOT_TALK_FILE = __DIR__.'/resources/phrases_zizibot.txt';
+    public const TALKCONFIG_FILENAME = __DIR__.'/resources/botTalk_config.json';
 
     /**
      * Add a freaky response in the bottom of the talk page.
@@ -37,7 +37,7 @@ 
         // ugly dependency
         $wiki = ServiceFactory::getMediawikiFactory();
         if (!$pageTitle) {
-            $pageTitle = 'Discussion utilisateur:' . $this::getBotName();
+            $pageTitle = 'Discussion utilisateur:'.$this::getBotName();
         }
         $page = new WikiPageAction($wiki, $pageTitle);
         $last = $page->page->getRevisions()->getLatest();
@@ -57,13 +57,13 @@ 
 
         // No response if time < 24h since last bot owner response
         if ($last->getUser() == self::getBotOwner()) {
-            $talkConfig['owner_last_time'] = (int)strtotime($last->getTimestamp());
+            $talkConfig['owner_last_time'] = (int) strtotime($last->getTimestamp());
             file_put_contents(self::TALKCONFIG_FILENAME, json_encode($talkConfig, JSON_THROW_ON_ERROR));
 
             return false;
         }
         // No response if time < 24h since last owner response
-        if (isset($talkConfig['owner_last_time']) && (int)$talkConfig['owner_last_time'] > (time() - 60 * 60 * 48)) {
+        if (isset($talkConfig['owner_last_time']) && (int) $talkConfig['owner_last_time'] > (time() - 60 * 60 * 48)) {
             echo "No response if time < 24h after last owner response\n";
 
             return false;
@@ -79,7 +79,7 @@ 
         $editInfo = new EditInfo(static::BOT_TALK_SUMMARY);
         $success = $page->addToBottomOfThePage($addText, $editInfo);
 
-        return (bool)$success;
+        return (bool) $success;
     }
 
     /**
@@ -110,13 +110,13 @@ 
         $lines = explode("\n", trim($text));
         $lastLine = $lines[count($lines) - 1];
         if (preg_match('#^(:*).+#', $lastLine, $matches) && !empty($matches[1])) {
-            $nextIdent = $matches[1] . ':';
+            $nextIdent = $matches[1].':';
             if (empty($author)) {
                 return $nextIdent;
             }
             // search author signature link to check that he wrote on the page bottom
             if (preg_match(
-                '#\[\[(?:User|Utilisateur|Utilisatrice):' . preg_quote($author, '#') . '[|\]]#i',
+                '#\[\[(?:User|Utilisateur|Utilisatrice):'.preg_quote($author, '#').'[|\]]#i',
                 $matches[0]
             )
             ) {
@@ -137,7 +137,7 @@ 
             throw new ConfigException('Pas de phrases disponibles pour TalkBot');
         }
 
-        return (string)trim($sentences[array_rand($sentences)]);
+        return (string) trim($sentences[array_rand($sentences)]);
     }
 
     /**
@@ -147,7 +147,7 @@ 
     public function botContribs(): string
     {
         $url
-            = 'https://fr.wikipedia.org/w/api.php?action=query&list=usercontribs&ucuser=' . $this::getBotName()
+            = 'https://fr.wikipedia.org/w/api.php?action=query&list=usercontribs&ucuser='.$this::getBotName()
             . '&ucnamespace=0&uclimit=40&ucprop=title|timestamp|comment&format=json';
 
         return file_get_contents($url);

src/Application/WikiPageAction.php

Spacing +9 added lines, -9 removed lines

@@ -63,7 +63,7 @@ 
             $this->page = $wiki->newPageGetter()->getFromTitle($title);
             $this->ns = $this->page->getPageIdentifier()->getTitle()->getNs();
         } catch (Throwable $e) {
-            throw new Exception('Erreur construct WikiPageAction ' . $e->getMessage() . $e->getFile() . $e->getLine(), $e->getCode(), $e);
+            throw new Exception('Erreur construct WikiPageAction '.$e->getMessage().$e->getFile().$e->getLine(), $e->getCode(), $e);
         }
     }
 
@@ -131,7 +131,7 @@ 
     public function getRedirect(): ?string
     {
         if ($this->getText() && preg_match('/^#REDIRECT(?:ION)? ?\[\[([^]]+)]]/i', $this->getText(), $matches)) {
-            return (string)trim($matches[1]);
+            return (string) trim($matches[1]);
         }
 
         return null;
@@ -222,7 +222,7 @@ 
             throw new Exception('That page does not exist');
         }
         $oldText = $this->getText();
-        $newText = $oldText . "\n" . $addText;
+        $newText = $oldText."\n".$addText;
 
         return $this->editPage($newText, $editInfo);
     }
@@ -239,7 +239,7 @@ 
             throw new Exception('That page does not exist');
         }
         $oldText = $this->getText();
-        $newText = $addText . $oldText;
+        $newText = $addText.$oldText;
 
         return $this->editPage($newText, $editInfo);
     }
@@ -254,7 +254,7 @@ 
         // "{{en}} {{zh}} {{ouvrage...}}"
         // todo test U
         if (preg_match_all(
-            '#(?<langTemp>{{[a-z][a-z]}} ?{{[a-z][a-z]}}) ?' . preg_quote($tplOrigin, '#') . '#i',
+            '#(?<langTemp>{{[a-z][a-z]}} ?{{[a-z][a-z]}}) ?'.preg_quote($tplOrigin, '#').'#i',
             $text,
             $matches
         )
@@ -268,7 +268,7 @@ 
         // hack // todo: autres patterns {{en}} ?
         // OK : {{en}} \n {{ouvrage}}
         if (preg_match_all(
-                "#(?<langTemp>{{(?<lang>[a-z][a-z])}} *\n?)?" . preg_quote($tplOrigin, '#') . '#i',
+                "#(?<langTemp>{{(?<lang>[a-z][a-z])}} *\n?)?".preg_quote($tplOrigin, '#').'#i',
                 $text,
                 $matches
             ) > 0
@@ -283,7 +283,7 @@ 
                 // example : {{en}} {{template|lang=ru}}
                 if (!empty($lang) && self::SKIP_LANG_INDICATOR !== $lang
                     && preg_match('#langue *=#', $tplReplace)
-                    && !preg_match('#langue *= ?' . $lang . '#i', $tplReplace)
+                    && !preg_match('#langue *= ?'.$lang.'#i', $tplReplace)
                     && !preg_match('#\| ?langue *= ?\n?\|#', $tplReplace)
                 ) {
                     echo sprintf(
@@ -306,7 +306,7 @@ 
                 // FIX dirty : {{en}} mais langue= avec value non définie sur new template...
                 if (!empty($lang) && preg_match('#\| ?(langue *=) ?\n? ?\|#', $tplReplace, $matchLangue) > 0) {
                     $previousTpl = $tplReplace;
-                    $tplReplace = str_replace($matchLangue[1], 'langue=' . $lang, $tplReplace);
+                    $tplReplace = str_replace($matchLangue[1], 'langue='.$lang, $tplReplace);
                     //dump('origin', $tplOrigin);
                     $text = str_replace($previousTpl, $tplReplace, $text);
                 }
@@ -321,7 +321,7 @@ 
                 // replace {template} and {{lang}} {template}
                 $text = str_replace($mention, $tplReplace, $text);
                 $text = str_replace(
-                    $matches['langTemp'][$num] . $tplReplace,
+                    $matches['langTemp'][$num].$tplReplace,
                     $tplReplace,
                     $text
                 ); // si 1er replace global sans

src/Infrastructure/InternetDomainParser.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
  */
 class InternetDomainParser implements InternetDomainParserInterface
 {
-    private const PATH_CACHE_PUBLIC_SUFFIX_LIST = __DIR__ . '/resources/public_suffix_list.dat';
+    private const PATH_CACHE_PUBLIC_SUFFIX_LIST = __DIR__.'/resources/public_suffix_list.dat';
 
     /** @var Rules */
     private $rules;
@@ -55,7 +55,7 @@ 
     public static function extractSubdomainString(string $httpURL): string
     {
         if (!ExternHttpClient::isHttpURL($httpURL)) {
-            throw new Exception('string is not an URL ' . $httpURL);
+            throw new Exception('string is not an URL '.$httpURL);
         }
 
         return parse_url($httpURL, PHP_URL_HOST);

src/Application/Http/ExternHttpClient.php

Spacing +6 added lines, -6 removed lines

@@ -60,13 +60,13 @@ 
         // idn_to_ascii('teßt.com',IDNA_NONTRANSITIONAL_TO_ASCII,INTL_IDNA_VARIANT_UTS46)
         // checkdnsrr($string, "A") // check DNS record
         if (!self::isHttpURL($url)) {
-            throw new DomainException('URL not compatible : ' . $url);
+            throw new DomainException('URL not compatible : '.$url);
         }
         $response = $this->client->get($url);
 
         if (200 !== $response->getStatusCode()) {
-            echo 'HTTP error ' . $response->getStatusCode();
-            $this->log->error('HTTP error ' . $response->getStatusCode() . ' ' . $response->getReasonPhrase());
+            echo 'HTTP error '.$response->getStatusCode();
+            $this->log->error('HTTP error '.$response->getStatusCode().' '.$response->getReasonPhrase());
 
             return null;
         }
@@ -82,7 +82,7 @@ 
      */
     public static function isHttpURL(string $url): bool
     {
-        return (bool)preg_match('#^https?://[^ ]+$#i', $url);
+        return (bool) preg_match('#^https?://[^ ]+$#i', $url);
     }
 
     /**
@@ -104,7 +104,7 @@ 
         $charset = $this->extractCharset($html) ?? 'WINDOWS-1252';
 
         if (empty($charset)) {
-            throw new DomainException('normalized html error and no charset found : ' . $url);
+            throw new DomainException('normalized html error and no charset found : '.$url);
         }
         try {
             $html2 = iconv($charset, 'UTF-8//TRANSLIT', $html);
@@ -113,7 +113,7 @@ 
                 return '';
             }
         } catch (Throwable $e) {
-            throw new DomainException("error converting : $charset to UTF-8 on " . $url, $e->getCode(), $e);
+            throw new DomainException("error converting : $charset to UTF-8 on ".$url, $e->getCode(), $e);
         }
 
         return $html2;

src/Domain/ExternLink/ExternPage.php

Spacing +3 added lines, -3 removed lines

@@ -69,7 +69,7 @@ 
     )
     {
         if (!ExternHttpClient::isHttpURL($url)) {
-            throw new Exception('string is not an URL ' . $url);
+            throw new Exception('string is not an URL '.$url);
         }
         $this->url = $url;
         $this->html = $html;
@@ -189,7 +189,7 @@ 
     {
         try {
             if (!ExternHttpClient::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');
@@ -200,7 +200,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);
         }