Skip to content

Fix prepare and bind user-details toggle targets - #890

Merged
darold merged 1 commit into
darold:masterfrom
dajiaohuang:fix/prepare-user-details-20261016
Oct 19, 2026
Merged

darold merged 1 commit into
darold:masterfrom
dajiaohuang:fix/prepare-user-details-20261016

Conversation

@dajiaohuang

Copy link
Copy Markdown
Contributor

Problem

The User(s) involved buttons in the generated Time consuming prepare and Time consuming bind reports cannot open their panels. The buttons target time-consuming-prepare-user-involved-rank-N and time-consuming-bind-user-involved-rank-N, but both panels are emitted with the misspelled ID time-consuming-prepapre-user-involved-rank-N. When both reports contain the same rank, this also duplicates the panel ID.

Change

Correct the two generated IDs to match the existing Bootstrap collapse targets. Add a self-contained synthetic-log CLI-to-HTML regression checking that each button targets its own user-details panel.

Validation

  • Exact base fb5e1bfe80e50180e77795bac49653cc92c6813d: both target assertions fail (1/3 pass); fixed version passes 3/3 on Perl 5.38.2.
  • Perl syntax, POD syntax and whitespace checks pass.
  • The failure is established from actual generated report markup; interactive click acceptance is not claimed.
  • Full existing suite and other Perl versions not run locally (fixed /tmp paths and upstream log fixtures outside the isolated check; JSON::XS absent). No production log or database connection used. Independent of Fix multiline prepare query continuation #888 and Fix prepare and bind sample metadata argument mapping #889.

@darold
darold merged commit 454656e into darold:master Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants