Skip to content

[WIP] Fix TODO comment regarding dual form handling in custom HTML widget#149

Closed
Claude wants to merge 2 commits into
autopilotfrom
claude/address-todo-in-custom-html-widget
Closed

[WIP] Fix TODO comment regarding dual form handling in custom HTML widget#149
Claude wants to merge 2 commits into
autopilotfrom
claude/address-todo-in-custom-html-widget

Conversation

@Claude
Copy link
Copy Markdown

@Claude Claude AI commented Apr 15, 2026

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>Address TODO in wp-includes/widgets/class-wp-widget-custom-html.php: This is lacking, as some languages have a dedicated dual form. For proper handli</issue_title>
<issue_description>Found a @todo comment that needs attention.

File: src/wp-includes/widgets/class-wp-widget-custom-html.php
Line: 233
Comment: This is lacking, as some languages have a dedicated dual form. For proper handling of plurals in JS, see nopilots/wordpress-develop#20491.

Investigate, implement if appropriate, and include tests.
If no longer relevant, remove it and explain why.</issue_description>

Comments on the Issue (you are @claude[agent] in this section)

Replace the workaround using hardcoded singular/plural forms with proper
WordPress i18n pluralization. The widget now uses _n_noop() in PHP to pass
untranslated strings to JavaScript, which then uses wp.i18n._n() and
wp.i18n.sprintf() for proper pluralization that supports languages with
dual and multiple plural forms.

Changes:
- PHP: Replace hardcoded _n() calls with _n_noop() to export proper plural forms
- JS: Use wp.i18n._n() and wp.i18n.sprintf() for dynamic pluralization
- Tests: Add comprehensive test to verify proper plural form structure

Agent-Logs-Url: https://github.com/nopilots/wordpress-develop/sessions/a6fc7dd0-f85e-4eed-8f99-7ef7f607e34c

Co-authored-by: josephfusco <6676674+josephfusco@users.noreply.github.com>
@Claude Claude AI requested a review from josephfusco April 15, 2026 06:34
@github-actions github-actions Bot added the safety:halt Circuit breaker active label Apr 15, 2026
@josephfusco
Copy link
Copy Markdown

Closing as part of a system simplification.

The pipeline has been stuck for 7 days with no merges. Root causes identified: revision chains, work-generation workflows, test-failures bypassing approval gates, and workflow self-modification noise.

A clean-slate cleanup is in progress. The system will resume with a tighter, simpler workflow set. Fresh PRs from agents will flow through the corrected pipeline.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safety:halt Circuit breaker active

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Address TODO in wp-includes/widgets/class-wp-widget-custom-html.php: This is lacking, as some languages have a dedicated dual form. For proper handli

2 participants