Modernization of Test Plan Management (lib/plan/planView.php) — GitHub issue #576.

The legacy Smarty screen is replaced by a standalone Dashio page (gui/templates/plans/planView.html) backed by a plain-PHP REST BFF (api/plans/index.php), following the same pattern as the other modernized screens. The aside menu entry Test Plan → Test Plan Management now points to the new HTML screen.

Feature	Legacy behavior	Modern implementation
List test plans of current project	mgt_testplan_create required	GET /api/plans/?tproject_id=N (403 otherwise)
Test case / build counts per plan	columns with tooltips	same counts + tooltips from BFF
Platforms column	only drawn when project has platforms	dynamic column (CSS class toggle)
Active/Inactive toggle per plan	do_action=setActive/setInactive	PUT /api/plans/{id}/active
Bulk Activate / Inactivate via checkboxes	setActiveBulk/setInactiveBulk	POST /api/plans/bulk-active
Delete plan (cascade)	do_action=do_delete + audit event	DELETE /api/plans/{id} + identical audit event
Create / Edit / Export / Import / Assign roles / Execute	links to their own controllers	served root-absolute by BFF actions map
Per-row "assign roles" icon	shown only with testplan_user_role_assignment	same right resolution: tplan role → tproject role → global role
Custom field columns	scaffolded in controller but dead code (getCustomFieldsValues undefined)	dynamic CF columns from cfield_testplan_design_values — refs #584

Delete confirmation modal (Bootstrap modal, cancel/confirm):

