From a928549d50bbb77b0d6e8dcc51a6f61f3585d6d0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Melega?= Date: Thu, 23 May 2024 16:46:44 +0200 Subject: [PATCH] refactor: remove $lineSeparator and replace `explode` with `preg_split` --- xExtension-WordHighlighter/extension.php | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/xExtension-WordHighlighter/extension.php b/xExtension-WordHighlighter/extension.php index b43115b..29b85e2 100644 --- a/xExtension-WordHighlighter/extension.php +++ b/xExtension-WordHighlighter/extension.php @@ -62,13 +62,12 @@ final class WordHighlighterExtension extends Minz_Extension $this->case_sensitive = (bool) Minz_Request::paramString('case_sensitive'); $this->separate_word_search = (bool) Minz_Request::paramString('separate_word_search'); - $lineSeparator = ((bool) strpos($configWordList, "\r\n")) ? "\r\n" : "\n"; $configObj = [ 'enable_in_article' => $this->enable_in_article, 'enable_logs' => $this->enable_logs, 'case_sensitive' => $this->case_sensitive, 'separate_word_search' => $this->separate_word_search, - 'words' => explode($lineSeparator, $configWordList), + 'words' => preg_split("/\r\n|\n|\r/", $configWordList), ]; $configJson = json_encode($configObj, WordHighlighterExtension::JSON_ENCODE_CONF); file_put_contents(join_path($staticPath, ('config.' . $current_user . '.json')), $configJson . PHP_EOL);