$this->strTable not passed correctly into DC_Table::initPicker()

[Defcon0]

Hello folks,

I just found a possible issue in DC_Table::initPicker(). In this function the intersection of the root nodes with the mounted nodes doesn't work because the active table isn't passed into $this->eliminateNestedPages():

// Calculate the intersection of the root nodes with the mounted nodes (see #1001)
if (!empty($this->root) && $arrRoot != $this->root)
{
	$arrRoot = $this->eliminateNestedPages(
		array_intersect(
			array_merge($arrRoot, $this->Database->getChildRecords($arrRoot, $this->strTable)),
			array_merge($this->root, $this->Database->getChildRecords($this->root, $this->strTable))
		)
	);
}

It should be

if (!empty($this->root) && $arrRoot != $this->root)
{
	$arrRoot = $this->eliminateNestedPages(
		array_intersect(
			array_merge($arrRoot, $this->Database->getChildRecords($arrRoot, $this->strTable)),
			array_merge($this->root, $this->Database->getChildRecords($this->root, $this->strTable))
		),
		$this->strTable
	);
}

After doing this, I don't have the issue of having too many entries in my category picker anymore.

Hope this can be fixed in a future release.

Bye

[leofeyer]

Fixed in 94c9091. Thank you @Defcon0.