diff options
author | Thomas Lange <code@nerdmind.de> | 2024-11-15 16:41:11 +0100 |
---|---|---|
committer | Thomas Lange <code@nerdmind.de> | 2024-11-15 17:10:11 +0100 |
commit | 5e076b70b79ddf7bc6e07122b4646961db280be5 (patch) | |
tree | 33d2a85bac6252db35913e42bfd9879925aaef6c | |
parent | eb773b95f08dc79df1acad96296a92a9c71d48f1 (diff) | |
download | blog-5e076b70b79ddf7bc6e07122b4646961db280be5.tar.gz blog-5e076b70b79ddf7bc6e07122b4646961db280be5.tar.xz blog-5e076b70b79ddf7bc6e07122b4646961db280be5.zip |
FunctionParser: No static property/methods anymore
Problem: When creating a new "FunctionParser" object, it should not know
the previously registered content functions which were saved in a static
class property and registered via static class method.
If one creates a new "FunctionParser" object, one would expect that the
array of registered content functions in that object is empty.
Changes:
- Make static property "functions" non-static
- Make static method "register" non-static
- Add new method "registerFromArray"
-rw-r--r-- | core/functions.php | 10 | ||||
-rw-r--r-- | core/namespace/Application.php | 1 | ||||
-rw-r--r-- | core/namespace/Parsers/FunctionParser.php | 19 |
3 files changed, 23 insertions, 7 deletions
diff --git a/core/functions.php b/core/functions.php index d4337a4..9202bad 100644 --- a/core/functions.php +++ b/core/functions.php @@ -101,6 +101,10 @@ function generateItemTemplateData(EntityInterface $Entity): array { $FunctionParser = new FunctionParser; $MarkdownParser = new MarkdownParser; + $FunctionParser->registerFromArray( + Application::getContentFunctions() + ); + $attribute = $Entity->getAll(['password']); $attribute = array_change_key_case($attribute, CASE_UPPER); @@ -159,7 +163,11 @@ function generateCategoryDataTree(array $category_data, $root = 0): array { function parseEntityContent(EntityInterface $Entity): string { $text = $Entity->get('body'); - $FunctionParser = new FunctionParser(); + $FunctionParser = new FunctionParser; + $FunctionParser->registerFromArray( + Application::getContentFunctions() + ); + $text = $FunctionParser->transform($text); if(Application::get('WRAP_EMOTICONS')) { diff --git a/core/namespace/Application.php b/core/namespace/Application.php index 01c36ac..1f1e6f6 100644 --- a/core/namespace/Application.php +++ b/core/namespace/Application.php @@ -224,7 +224,6 @@ class Application { contain only numbers, uppercase letters and underscores!'); } - FunctionParser::register($name, $callback); self::$contentFunctions[$name] = $callback; } diff --git a/core/namespace/Parsers/FunctionParser.php b/core/namespace/Parsers/FunctionParser.php index 02570a0..94a4c92 100644 --- a/core/namespace/Parsers/FunctionParser.php +++ b/core/namespace/Parsers/FunctionParser.php @@ -3,7 +3,7 @@ namespace Parsers; use ReflectionFunction; class FunctionParser implements ParserInterface { - private static $functions = []; + private $functions = []; #=============================================================================== # Main regex for matching the whole function call @@ -39,19 +39,28 @@ class FunctionParser implements ParserInterface { #=============================================================================== # Register function #=============================================================================== - public static function register(string $name, callable $callback): void { + public function register(string $name, callable $callback): void { $Function = new ReflectionFunction($callback); - self::$functions[$name] = [ + $this->functions[$name] = [ 'callback' => $callback, 'required' => $Function->getNumberOfRequiredParameters() ]; } #=============================================================================== + # Register multiple functions from array + #=============================================================================== + public function registerFromArray(array $functions): void { + foreach($functions as $name => $callback) { + $this->register($name, $callback); + } + } + + #=============================================================================== # Parse functions #=============================================================================== public function parse(string $text): array { - $functionNames = array_keys(self::$functions); + $functionNames = array_keys($this->functions); $functionNames = implode('|', $functionNames); $pattern = self::FUNCTION_PATTERN; @@ -72,7 +81,7 @@ class FunctionParser implements ParserInterface { # Transform functions #=============================================================================== public function transform(string $text): string { - $functionData = self::$functions; + $functionData = $this->functions; $functionNames = array_keys($functionData); $functionNames = implode('|', $functionNames); |