Dispositif / Wikibot

src/Application/WikiPageAction.php

Spacing +10 added lines, -10 removed lines

@@ -54,7 +54,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);
         }
     }
 
@@ -73,7 +73,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
         )
@@ -87,7 +87,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
@@ -102,7 +102,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(
@@ -125,7 +125,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);
                 }
@@ -140,7 +140,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
@@ -210,7 +210,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;
@@ -259,7 +259,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);
     }
@@ -315,7 +315,7 @@ 
             throw new Exception('That page does not exist');
         }
         $oldText = $this->getText();
-        $newText = $addText . $oldText;
+        $newText = $addText.$oldText;
 
         return $this->editPage($newText, $editInfo);
     }
@@ -343,7 +343,7 @@ 
         foreach ($refs as $ref) {
             if (preg_match(
                     '#(?<url>https?://(?:www\.)?lemonde\.fr/[^ \]]+)#i',
-                    (string)$ref,
+                    (string) $ref,
                     $matches
                 ) > 0
             ) {

src/Application/CLI/Monitor.php

Spacing +3 added lines, -3 removed lines

@@ -16,7 +16,7 @@ 
 use Mediawiki\Api\MediawikiFactory;
 use Normalizer;
 
-include __DIR__ . '/../myBootstrap.php';
+include __DIR__.'/../myBootstrap.php';
 
 $process = new Monitor();
 $process->run();
@@ -97,7 +97,7 @@ 
             [
                 'page' => $title ?? '',
                 'verify' => date("Y-m-d H:i:s"),
-                'altered' => (int)$stat,
+                'altered' => (int) $stat,
             ]
         );
     }
@@ -147,6 +147,6 @@ 
 
         }
 
-        return (int)round(($count - $found) / count($data) * 100);
+        return (int) round(($count - $found) / count($data) * 100);
     }
 }

src/Domain/ExternLink/Validators/RobotNoIndexValidator.php

Spacing +5 added lines, -5 removed lines

@@ -36,14 +36,14 @@ 
         if (
             !empty($robots)
             && (
-                stripos((string)$robots, 'noindex') !== false
-                || stripos((string)$robots, 'none') !== false
+                stripos((string) $robots, 'noindex') !== false
+                || stripos((string) $robots, 'none') !== false
             )
         ) {
-            $this->log->notice('robots NOINDEX : ' . $this->url);
+            $this->log->notice('robots NOINDEX : '.$this->url);
 
             if (empty($this->pageData['meta']['prettyDomainName'])) {
-                $this->log->warning('No prettyDomainName for ' . $this->url);
+                $this->log->warning('No prettyDomainName for '.$this->url);
 
                 return true;
             }
@@ -57,7 +57,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;
         }

src/Domain/Publisher/BnfMapper.php

Spacing +3 added lines, -3 removed lines

@@ -37,7 +37,7 @@ 
         $this->xml = $xml;
 
         // skip multi-records
-        $nbResults = (int)$xml->xpath('//srw:numberOfRecords[1]')[0] ?? 0;
+        $nbResults = (int) $xml->xpath('//srw:numberOfRecords[1]')[0] ?? 0;
         if (1 !== $nbResults) {
             return [];
         }
@@ -121,7 +121,7 @@ 
         $res = [];
         foreach ($elements as $element) {
             if (isset($element) && $element instanceof SimpleXMLElement) {
-                $res[] = (string)$element;
+                $res[] = (string) $element;
             }
         }
 
@@ -185,7 +185,7 @@ 
     {
         $raw = $this->xpath2string('//mxc:datafield[@tag="215"]/mxc:subfield[@code="a"]');
         if (!empty($raw) && preg_match('#(\d{2,}) p\.#', $raw, $matches) > 0) {
-            return (string)$matches[1];
+            return (string) $matches[1];
         }
 
         return null;

src/Domain/ExternLink/ExternRefTransformer.php

Spacing +15 added lines, -15 removed lines

@@ -35,13 +35,13 @@ 
     use SummaryExternTrait, 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 REPLACE_410 = 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 bool $skipSiteBlacklisted = true;
     public bool $skipRobotNoIndex = true;
@@ -93,12 +93,12 @@ 
         }
         $this->registrableDomain = $this->urlChecker->getRegistrableDomain($url); // hack
         if ($this->isSiteBlackListed()) {
-            $this->log->debug('Site blacklisted : ' . $this->registrableDomain);
+            $this->log->debug('Site blacklisted : '.$this->registrableDomain);
             return $url;
         }
 
         if (!$this->validateConfigWebDomain($this->registrableDomain)) {
-            $this->log->debug('Domain not validate by config : ' . $this->registrableDomain);
+            $this->log->debug('Domain not validate by config : '.$this->registrableDomain);
             return $url;
         }
 
@@ -150,7 +150,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;
@@ -168,7 +168,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;
         }
@@ -179,9 +179,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é");
         }
     }
 
@@ -206,7 +206,7 @@ 
         if ($pageData === []
             || (empty($pageData['JSON-LD']) && empty($pageData['meta']))
         ) {
-            $this->log->notice('No metadata : ' . $url);
+            $this->log->notice('No metadata : '.$url);
 
             return true;
         }
@@ -221,7 +221,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;
         }
@@ -274,7 +274,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;
     }
@@ -330,7 +330,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/Application/WikiBotConfig.php

Spacing +6 added lines, -7 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'];
@@ -97,9 +97,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;
     }
 
     protected static function getBotOwner()
@@ -155,7 +154,7 @@ 
 
     protected function getBotTalkPageTitle(): string
     {
-        return self::TALK_PAGE_PREFIX . $this::getBotName();
+        return self::TALK_PAGE_PREFIX.$this::getBotName();
     }
 
     /**
@@ -266,8 +265,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/CLI/lastExternRefProcess.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
 //echo count($titles)." titles\n";
 $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 );
+$list = new PageList($filtered);
 echo ">".$list->count()." dans liste\n";
 
 new ExternRefWorker($botConfig, $wiki, $list, null, new InternetDomainParser());

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';
 
     private readonly Rules $rules;
 
@@ -55,7 +55,7 @@ 
     public static function extractSubdomainString(string $httpURL): string
     {
         if (!HttpUtil::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

@@ -54,7 +54,7 @@ 
     )
     {
         if (!HttpUtil::isHttpURL($url)) {
-            throw new Exception('string is not an URL ' . $url);
+            throw new Exception('string is not an URL '.$url);
         }
         $this->url = $url;
     }
@@ -170,7 +170,7 @@ 
     {
         try {
             if (!HttpUtil::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');
@@ -181,7 +181,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);
         }