Skip to content

refactor: DRY page layout wrapper in upses action routing #15

@somethingwithproof

Description

@somethingwithproof

Problem

upses.php repeats the same page wrapper sequence (top_header(), renderer, bottom_footer()) in multiple action branches.

Proposed change

  • introduce a shared UI layout wrapper helper
  • route edit/list rendering branches through the helper
  • keep behavior unchanged while reducing duplicated wrapper logic

Tests

  • add standalone regression test for wrapper call ordering and callback execution
  • include a routing assertion that action branches use the shared helper
  • run syntax checks for touched files

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions