Collapse: Fix multiple ids calls

[louismaximepiton]

Description

With git bisect, it seems the breaking change appeared from #37438.
The only change in here was that the parseSelector was applied in any case and not on some specific cases anymore. Reverting this change.

Motivation & Context

Break some (wrong?) use cases.

Type of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Refactoring (non-breaking change)
• Breaking change (fix or feature that would change existing functionality)

Checklist

• I have read the contributing guidelines
• My code follows the code style of the project (using npm run lint)
• My change introduces changes to the documentation
• I have updated the documentation accordingly
• I have added tests to cover my changes
• All new and existing tests passed

Live previews

• https://deploy-preview-38989--twbs-bootstrap.netlify.app/

Tried it with the minimal code:

<button data-bs-toggle="collapse" data-bs-target="#hello,#world">
  collapse
</button>

<div id="hello" class="collapse">HELLO</div>
<div id="world" class="collapse">WORLD</div>

Related issues

Closes #38974.

[XhmikosR]

Nice git bisecting!

Confirmed it fixes #38974. @GeoSot LGTY?

[XhmikosR]

As for the tests, yeah, ideally we should have one, because apparently it's not covered by the existent tests :)

[Parasuramragu]

its an clear review in this program
I had checked the program in this compailer in this program and i have check the program in VS Code

[julien-deramond]

Nice analysis 💪 LGTM!