Dispositif / Wikibot

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

src/Domain/ExternLink/ExternHttpErrorLogic.php

Spacing +6 added lines, -6 removed lines

@@ -14,7 +14,7 @@ 
 
 class ExternHttpErrorLogic
 {
-    public const LOG_REQUEST_ERROR = __DIR__ . '/../../Application/resources/external_request_error.log';
+    public const LOG_REQUEST_ERROR = __DIR__.'/../../Application/resources/external_request_error.log';
 
     /**
      * @var LoggerInterface
@@ -46,12 +46,12 @@ 
             }
             return $url;
         } elseif (preg_match('#401 Unauthorized#i', $errorMessage)) {
-            $this->log->notice('401 Unauthorized : skip ' . $url);
+            $this->log->notice('401 Unauthorized : skip '.$url);
 
             return $url;
         } else {
             //  autre : ne pas générer de {lien brisé}, car peut-être 404 temporaire
-            $this->log->warning('erreur sur extractWebData ' . $errorMessage);
+            $this->log->warning('erreur sur extractWebData '.$errorMessage);
 
             //file_put_contents(self::LOG_REQUEST_ERROR, $this->domain."\n", FILE_APPEND);
 
@@ -78,7 +78,7 @@ 
     {
         $text = str_replace(['https://', 'http://', 'www.'], '', $url);
         if (strlen($text) > 30) {
-            $text = substr($text, 0, 30) . '…';
+            $text = substr($text, 0, 30).'…';
         }
 
         return $text;
@@ -86,7 +86,7 @@ 
 
     protected function log403(string $url): void
     {
-        $this->log->warning('403 Forbidden : ' . $url);
-        file_put_contents(self::LOG_REQUEST_ERROR, '403 Forbidden : ' . $url . "\n", FILE_APPEND);
+        $this->log->warning('403 Forbidden : '.$url);
+        file_put_contents(self::LOG_REQUEST_ERROR, '403 Forbidden : '.$url."\n", FILE_APPEND);
     }
 }
\ No newline at end of file

src/Domain/ExternLink/ExternRefTransformer.php

Spacing +13 added lines, -13 removed lines

@@ -31,12 +31,12 @@ 
     use SummaryExternTrait, RobotsRulesTrait, PublisherLogicTrait;
 
     public const HTTP_REQUEST_LOOP_DELAY = 10;
-    public const SKIP_DOMAIN_FILENAME = __DIR__ . '/../resources/config_skip_domain.txt';
+    public const SKIP_DOMAIN_FILENAME = __DIR__.'/../resources/config_skip_domain.txt';
     public const REPLACE_404 = true;
-    public const CONFIG_PRESSE = __DIR__ . '/../resources/config_presse.yaml';
-    public const CONFIG_NEWSPAPER_JSON = __DIR__ . '/../resources/data_newspapers.json';
-    public const CONFIG_SCIENTIFIC_JSON = __DIR__ . '/../resources/data_scientific_domain.json';
-    public const CONFIG_SCIENTIFIC_WIKI_JSON = __DIR__ . '/../resources/data_scientific_wiki.json';
+    public const CONFIG_PRESSE = __DIR__.'/../resources/config_presse.yaml';
+    public const CONFIG_NEWSPAPER_JSON = __DIR__.'/../resources/data_newspapers.json';
+    public const CONFIG_SCIENTIFIC_JSON = __DIR__.'/../resources/data_scientific_domain.json';
+    public const CONFIG_SCIENTIFIC_WIKI_JSON = __DIR__.'/../resources/data_scientific_wiki.json';
 
     public $skipSiteBlacklisted = true;
     public $skipRobotNoIndex = true;
@@ -160,7 +160,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;
@@ -178,7 +178,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;
         }
@@ -192,9 +192,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é");
         }
     }
 
@@ -219,7 +219,7 @@ 
         if ($pageData === []
             || (empty($pageData['JSON-LD']) && empty($pageData['meta']))
         ) {
-            $this->log->notice('No metadata : ' . $url);
+            $this->log->notice('No metadata : '.$url);
 
             return true;
         }
@@ -234,7 +234,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;
         }
@@ -291,7 +291,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;
     }
@@ -349,7 +349,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/RobotsRulesTrait.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
                 || stripos($robots, 'none') !== false
             )
         ) {
-            $this->log->notice('robots NOINDEX : ' . $url);
+            $this->log->notice('robots NOINDEX : '.$url);
 
             return !$this->isNoIndexDomainWhitelisted($pageData['meta']['prettyDomainName']);
         }
@@ -38,7 +38,7 @@ 
     protected function isNoIndexDomainWhitelisted(?string $prettyDomain): bool
     {
         if (in_array($prettyDomain ?? '', $this->noindexWhitelist)) {
-            $this->log->notice('ROBOT_NOINDEX_WHITELIST ' . $prettyDomain);
+            $this->log->notice('ROBOT_NOINDEX_WHITELIST '.$prettyDomain);
 
             return true;
         }