Dispositif / Wikibot

src/Application/Examples/testExternLink.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     // 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";

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/ExternPageFactory.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
     /**
      * @throws Exception
      */
-    public static function fromURL($url, ExternHttpClientInterface $httpClient,LoggerInterface $logger = null): ExternPage
+    public static function fromURL($url, ExternHttpClientInterface $httpClient, LoggerInterface $logger = null): ExternPage
     {
         if (!ExternHttpClient::isHttpURL($url)) {
             throw new Exception('string is not an URL '.$url);

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

src/Domain/ExternLink/CheckURL.php

Spacing +4 added lines, -4 removed lines

@@ -44,7 +44,7 @@ 
         $this->url = $url;
         $this->registrableDomain = null;
         if (!ExternHttpClient::isHttpURL($url)) {
-            $this->log->debug('Skip : not a valid URL : ' . $url);
+            $this->log->debug('Skip : not a valid URL : '.$url);
             return false;
         }
 
@@ -52,7 +52,7 @@ 
             return false;
         }
         if (!ExternHttpClient::isHttpURL($url)) {
-            throw new Exception('string is not an URL ' . $url);
+            throw new Exception('string is not an URL '.$url);
         }
 
         $this->findRegistrableDomain();
@@ -74,7 +74,7 @@ 
         try {
             $this->registrableDomain = (new InternetDomainParser())->getRegistrableDomainFromURL($this->url);
         } catch (Exception $e) {
-            $this->log->warning('Skip : not a valid URL : ' . $this->url);
+            $this->log->warning('Skip : not a valid URL : '.$this->url);
             return null;
         }
         return $this->registrableDomain;
@@ -91,7 +91,7 @@ 
      */
     protected function hasForbiddenFilenameExtension(): bool
     {
-        return (bool)preg_match(
+        return (bool) preg_match(
             '#\.(pdf|jpg|jpeg|gif|png|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
         );

src/Domain/Interfaces/ExternHttpClientInterface.php

Spacing +1 added lines, -1 removed lines

@@ -11,5 +11,5 @@
 
 interface ExternHttpClientInterface
 {
-    public function getHTML(string $url, ?bool $normalized=false): ?string;
+    public function getHTML(string $url, ?bool $normalized = false): ?string;
 }

src/Application/ExternRefWorker.php

Spacing +6 added lines, -7 removed lines

@@ -25,7 +25,7 @@ 
     public const SLEEP_AFTER_EDITION = 15; // sec
     public const MINUTES_DELAY_AFTER_LAST_HUMAN_EDIT = 10; // minutes
     public const CHECK_EDIT_CONFLICT = true;
-    public const ARTICLE_ANALYZED_FILENAME = __DIR__ . '/resources/article_externRef_edited.txt';
+    public const ARTICLE_ANALYZED_FILENAME = __DIR__.'/resources/article_externRef_edited.txt';
     public const SKIP_ADQ = false;
     public const SKIP_LASTEDIT_BY_BOT = false;
     public const CITATION_NUMBER_ON_FIRE = 15;
@@ -58,7 +58,7 @@ 
             $result = $this->transformer->process($refContent, $this->summary);
         } catch (Throwable $e) {
             echo "** Problème détecté 234242\n";
-            $this->log->critical($e->getMessage() . " " . $e->getFile() . ":" . $e->getLine());
+            $this->log->critical($e->getMessage()." ".$e->getFile().":".$e->getLine());
             // TODO : parse $e->message -> variable process, taskName, botflag...
 
             return $refContent;
@@ -112,10 +112,10 @@ 
         $prefixSummary = ($this->summary->isBotFlag()) ? 'bot ' : '';
         $suffix = '';
         if (isset($this->summary->memo['count article'])) {
-            $suffix .= ' ' . $this->summary->memo['count article'] . 'x {article}';
+            $suffix .= ' '.$this->summary->memo['count article'].'x {article}';
         }
         if (isset($this->summary->memo['count lien web'])) {
-            $suffix .= ' ' . $this->summary->memo['count lien web'] . 'x {lien web}';
+            $suffix .= ' '.$this->summary->memo['count lien web'].'x {lien web}';
         }
         if (isset($this->summary->memo['presse'])) {
             $suffix .= '