Please read the docblock, $id is a string, thus, $module->id is casted to be a string.

protected static function &load() calls public static function getModuleList()

gives back an array with $id as integer

ids in joomla databases are integers.
so the dockblock is wrong!

right?

$id is parsed here:

$match[1] is string type.

array:2 [
  0 => "{loadmoduleid 134}"
  1 => "134"
]

The docblock is correct.

Please do something with wood.

Am 29. April 2025 19:09:40 UTC schrieb Quy Ton @.***>:

QuyTon left a comment (#45408)

$id is parsed here:

joomla-cms/plugins/content/loadmodule/src/Extension/LoadModule.php

Lines 164 to 180 in 78d2b75

	if (str_contains($article->text, '{loadmoduleid ')) {
	// Find all instances of plugin and put in $matchesmodid for loadmoduleid
	preg_match_all($regexmodid, $article->text, $matchesmodid, PREG_SET_ORDER);
	// If no matches, skip this
	if ($matchesmodid) {
	foreach ($matchesmodid as $match) {
	$id = trim($match[1]);
	$output = $this->loadID($id);
	// We should replace only first occurrence in order to allow positions with the same name to regenerate their content:
	if (($start = strpos($article->text, $match[0])) !== false) {
	$article->text = substr_replace($article->text, $output, $start, \strlen($match[0]));
	}
	}
	}
	}

$match[1] is string type.

array:2 [
 0 => "{loadmoduleid 134}"
 1 => "134"
]

The docblock is correct.

--
Reply to this email directly or view it on GitHub:
#45408 (comment)
You are receiving this because you authored the thread.

Message ID: @.***>

Closing as expected behavior. Thanks for reporting.