Möglicherweise Fehler in der Kategorieübersicht

[martin2002]

Hi...

Ich habe beim upgrade einer Seite nach version 4.8.7 auf der ich plugins, die die Chains Contenido.CategoryList.Columns und RenderColumns benutzen, laufen habe wahrscheinlich einen fehler beim aufbau der kategorieübersicht entdeckt:

(datei includes/include.str_overview.php)
zeile 438 (code für chain Columns)

Code: Alles auswählen

$tmplistColumns = $chainEntry->execute(array());

if (is_array($tmplistColumns))
{
	$listColumns = $tmplistColumns;
}

so ist der originalcode... das führt aber dazu, dass nur die zusätzlichen spalten aus der letzten chainfunktion eingefügt werden.
sowas wie:

Code: Alles auswählen

$listColumns = array_merge($listColumns, $tmplistColumns);

sollte meiner ansicht nach hier stehen.

außerdem gibt es wie ich glaube auch ein problem beim aufbau der content zellen. jede ausgabe einer chainfunktion von RenderColumn wird in ein seperates <td></td> ausgegeben... da es nach mein verständnis der funktionalität mehrere chainfunktionen für eine zusätzliche spalte geben kann, sind später zu viele zellen in einer zeile. ich habe zeile 803 bis 808

Code: Alles auswählen

foreach ($columnContents as $columnContent)
{
	$columnInfo[] = '<td class="str-style-c">'.$columnContent.'</td>';
}	                			
       			
$columns[] = implode("", $columnInfo);

duch folgenden code ersetzt:

Code: Alles auswählen

$columns[] = '<td class="str-style-c">'.implode("", $columnContents).'</td>';

zu meinem erstaunen hatten die leerzeilen zwischen kategoriebäumen und der tabellenkopf immer noch überbreite. ein genaueres hinsehen ergab, dass für diese zeilen colspan="3" eingestellt ist. (zeile 446 und 557)
ich habe diese formatierung auch entfernt und die ansicht ist in meinem system nun korrekt.

was meint ihr dazu?
falls ich da bei der funktion etwas missverstanden habe, klärt mich bitte auf...

greets
Martin.

[emergence]

klingt nach bug...

verschoben...

[timo.trautmann_4fb]

Problem bestätigt. Checkin und fixxed wie beschrieben.