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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pfrenssen’s picture

Status: Active » Needs review
Issue tags: +Novice
FileSize
890 bytes
pfrenssen’s picture

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

pfrenssen’s picture

Component: base system » documentation
Issue summary: View changes
jhodgdon’s picture

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

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

Xano’s picture

jhodgdon’s picture

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.