fix: Use multi-byte string methods

lib/Documents/src/AbstractDocumentFactory.php

@@ -231,7 +231,7 @@ public function updateDocument(DocumentInterface $document): DocumentInterface
                 }
             }
 
-            $document->setFolder(substr(hash("crc32b", date('YmdHi')), 0, 12));
+            $document->setFolder(\mb_substr(hash("crc32b", date('YmdHi')), 0, 12));
         }
 
         $document->setFilename($this->getFileName());

lib/Documents/src/Models/DocumentTrait.php

@@ -271,7 +271,7 @@ public function isEmbed(): bool
 
     protected function initDocumentTrait(): void
     {
-        $this->setFolder(substr(hash("crc32b", date('YmdHi')), 0, 12));
+        $this->setFolder(\mb_substr(hash("crc32b", date('YmdHi')), 0, 12));
     }
 
     #[

lib/Documents/src/TwigExtension/DocumentExtension.php

@@ -70,7 +70,7 @@ public function getFilters(): array
     public function formatBytes($bytes, int $precision = 2): string
     {
         $size = ['B','kB','MB','GB','TB','PB','EB','ZB','YB'];
-        $factor = floor((strlen((string) $bytes) - 1) / 3);
+        $factor = floor((\mb_strlen((string) $bytes) - 1) / 3);
         return sprintf("%.{$precision}f", (int) $bytes / pow(1024, $factor)) . @$size[$factor];
     }
 

lib/EntityGenerator/src/Field/NonVirtualFieldGenerator.php

@@ -58,7 +58,7 @@ protected function getMaxDefaultValuesLength(): int
         $max = 0;
         foreach (explode(',', $this->field->getDefaultValues()) as $value) {
             $value = trim($value);
-            $max = max($max, strlen($value));
+            $max = max($max, \mb_strlen($value));
         }
 
         return $max > 0 ? $max : 250;

lib/EntityGenerator/tests/mocks/JoinedTableDefaultValuesResolver.php

@@ -19,7 +19,7 @@ public function getMaxDefaultValuesLengthAmongAllFields(NodeTypeFieldInterface $
         // get max length of exploded default values
         $max = 0;
         foreach ($this->getDefaultValuesAmongAllFields($field) as $value) {
-            $max = max($max, strlen($value));
+            $max = max($max, \mb_strlen($value));
         }
 
         return $max > 0 ? $max : 250;

lib/Models/src/Roadiz/Utils/StringHandler.php

@@ -206,7 +206,7 @@ public static function replaceLast(string $search, string $replace, string $subj
         $pos = strrpos($subject, $search);
 
         if ($pos !== false) {
-            $subject = substr_replace($subject, $replace, $pos, strlen($search));
+            $subject = \substr_replace($subject, $replace, $pos, \mb_strlen($search));
         }
 
         return $subject;

lib/Random/src/PasswordGenerator.php

@@ -30,11 +30,11 @@ public function generatePassword(int $length = 12)
         $all = '';
         $password = '';
         foreach ($sets as $set) {
-            $password .= $set[array_rand(str_split($set))];
+            $password .= $set[array_rand(\mb_str_split($set))];
             $all .= $set;
         }
 
-        $all = str_split($all);
+        $all = \mb_str_split($all);
         for ($i = 0; $i < $length - count($sets); $i++) {
             $password .= $all[array_rand($all)];
         }

lib/RoadizCompatBundle/src/Controller/Controller.php

@@ -258,7 +258,7 @@ protected function generateUrl($route, array $parameters = [], int $referenceTyp
     public static function getCalledClass()
     {
         $className = get_called_class();
-        if (strpos($className, "\\") !== 0) {
+        if (\mb_strpos($className, "\\") !== 0) {
             $className = "\\" . $className;
         }
         return $className;

lib/RoadizCompatBundle/src/DependencyInjection/Compiler/ThemesTranslatorPathsCompilerPass.php

@@ -48,8 +48,8 @@ private function registerThemeTranslatorResources(ContainerBuilder $container):
                     ->followLinks()
                     ->files()
                     ->filter(function (\SplFileInfo $file) {
-                        return 2 <= substr_count($file->getBasename(), '.') &&
-                            preg_match('/\.\w+$/', $file->getBasename());
+                        return 2 <= \mb_substr_count($file->getBasename(), '.') &&
+                            \preg_match('/\.\w+$/', $file->getBasename());
                     })
                     ->in($translationFolder)
                     ->sortByName()
@@ -70,7 +70,7 @@ private function registerThemeTranslatorResources(ContainerBuilder $container):
                         'scanned_directories' => $scannedDirectories = [$translationFolder],
                         'cache_vary' => [
                             'scanned_directories' => array_map(static function (string $dir) use ($projectDir): string {
-                                return str_starts_with($dir, $projectDir . '/') ? substr($dir, 1 + \strlen($projectDir)) : $dir;
+                                return str_starts_with($dir, $projectDir . '/') ? \mb_substr($dir, 1 + \mb_strlen($projectDir)) : $dir;
                             }, $scannedDirectories),
                         ],
                     ]

lib/RoadizCoreBundle/src/Api/ListManager/SolrSearchListManager.php

@@ -56,7 +56,7 @@ public function handle(bool $disabled = false)
          * Query must be longer than 3 chars or Solr might crash
          * on highlighting fields.
          */
-        if (strlen($query) > 3) {
+        if (\mb_strlen($query) > 3) {
             $this->searchResults = $this->searchHandler->searchWithHighlight(
                 $query, # Use ?q query parameter to search with
                 $this->criteria, # a simple criteria array to filter search results

lib/RoadizCoreBundle/src/Console/FilesExportCommand.php

@@ -106,7 +106,7 @@ protected function zipFolder(ZipArchive $zip, string $folder, string $prefix = "
             if (!$file->isDir()) {
                 // Get real and relative path for current file
                 $filePath = $file->getRealPath();
-                $relativePath = substr($filePath, strlen($folder) + 1);
+                $relativePath = \mb_substr($filePath, \mb_strlen($folder) + 1);
 
                 // Add current file to archive
                 $zip->addFile($filePath, $prefix . '/' . $relativePath);

lib/RoadizCoreBundle/src/Console/UsersCreationCommand.php

@@ -145,7 +145,7 @@ private function executeUserCreation(
         }
 
         if ($input->getOption('password')) {
-            if (strlen($input->getOption('password')) < 5) {
+            if (\mb_strlen($input->getOption('password')) < 5) {
                 throw new \InvalidArgumentException('Password is too short.');
             }
 

lib/RoadizCoreBundle/src/Controller/RedirectionController.php

@@ -43,7 +43,7 @@ public function redirectAction(Request $request, Redirection $redirection): Redi
 
         if (
             null !== $redirection->getRedirectUri() &&
-            strlen($redirection->getRedirectUri()) > 0
+            \mb_strlen($redirection->getRedirectUri()) > 0
         ) {
             return new RedirectResponse($redirection->getRedirectUri(), $redirection->getType());
         }

lib/RoadizCoreBundle/src/CustomForm/CustomFormHelper.php

@@ -203,7 +203,7 @@ protected function getDocumentFolderForCustomForm(): ?Folder
             ->findOrCreateByPath(
                 'custom_forms/' .
                 $this->customForm->getCreatedAt()->format('Ymd') . '_' .
-                substr($this->customForm->getDisplayName(), 0, 30)
+                \mb_substr($this->customForm->getDisplayName(), 0, 30)
             );
     }
 

lib/RoadizCoreBundle/src/DependencyInjection/Compiler/DoctrineMigrationCompilerPass.php

@@ -29,9 +29,9 @@ private function checkIfBundleRelativePath(string $path, ContainerBuilder $conta
     {
         if (isset($path[0]) && $path[0] === '@') {
             $pathParts  = explode('/', $path);
-            $bundleName = substr($pathParts[0], 1);
+            $bundleName = \mb_substr($pathParts[0], 1);
             $bundlePath = $this->getBundlePath($bundleName, $container);
-            return $bundlePath . substr($path, strlen('@' . $bundleName));
+            return $bundlePath . \mb_substr($path, \mb_strlen('@' . $bundleName));
         }
 
         return $path;

lib/RoadizCoreBundle/src/Doctrine/EventSubscriber/NodesSourcesInheritanceSubscriber.php

@@ -66,7 +66,7 @@ public function loadClassMetadata(LoadClassMetadataEventArgs $eventArgs): void
                 $nodeTypes = $this->nodeTypes->all();
                 $map = [];
                 foreach ($nodeTypes as $type) {
-                    $map[strtolower($type->getName())] = $type->getSourceEntityFullQualifiedClassName();
+                    $map[\mb_strtolower($type->getName())] = $type->getSourceEntityFullQualifiedClassName();
                 }
                 $metadata->setDiscriminatorMap($map);
 

lib/RoadizCoreBundle/src/Doctrine/ORM/SimpleQueryBuilder.php

@@ -31,7 +31,7 @@ public function __construct(QueryBuilder $queryBuilder)
      */
     public function getParameterKey(string $key): string
     {
-        return strtolower(str_replace('.', '_', $key));
+        return \mb_strtolower(str_replace('.', '_', $key));
     }
 
     /**
@@ -57,7 +57,7 @@ public function buildExpressionWithBinding($value, string $prefix, string $key):
      */
     public function buildExpressionWithoutBinding($value, string $prefix, string $key, string $baseKey = null)
     {
-        if (strlen($prefix) > 0 && substr($prefix, -strlen('.')) !== '.') {
+        if (\mb_strlen($prefix) > 0 && \mb_substr($prefix, -\mb_strlen('.')) !== '.') {
             $prefix .= '.';
         }
 
@@ -111,7 +111,7 @@ public function buildExpressionWithoutBinding($value, string $prefix, string $ke
                         $fullKey = sprintf('LOWER(%s)', $prefix . $key);
                         return $this->queryBuilder->expr()->like(
                             $fullKey,
-                            $this->queryBuilder->expr()->literal(strtolower($value[1] ?? ''))
+                            $this->queryBuilder->expr()->literal(\mb_strtolower($value[1] ?? ''))
                         );
                     case 'NOT IN':
                         return $this->queryBuilder->expr()->notIn($prefix . $key, ':' . $baseKey);

lib/RoadizCoreBundle/src/Document/MessageHandler/DocumentExifMessageHandler.php

@@ -82,7 +82,7 @@ private function getCopyright(array $exif): ?string
         foreach ($exif as $key => $section) {
             if (is_array($section)) {
                 foreach ($section as $skey => $value) {
-                    if (strtolower($skey) === 'copyright') {
+                    if (\mb_strtolower($skey) === 'copyright') {
                         return $value;
                     }
                 }
@@ -99,18 +99,18 @@ private function getCopyright(array $exif): ?string
     private function getDescription(array $exif): ?string
     {
         foreach ($exif as $key => $section) {
-            if (is_string($section) && strtolower($key) === 'imagedescription') {
+            if (is_string($section) && \mb_strtolower($key) === 'imagedescription') {
                 return $section;
             } elseif (is_array($section)) {
-                if (strtolower($key) == 'comment') {
+                if (\mb_strtolower($key) == 'comment') {
                     $comment = '';
                     foreach ($section as $value) {
                         $comment .= $value . PHP_EOL;
                     }
                     return $comment;
                 } else {
                     foreach ($section as $skey => $value) {
-                        if (strtolower($skey) == 'comment') {
+                        if (\mb_strtolower($skey) == 'comment') {
                             return $value;
                         }
                     }

lib/RoadizCoreBundle/src/Entity/NodeType.php

@@ -478,7 +478,7 @@ public function getSourceEntityClassName(): string
     #[SymfonySerializer\Ignore]
     public function getSourceEntityTableName(): string
     {
-        return 'ns_' . strtolower($this->getName());
+        return 'ns_' . \mb_strtolower($this->getName());
     }
 
     /**

lib/RoadizCoreBundle/src/Entity/Role.php

@@ -193,7 +193,7 @@ public function removeGroup(Group $group): Role
     #[SymfonySerializer\Groups(['role'])]
     public function getClassName(): string
     {
-        return str_replace('_', '-', strtolower($this->getRole()));
+        return str_replace('_', '-', \mb_strtolower($this->getRole()));
     }
 
     /**

lib/RoadizCoreBundle/src/Entity/Setting.php

@@ -135,7 +135,7 @@ public function getName(): string
      */
     public function setName(?string $name)
     {
-        $this->name = trim(strtolower($name ?? ''));
+        $this->name = trim(\mb_strtolower($name ?? ''));
         $this->name = (new UnicodeString($this->name))
             ->ascii()
             ->toString();

lib/RoadizCoreBundle/src/Entity/User.php

@@ -741,7 +741,7 @@ public function setChroot(Node $chroot = null): User
     public function getGravatarUrl(string $type = "identicon", string $size = "200"): string
     {
         if (null !== $this->getEmail()) {
-            return "https://www.gravatar.com/avatar/" . md5(strtolower(trim($this->getEmail()))) . "?d=" . $type . "&s=" . $size;
+            return "https://www.gravatar.com/avatar/" . md5(\mb_strtolower(trim($this->getEmail()))) . "?d=" . $type . "&s=" . $size;
         }
         return '';
     }
@@ -932,7 +932,7 @@ public function hasGroup(string $name): bool
      */
     public function hasRole(string $role): bool
     {
-        return in_array(strtoupper((string) $role), $this->getRoles(), true);
+        return in_array(\mb_strtoupper((string) $role), $this->getRoles(), true);
     }
 
     /**

lib/RoadizCoreBundle/src/Exception/ExceptionViewer.php

@@ -266,8 +266,8 @@ public function isFormatJson(Request $request): bool
         if (
             $request->headers->get('Content-Type') &&
             (
-                0 === strpos($request->headers->get('Content-Type'), 'application/json') ||
-                0 === strpos($request->headers->get('Content-Type'), 'application/ld+json')
+                0 === \mb_strpos($request->headers->get('Content-Type'), 'application/json') ||
+                0 === \mb_strpos($request->headers->get('Content-Type'), 'application/ld+json')
             )
         ) {
             return true;

lib/RoadizCoreBundle/src/Form/Constraint/HexadecimalColorValidator.php

@@ -12,7 +12,7 @@ class HexadecimalColorValidator extends ConstraintValidator
     public function validate(mixed $value, Constraint $constraint): void
     {
         if ($constraint instanceof HexadecimalColor) {
-            if (null !== $value && preg_match('#\#[0-9a-f]{6}#', strtolower($value)) === 0) {
+            if (null !== $value && preg_match('#\#[0-9a-f]{6}#', \mb_strtolower($value)) === 0) {
                 $this->context->addViolation($constraint->message);
             }
         }

lib/RoadizCoreBundle/src/Form/Constraint/NonSqlReservedWordValidator.php

@@ -14,7 +14,7 @@ public function validate(mixed $value, Constraint $constraint): void
     {
         if (null !== $value) {
             $fieldName = StringHandler::variablize($value);
-            $lowerName = strtolower($value);
+            $lowerName = \mb_strtolower($value);
             if (
                 in_array($value, NonSqlReservedWord::$forbiddenNames) ||
                 in_array($lowerName, NonSqlReservedWord::$forbiddenNames) ||

lib/RoadizCoreBundle/src/Form/Constraint/SimpleLatinStringValidator.php

@@ -12,7 +12,7 @@ class SimpleLatinStringValidator extends ConstraintValidator
     public function validate(mixed $value, Constraint $constraint): void
     {
         if ($constraint instanceof SimpleLatinString) {
-            if (null !== $value && preg_match('#[^a-z_\s\-]#', strtolower($value)) === 1) {
+            if (null !== $value && preg_match('#[^a-z_\s\-]#', \mb_strtolower($value)) === 1) {
                 $this->context->addViolation($constraint->message);
             }
         }

lib/RoadizCoreBundle/src/Form/Constraint/UniqueTagNameValidator.php

@@ -80,6 +80,6 @@ protected function tagNameExists(string $name): bool
      */
     protected function isMulti(?string $value): bool
     {
-        return (bool) strpos($value ?? '', ',');
+        return (bool) \mb_strpos($value ?? '', ',');
     }
 }

lib/RoadizCoreBundle/src/ListManager/Paginator.php

@@ -251,7 +251,7 @@ protected function getSearchQueryBuilder(string $alias): QueryBuilder
         foreach ($this->getSearchableFields() as $field) {
             $orX[] = $qb->expr()->like(
                 'LOWER(' . $alias . '.' . $field . ')',
-                $qb->expr()->literal('%' . mb_strtolower($this->searchPattern) . '%')
+                $qb->expr()->literal('%' . \mb_strtolower($this->searchPattern) . '%')
             );
         }
         $qb->andWhere($qb->expr()->orX(...$orX));

lib/RoadizCoreBundle/src/Mailer/ContactFormManager.php

@@ -526,7 +526,7 @@ protected function isFieldPrivate(FormInterface $form): bool
         ];
         return
             is_string($key) &&
-            (substr($key, 0, 1) === '_' || \in_array($key, $privateFieldNames))
+            (\mb_substr($key, 0, 1) === '_' || \in_array($key, $privateFieldNames))
         ;
     }
 

lib/RoadizCoreBundle/src/Node/NodeFactory.php

@@ -80,7 +80,7 @@ public function create(
         if (true === $repository->exists($nodeName)) {
             $nodeName = $this->nodeNamePolicy->getSafeNodeName($source);
         }
-        if (mb_strlen($nodeName) > 250) {
+        if (\mb_strlen($nodeName) > 250) {
             throw new \InvalidArgumentException(sprintf('Node name "%s" is too long.', $nodeName));
         }
         $node->setNodeName($nodeName);

lib/RoadizCoreBundle/src/Node/NodeNameChecker.php

@@ -35,14 +35,14 @@ public function getCanonicalNodeName(NodesSources $nodeSource): string
             $title = StringHandler::slugify($nodeSource->getTitle());
             if ($nodeSource->isReachable() || !$this->useTypedSuffix) {
                 // truncate title to 250 chars if needed
-                if (strlen($title) > self::MAX_LENGTH) {
-                    $title = substr($title, 0, self::MAX_LENGTH);
+                if (\mb_strlen($title) > self::MAX_LENGTH) {
+                    $title = \mb_substr($title, 0, self::MAX_LENGTH);
                 }
                 return $title;
             }
             // truncate title if title + suffix + 1 exceed 250 chars
-            if ((strlen($title) + strlen($nodeTypeSuffix) + 1) > self::MAX_LENGTH) {
-                $title = substr($title, 0, self::MAX_LENGTH - (strlen($nodeTypeSuffix) + 1));
+            if ((\mb_strlen($title) + \mb_strlen($nodeTypeSuffix) + 1) > self::MAX_LENGTH) {
+                $title = \mb_substr($title, 0, self::MAX_LENGTH - (\mb_strlen($nodeTypeSuffix) + 1));
             }
             return sprintf(
                 '%s-%s',
@@ -63,11 +63,11 @@ public function getSafeNodeName(NodesSources $nodeSource): string
         $uniqueId = uniqid();
 
         // truncate canonicalNodeName if canonicalNodeName + uniqueId + 1 exceed 250 chars
-        if ((strlen($canonicalNodeName) + strlen($uniqueId) + 1) > self::MAX_LENGTH) {
-            $canonicalNodeName = substr(
+        if ((\mb_strlen($canonicalNodeName) + \mb_strlen($uniqueId) + 1) > self::MAX_LENGTH) {
+            $canonicalNodeName = \mb_substr(
                 $canonicalNodeName,
                 0,
-                self::MAX_LENGTH - (strlen($uniqueId) + 1)
+                self::MAX_LENGTH - (\mb_strlen($uniqueId) + 1)
             );
         }
 
@@ -84,11 +84,11 @@ public function getDatestampedNodeName(NodesSources $nodeSource): string
         $timestamp = $nodeSource->getPublishedAt()->format('Y-m-d');
 
         // truncate canonicalNodeName if canonicalNodeName + uniqueId + 1 exceed 250 chars
-        if ((strlen($canonicalNodeName) + strlen($timestamp) + 1) > self::MAX_LENGTH) {
-            $canonicalNodeName = substr(
+        if ((\mb_strlen($canonicalNodeName) + \mb_strlen($timestamp) + 1) > self::MAX_LENGTH) {
+            $canonicalNodeName = \mb_substr(
                 $canonicalNodeName,
                 0,
-                self::MAX_LENGTH - (strlen($timestamp) + 1)
+                self::MAX_LENGTH - (\mb_strlen($timestamp) + 1)
             );
         }
 

lib/RoadizCoreBundle/src/Node/NodeTranstyper.php

@@ -287,7 +287,7 @@ protected function mockTranstype(NodeTypeInterface $nodeType): void
 
         $translation = new Translation();
         $translation->setAvailable(true);
-        $translation->setLocale(substr($uniqueId, 0, 10));
+        $translation->setLocale(\mb_substr($uniqueId, 0, 10));
         $translation->setName('test' . $uniqueId);
         $this->getManager()->persist($translation);