Dispositif / Wikibot

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

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/Application/RecentChangeWorker.php

Spacing +3 added lines, -3 removed lines

@@ -41,15 +41,15 @@
 
     public function trackUser(string $user): void
     {
-        echo "**** TRACK " . $user . "*****\n";
+        echo "**** TRACK ".$user."*****\n";
 
         $titles = $this->getLastEditsbyUser($user);
 
         // filter titles already in edited.txt
         $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 ); // TODO PageList factory in App ?
-        echo ">" . $list->count() . " dans liste\n";
+        $list = new PageList($filtered); // TODO PageList factory in App ?
+        echo ">".$list->count()." dans liste\n";
 
         $this->consumeList($list);
     }

src/Application/ExternRefWorker.php

Spacing +6 added lines, -7 removed lines

@@ -27,7 +27,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;
@@ -60,7 +60,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;
@@ -120,10 +120,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 .= ' 

src/Application/CLI/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";