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
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions