Skip to content

Fix params2list() duplicate key behavior#2551

Open
ike709 wants to merge 2 commits intoOpenDreamProject:masterfrom
ike709:paramity
Open

Fix params2list() duplicate key behavior#2551
ike709 wants to merge 2 commits intoOpenDreamProject:masterfrom
ike709:paramity

Conversation

@ike709
Copy link
Copy Markdown
Collaborator

@ike709 ike709 commented Apr 5, 2026

At some point BYOND changed duplicate key behavior from "use the last value" to "make a list of all the values".

From the ref:

If you have multiple items with the same name, they will be combined into a list of text strings. For example, "key=value1;key=value2" would set list["key"] to a list containing "value1" and "value2", not necessarily in that order.

See also #2444 and #2427. This ticks a box on the latter.

@boring-cyborg boring-cyborg bot added the Runtime Involves the OpenDream server/runtime label Apr 5, 2026
@github-actions github-actions bot added the size/S label Apr 5, 2026
@ike709 ike709 requested a review from wixoaGit April 5, 2026 06:30
@github-actions github-actions bot added size/XS and removed size/S labels Apr 6, 2026
@ike709 ike709 requested a review from wixoaGit April 6, 2026 07:32
@ike709 ike709 changed the title Update params2list() duplicate key behavior Fix params2list() duplicate key behavior Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Runtime Involves the OpenDream server/runtime size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants