Wrong order of sub-meshes upon upload

[canny]

It seems that now we have deterministic link order when uploading multiple meshes in a DAE or GLTF/GLB file, however even though it puts the first object as root, based on alphanumerical order, it puts the rest of the links in reverse order.

Test this:

1. Create a DAE or GLTF file where each mesh is named A through E.

2. Confirm that the file has the meshes listed in order of A....E by opening it in a text editor (doesn't work for GLB, obviously, but does for GLTF or DAE)

3. Upload 'as you do'

4. Check link order, and find them to be in the order A, E, D, C, B where A is root.***

*** In the case of DAE, this seems to be true for uploading in the latest FS Beta (7.2.0.78879), but besides root being 'A' in all my tests, the other links seems to end up random when uploaded with the SL Viewer, not even just in reverse order.

Expected behaviour is to have all links in alphanumerical order, where A is root and is then followed by B, C, D, E, regardless of GLTF or DAE.

Related Canny, which can be closed: https://feedback.secondlife.com/bug-reports/p/random-root-object-assignment-upon-upload-of-multi-object-dae-file

https://secondlife.canny.io/admin/board/bug-reports/p/wrong-order-of-sub-meshes-upon-upload

---

This repo is using Opire - what does it mean? 👇

💵 Everyone can add rewards for this issue commenting /reward 100 (replace 100 with the amount).
🕵️‍♂️ If someone starts working on this issue to earn the rewards, they can comment /try to let everyone know!
🙌 And when they open the PR, they can comment /claim #4515 either in the PR description or in a PR's comment.

🪙 Also, everyone can tip any user commenting /tip 20 @canny[bot] (replace 20 with the amount, and @canny[bot] with the user to tip).

📖 If you want to learn more, check out our documentation.

[canny]

This issue has been linked to a Canny post: Wrong order of sub-meshes upon upload 🎉

[akleshchev]

Check link order, and find them to be in the order A, E, D, C, B where A is root.***

How do you check link order? Ordering was implemented in #4294

[marchcat]

Right. Doing that in llmeshrepository.cpp looks more appropriate.

[NyxXaos]

How to check link order in the SL Viewer I don't know, but you can see the link number in the Firestorm viewer in the Toolbox. You can also list link numbers and link names using a script, or have them displayed with hover text.

[NyxXaos]

Here's a simple LSL test script I created, that sets a hover text with info about object name and link number... just drop it onto the uploaded linkset.

default
{
    state_entry()
    {
        integer iTotalLinks = llGetNumberOfPrims();
        if(iTotalLinks > 1)
        {
            integer iCurrentLink = llGetLinkNumber();
            for(; iCurrentLink < iTotalLinks + 1; iCurrentLink++)
                llSetLinkPrimitiveParamsFast(iCurrentLink, [
                PRIM_TEXT,
                "(link #" + (string)iCurrentLink + ")\n" + llGetLinkName(iCurrentLink),
                <1.0, 0.0, 0.0>,
                1.0
                ]);
        }
    }
}

Preview....

[NyxXaos]

Suggested test plan:

1. Upload a mesh with several submesheses, one upload using DAE, one with GLTF, each mesh named in alphanumerical order.
2. Rez the uploads onto the ground, drop the above-provided scripts onto both linksets
3. Take note of what the hovertexts show, correct behaviour should be that the link numbers correspond to the object names:

A = # 1
B = # 2
C = # 3 and so on...

Attached: zip with a DAE, a GLTF, and the original Blend file

Mesh Order - Everything.zip

p.s. if you have issues uploading the provided .gltf file, see https://feedback.secondlife.com/bug-reports/p/gltf-uploader-not-handling-20-in-gltf-file-reference-to-bin-file