\Xmf\Metagen::generateDescription() and xlanguage

[GregMage]

This method does not work with xlanguage. With the first language, no problem but if we activate the second then Xoops crashes.

To avoid the problem I wrote this in my module:

$xoTheme->addMeta('meta', 'description', \Xmf\Metagen::generateDescription(preg_replace("/\[.*?\]/", '', $content->getVar('content_text')), 30));

I don't know if it's really good what I'm doing. We can add this code to the method "asPlainText()"

$text = preg_replace('/\[.*?\]/', ' ', $text);

The full code:

    protected static function asPlainText($rawText)
    {
        $text = $rawText;
        $text = static::html2text($text);
        $text = static::purifyText($text);

        $text = str_replace(array("\n", "\r"), ' ', $text);
        $text = preg_replace('/[ ]* [ ]*/', ' ', $text);
        $text = preg_replace('/\[.*?\]/', ' ', $text);

        return trim($text);
    }

[geekwright]

To build a clean description, we need to have the xlanguage processing for the string field to be run.

I am thinking a preload could help out with this.

We could add a preload that would do the language based selection to xlanguage, and trigger an event in Metagen to condition the string before processing. If xlanguage is not installed there would be no processing, and no changes would be made to the string.

Would this solve the issues?

[GregMage]

I solved the problem like this:

	public static function generateDescriptionTagSafe($text, $wordCount = 100)
    {
		if (xoops_isActiveModule('xlanguage')){
			$text = XoopsModules\Xlanguage\Utility::cleanMultiLang($text);
		}
		$text = \Xmf\Metagen::generateDescription($text, $wordCount);
		return $text;
	}
	
	public static function TagSafe($text)
    {
		if (xoops_isActiveModule('xlanguage')){
			$text = XoopsModules\Xlanguage\Utility::cleanMultiLang($text);
		}
		return $text;
	}