Dispositif / Wikibot

src/Domain/Publisher/SeoSanitizer.php

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
         }
 
         // rebuild title but keep only the first 2 segments
-        return trim($titleSegments[0]) . self::REBUILD_SEPARATOR . trim($titleSegments[1]);
+        return trim($titleSegments[0]).self::REBUILD_SEPARATOR.trim($titleSegments[1]);
     }
 
     /**
@@ -102,7 +102,7 @@ 
 
         return array_values(array_filter(
             $seoSegments,
-            function ($segment) use ($prettyDomainName, $siteName) {
+            function($segment) use ($prettyDomainName, $siteName) {
                 $strippedSegment = mb_strtolower(TextUtil::stripPunctuation(TextUtil::stripAccents($segment)));
                 $strippedSegment = str_replace(['.', '-', ' '], '', $strippedSegment);
 

src/Application/WikiBotConfig.php

Spacing +7 added lines, -8 removed lines

@@ -33,7 +33,7 @@ 
 {
     public const VERSION = '1.0';
 
-    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;
 
@@ -156,9 +156,9 @@ 
      */
     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);
     }
 
     /**
@@ -169,9 +169,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;
     }
 
     /**
@@ -227,14 +226,14 @@ 
 
     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
     {
         if (class_exists(SMS::class)) {
             try {
-                new SMS($this::getBotName() . ' {stop} by ' . $lastEditor);
+                new SMS($this::getBotName().' {stop} by '.$lastEditor);
             } catch (Exception $smsException) {
                 unset($smsException);
             }

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