[ilicfilip]

Row actions (toggle autoload, delete, create option with value false) could miss the option shown in the table, or reach the server with an empty name.

What was wrong

• jQuery .data() parses the name. handleTableActions read the name with button.data( 'option' ), which turns JSON-like names into values: ["foo"] became an array (sent as option_name[]=foo), {"a":1} an object, and null an empty value. Bulk delete already read the raw attribute with getAttribute( 'data-option' ).
• sanitize_text_field() changed names. The three single-option routes ran option_name through it, so a b became a b, a<b>c became ac, and so on. The action then missed the option or targeted a different one.
• <tr> ids repeated across tables. An option listed in both "All options" and another table got the same row id in both.

Changes

• handleTableActions reads button.attr( 'data-option' ).
• update-autoload, delete-option and create-option-false share get_option_name_arg(): type: string, minLength: 1, maxLength: 191, no sanitize_callback. Arrays and objects now get a 400 instead of relying on sanitize_text_field() returning ''.
• New getTableRowId() prefixes row ids with the table id. Single actions and bulk delete look rows up through it.

Testing

On a local site:

• REST as admin: array, object, empty and 192-character names return 400; aaaoo two spaces and aaaoo<b>tag are toggled exactly, no altered copy is created
• Browser, real row buttons in "All options": toggle and delete ["siteurl"], {"a":1}, null, 123, aaaoo two spaces, aaaoo<b>tag; each request sends the exact name, returns 200, and the options are gone from the database afterwards; siteurl unchanged
• "Unused" (server-side): remove autoload from a JSON-named option, row drops out of the list
• Bulk delete two options (one JSON-named) from "All options"
• No duplicate plugin ids on the page; console clean on load and after the actions
• "Create option with value false" button (same code path, not clicked)

Lint: PHPCS clean, PHPStan unchanged (44 existing errors), lint:js only the existing prettier error in the migration code.

🤖 Generated with Claude Code

[github-actions]

Test on Playground
Test this pull request on the Playground
or download the zip