SoundUtils.applyPropertiesFromId sets SoundId value twice if given a table.
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 72/100
Research direction
Find SoundUtils.applyPropertiesFromId and trace the table handling described in the issue. Check the related tests or examples for SoundUtils.playFromId and LoopedSoundPlayer, then reproduce with a numeric SoundId in Roblox Studio. Done when the converted asset ID is the value assigned and the error no longer occurs.
Written by the indexing model from the issue text.
Description
Describe the bug
If SoundUtils.applyPropertiesFromId is given a table, it converts the table's SoundId value to a rbxassetid and sets the Sound's SoundId property to that, then sets it again using the original, unconverted value. If the original value was a number, this results in an error.
To Reproduce
Steps to reproduce the behavior:
- Create a script that creates and plays sound with SoundUtils, passing a table as an argument. Make sure the table's SoundId value is a number.
SoundUtils.playFromIdandLoopedSoundPlayerboth work for this. - Test in Studio
- See error in output and Sound instance with a number in the SoundId property.
Expected behavior
The SoundId should use the converted value.
- Dominant language
- Lua
- Stars
- 632
- Forks
- 148
- Avg merge
- 15h 41m
- Merged PRs (30d)
- 13
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Quenty/NevermoreEngine
-
bug
Difficulty 3/5 1-2 days Newbie friendliness 45/100
Quenty/NevermoreEngine#670 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 3/5 1-2 days Newbie friendliness 52/100
Quenty/NevermoreEngine#583 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 3/5 1-2 days Newbie friendliness 35/100
Quenty/NevermoreEngine#507 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 3/5 1-2 days Newbie friendliness 45/100
Quenty/NevermoreEngine#471 · 1 reaction ·
Maintainers usually reply within 1 day
-
Implement Deferred/Adapted Signal using LuaMay be free again A pull request for this issue was closed without being merged. Open
Difficulty 5/5 Over a week Newbie friendliness 30/100
Quenty/NevermoreEngine#463 · 1 comment ·
Maintainers usually reply within 1 day
All issues in Quenty/NevermoreEngine
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
obsidian-nvim/obsidian.nvim#1027 ·
Maintainers usually reply within 1 day
-
area: top panel enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
AndyHazz/bookshelf.koplugin#599 · 2 comments ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
severity: low
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
luainkernel/lunatik#2460 ·
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 1 day