PSMList 0.4.0 PSM-040-048

in genereicServiceTools<br/><br/> private int $strategySlugnameLength = 16;<br/><br/><br/>and <br/><br/> public function getStrategySlugnameLength(): int {<br/> return $this->strategySlugnameLength;<br/> }

/**<br/> * Returns a random uppercase string of the desired length, only using read-safe letters and numbers.<br/> * If no length is specified, it uses the default strategy length.<br/> * @param int|null $length Number of characters in the returned string<br/> * @return string<br/> */<br/> public function random_safe_str(?int $length = null) :string {<br/> $entityClasses = [Ship::class, Crew::class, Treasure::class, Keyword::class];<br/> $characters = '123456789ABCDEFGHJKLMNPQRSTUVWXYZ';<br/><br/> // Use the default strategy if the parameter isn't specified<br/> $length = $length ?? $this->getStrategySlugnameLength();<br/><br/> // Generate new slugnames until we got one that's unique<br/> do {<br/> $randomString = '';<br/> for ($i = 0; $i < $length; $i++) {<br/> $index = rand(0, strlen($characters) - 1);<br/> $randomString .= $characters[$index];<br/> }<br/> } while ($this->doesSlugnameExists($entityClasses, $randomString));<br/><br/> return $randomString;<br/> }

also don't forget to remove the parameter to all the calls of safe_str unlesss we really want another value