[armanist]

Closes #541.

What changed

• The DemoWeb and Toolkit templates redirect back with back() instead of the manual referrer fallback:
  • redirect(get_referrer() ?? base_url()) becomes back() in DemoWeb CommentController (twice) and the Reset middleware, and in Toolkit DatabaseController and the CreateTable middleware.
  • redirectWith(get_referrer() ?? base_url(), $data) becomes back($data) in the DemoWeb Comment and Editor middlewares.
• Added the Changed entry to CHANGELOG.md.

Why

#540 added back() and the back() helper, so generated code no longer needs to repeat the referrer fallback. The generated code also gets the same-origin check of back().

Breaking changes

None. Generated redirects behave as before for a same-origin referrer, and fall back to base_url() for an external one. Modules generated earlier keep their code.

Testing

New tests/Unit/Module/Templates/RedirectTemplatesTest.php, for the DemoWeb and Toolkit templates:

• no template contains get_referrer() ?? base_url()
• the templates redirect with return back(

Both checks fail on the templates before this change.

Notes for reviewers

• Left unchanged on purpose: the redirects to a fixed target (redirect(base_url(true) . '/' . current_lang() . '/signin') and the redirectWith(...) ones), which back() cannot express, and the Guest middleware. Its fallback is base_url(true) . '/' . current_lang(), not base_url(), so back() would change its behavior.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!