Bug in SalInstanceTreeView::set_column_editables() ?

[Jan Rheinländer]

Hi,

maybe there is a bug in SalInstanceTreeView::set_column_editables():

It calls SvTabListBox::SetTabEditable(), which sets the EDITABLE flag in 
mvTabList array of SvLBoxTab.

But the parent class SvTreeListBox keeps another array of SvLBoxTab 
called aTabs.

When the user presses the mouse button on a cell in a treeview, 
SvImpLBox::MouseButtonDown() is called.

It gets the tab SvTreeListBox::GetTab(), which retrieves it from the 
aTabs array.

Then it checks the EDITABLE flag with SvLBoxTab::IsEditable().

But of course the EDITABLE flag has been set in the mvTabList array of 
the subclass SvTabListBox.

The net result is that only the first column is editable, since that is 
always set to editable by default (in both tab arrays).

Is this a bug? And how to fix it? What is the point of the second tab 
array in the subclass?

Jan


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice/attachments/20250114/f98f3277/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 321 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/libreoffice/attachments/20250114/f98f3277/attachment.sig>