Bulk mesh uploader: after sorting by LI, clicking a row opens the upload window for the wrong model (the item at that position in name order) #1463

New issue

Open

opened 2026-11-05 08:39:42 +00:00 by sky_line · 0 comments

sky_line commented

2026-11-05 08:39:42 +00:00

Owner

Bug

In the bulk mesh uploader, after sorting the list by LI (land impact), clicking a row opens the upload window for the wrong item. For example, clicking the top row opens the upload window for the item that was at the top by name, before sorting, not the item that was clicked.

Repro

Open the bulk mesh upload and add several models.
Sort the list by the LI column.
Click the top row (or any row).
The upload / preview window opens for a different model: the one at that row index in the unsorted (name) order.

Expected

Clicking a row opens the window for exactly the model shown in that row, whatever the sort column or order.

Likely cause

The row's position (index) in the displayed list is used to look up the model in the underlying, unsorted array. The row should carry the model's identity (an id or value) instead, and the lookup should use that, or the sorted order should be mapped back.

Code: llfloatermeshbulkupload.cpp / llmeshbulkupload.cpp (and the scroll list's selection and double-click handlers).

## Bug In the bulk mesh uploader, after sorting the list by LI (land impact), clicking a row opens the upload window for the wrong item. For example, clicking the top row opens the upload window for the item that was at the top **by name**, before sorting, not the item that was clicked. ## Repro 1. Open the bulk mesh upload and add several models. 2. Sort the list by the LI column. 3. Click the top row (or any row). 4. The upload / preview window opens for a different model: the one at that row index in the unsorted (name) order. ## Expected Clicking a row opens the window for exactly the model shown in that row, whatever the sort column or order. ## Likely cause The row's position (index) in the displayed list is used to look up the model in the underlying, unsorted array. The row should carry the model's identity (an id or value) instead, and the lookup should use that, or the sorted order should be mapped back. Code: `llfloatermeshbulkupload.cpp` / `llmeshbulkupload.cpp` (and the scroll list's selection and double-click handlers).

sky_line added the

bug

label

2026-11-05 08:39:42 +00:00

sky_line added this to the Slipstream project

2026-11-05 08:39:42 +00:00