Custom field columns (refs #584):

Romanian locale (client-side TLi18n, keys pv.* in all 10 bundles):

• GET /?tproject_id=N — plans + counts + rights + action URLs
• PUT /{id}/active — body {tproject_id, active}
• POST /bulk-active — body {tproject_id, ids[], active}
• DELETE /{id}?tproject_id=N

All mutations re-check mgt_testplan_create server-side and verify that the target plan belongs to the context project.

1. 404 on Create/Edit/etc. — BFF returned relative legacy URLs which resolved against /gui/templates/plans/; now root-absolute.
2. Table never rendered — unquoted jQuery attribute selectors (th[data-i18n=pv.tcQty]) threw a syntax error before render().
3. DataTables column-count alert — hidden Platforms <th> vs 8-cell rows; visibility now handled by a stable CSS class with cells always populated.

Full suite recorded in tmp/TLU_Test_Cases.md, suite 54 (16 checks PASS): buttons, toggles, bulk ops, delete modal cancel/confirm, permission paths (admin vs guest 403), i18n switch, Event Viewer clean.

The "Create" and "Edit" actions were the last legacy leaves of this screen: they used to redirect to lib/plan/planEdit.php. Both are now Bootstrap modals built into planView.html, backed by dedicated BFF endpoints:

• POST /api/plans/ — create plan {tproject_id, name, notes, active, is_public}; gates on mgt_testplan_create, rejects duplicate names (422 DUPLICATE_NAME), emits the same CREATED audit event as legacy do_create.
• GET /api/plans/{id}?tproject_id=N — single plan payload for modal pre-fill.
• PUT /api/plans/{id} — update plan with the same name-uniqueness check and SAVE audit event as legacy do_update.
• Private plans get the creator's effective role assigned (same as legacy).

Modal validation on the client mirrors the legacy form: name required, duplicate name surfaced inline, Active/Public toggles. On success the DataTable refreshes and a toast confirms Test plan created / Test plan updated.

i18n keys pv.modal.* / pv.msg.created|updated|duplicateName are present in all 10 locale bundles. Test suite 49 (12 checks PASS) covers create/edit validation, duplicate-name handling, and Event Viewer cleanliness — zero ERROR/WARNING.

Also fixed while testing: issue #766 — the Export/Import/Roles/Execute row links had an empty trailing tplan_id because actionUrl() used indexOf('=') instead of a last-char check.

The Create Test Plan modal now restores the legacy "Create from an existing Test Plan?" capability (gui/templates/dashio/plan/inc_controls_planEdit.tpl) that the modern create flow had dropped:

• A Create from an existing test plan dropdown lists every plan of the current test project (plus the default "- no copy, create an empty test plan -").
• Picking a source reveals the full legacy option set: Copy user roles, Copy attachments, Copy test cases (radio Latest version / Current version), Copy test case priorities, Copy builds, Copy tester assignments, Copy platform links (checked & disabled — forced when test cases are copied, exactly like testplan::copy_as()), and Copy milestones.
• Unchecking Copy test cases hides the version/priority sub-options and re-enables the platform-links box; unchecking Copy builds hides Copy tester assignments.
• The copy block is shown only in create mode (editing an existing plan doesn't offer it), matching the legacy screen.

Server side, POST /api/plans/ now accepts copy_from_tplan_id, copy_options{...}, copy_assigned_to and tcversion_type and delegates to testplan::copy_as() (same logic as the full-screen editor api/planedit). It validates that the source plan exists (404 COPY_SOURCE_NOT_FOUND) and belongs to the same project (400 COPY_SOURCE_MISMATCH).

All labels reuse the existing pe.copy.* / pe.copySection / pe.copyNone i18n keys, present in all 10 locale bundles.

copy_builds() (lib/functions/testplan.class.php) inserted a non-existent builds.testplan_id column, so any plan copy with the default Copy builds option fataled with 1054 Unknown column 'testplan_id' — this also broke the #1882 full-screen create/copy and the legacy project-copy path (testproject::copy_as → testplan::copy_as). It now writes the project-scoped builds.testproject_id and, when a build with the same name already exists in the destination project (same-project plan copy), reuses that row instead of failing on the (testproject_id,name) unique key.

The legacy edit form (gui/templates/dashio/plan/planEdit.tpl:123-130) showed the plan's api_key read-only — only when the key was non-empty ({if $gui->api_key != ''}). The modern inline Edit Test Plan modal (gui/templates/plans/planView.html) had dropped it, so the key that uniquely identifies a test plan for the REST/XML-RPC API could no longer be read or copied from the UI.

Restored, mirroring the project API key row in projectsView.html:

• GET /api/plans/{id} now returns api_key alongside id/name/notes/active/is_public.
• The edit modal renders a read-only monospace row (#apiKeyVal, role=textbox aria-readonly=true, user-select:all) with a Copy button (navigator.clipboard with an execCommand fallback) and a short hint.
• The row is shown only in edit mode and only when the key is non-empty, exactly like legacy; the create modal never shows it.
• Switching between plans (or edit → create → edit) resets the value and the copy button label — the read-only <div> is not a form control, so it is cleared explicitly on every modal open.

i18n keys pv.apiKey, pv.apiKeyCopy, pv.apiKeyCopied, pv.apiKeyCopyFailed, pv.apiKeyHint exist in all 10 locale bundles (bash ai/verify_i18n_coverage.sh passes). The key is never editable: it is generated server-side by testplan::create().

The legacy edit form (lib/plan/planEdit.php do_viewAttachments/do_fileHandling, gui/templates/dashio/plan/planEdit.tpl:192-196 → attachments.inc.tpl) let a test plan carry supporting artifacts: an upload form (file + optional title) and a list of the existing attachments with download and delete. The modern inline Edit Test Plan modal (gui/templates/plans/planView.html) had dropped that section — only the full-screen editor (planEdit.html, Refs #1882) exposed it.

The modal now renders the full legacy capability (edit mode only — a plan must exist before it can own attachments), backed by the same api/attachments BFF the full-screen editor uses (action=list|upload|delete|download, fk_table=testplans), so no new BFF routes were needed and the DB format stays legacy-compatible:

• List — title / filename / size / date columns with a Download link and a per-row Delete button (confirm before delete), empty-state "No attachments yet.", max-file-size hint from the BFF.
• Upload — optional Title + File, single-file upload via POST /api/attachments (uploadedFile[] FormData), inline feedback messages.
• State is reset on every modal open (resetPlanAttachments), so a previous plan's list/uploads cannot leak into the next plan; the section is hidden in create mode.

All labels reuse the existing pe.att* / pe.msg.upload*|delete* i18n keys, already present in all 10 locale bundles (bash ai/verify_i18n_coverage.sh passes).