Forums Feature Requests Thread

[Enhancement] Cross-cutting infra: Orphaned partial: admin-branding-tabs.php (replaced by inline brn-tabs nav in /admin/branding)

Patrick Bass · Jun 6 · 10 · 1 Locked
[Normal Priority] [Feature Enhancement] [Deployed to Production]
🚀 OP Jun 6, 2026 8:05pm

Area: Cross-cutting infra (audit p14) · Surface: platform/templates/partials/admin-branding-tabs.php · Dimension: deadcode · Severity: minor

templates/partials/admin-branding-tabs.php is a tab-strip partial for /admin/branding that no template includes. /admin/branding/index.php renders the tab strip inline (brn-tabs nav, line 113). The only reference is a comment in the (also-orphaned) admin-config-tabs.php.

Evidence

`$ grep -rn "admin-branding-tabs" src/ templates/ public/ bin/ --include=*.php | grep -v 'templates/partials/admin-branding-tabs.php:'` → only one hit: templates/partials/admin-config-tabs.php:4 (' * Mirrors templates/partials/admin-branding-tabs.php.') — a cross-docblock mention, not an include. /admin/branding (templates/admin/branding/index.php:113) renders its tabs inline with `<nav class="brn-tabs">` over $brandingTabs.

Suggested fix. Delete templates/partials/admin-branding-tabs.php (and admin-config-tabs.php together, since they only cross-reference each other).

Filed by the automated tenant-app audit and adversarially evidence-verified. Status: verified. Open — not yet actioned.


Patrick Bass
@mobieus

🚀 Jun 7, 2026 10:24am

Resolved and deployed. Commit ca230723ea.

templates/partials/admin-branding-tabs.php deleted — /admin/branding builds its tabs inline; this partial was unreferenced.

Closed as: shipped.


Patrick Bass
@mobieus

Log in or register to reply to this thread.