The documentation of ControllerBase::redirect() is incomplete. Somebody accidentally left a word out.

CommentFileSizeAuthor
#1 2103163-1-controllerbase_redirect_documentation.patch890 bytespfrenssen

Comments

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Active » Needs review
Issue tags: +Novice
StatusFileSize
new 2103163-1-controllerbase_redirect_documentation.patch890 bytes
pfrenssen’s picture

pfrenssen
Location Sofia
commented

Just checked if this needs a reroll, the patch still applies without problems.

pfrenssen’s picture

pfrenssen
Location Sofia
commented
Component: base system » documentation
Issue summary: View changes
jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Thanks! Probably because it was not in the "documentation" component before, no one noticed this issue. The patch looks good.

pfrenssen’s picture

pfrenssen
Location Sofia
commented

Indeed, I figured it could use a little nudge in the right direction :) Thanks a lot for reviewing!

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

1: 2103163-1-controllerbase_redirect_documentation.patch queued for re-testing.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thanks again! Committed to 8.x.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.