Skip to content

Fix duplicate indices in ExprIndices#8343

Merged
sovdeeth merged 2 commits intodev/featurefrom
feature/fix-duplicate-indices
Jan 5, 2026
Merged

Fix duplicate indices in ExprIndices#8343
sovdeeth merged 2 commits intodev/featurefrom
feature/fix-duplicate-indices

Conversation

@UnderscoreTud
Copy link
Member

Problem

Trying to get the indices of a list that has a sublist of multiple values, will return the index for that sublist multiple times.

set {_list::sublist::*} to 1 and 2
broadcast indices of {_list::*} # 'sublist and sublist`

Solution

Ensure all the keys returned by the expression are unique

Testing Completed

ExprIndices.sk and manual testing

Supporting Information

N/A


Completes: none
Related: none
AI assistance: none

@UnderscoreTud UnderscoreTud requested a review from a team as a code owner January 3, 2026 01:08
@UnderscoreTud UnderscoreTud requested review from Burbulinis and TheMug06 and removed request for a team January 3, 2026 01:08
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Jan 3, 2026
@skriptlang-automation skriptlang-automation bot added feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. and removed needs reviews A PR that needs additional reviews labels Jan 3, 2026
@sovdeeth sovdeeth merged commit 1fa2327 into dev/feature Jan 5, 2026
10 checks passed
@skriptlang-automation skriptlang-automation bot added completed The issue has been fully resolved and the change will be in the next Skript update. and removed feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. labels Jan 5, 2026
@sovdeeth sovdeeth added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label Jan 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. completed The issue has been fully resolved and the change will be in the next Skript update.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments