Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
+++ b/core/modules/trash/src/Controller/TrashDeleteController.php
@@ -0,0 +1,126 @@
+ return $this->redirect($this->getRedirectUrl($entity)
+ ->getRouteName(), $this->getRedirectUrl($entity)->getRouteParameters());
Comment | File | Size | Author |
---|---|---|---|
#11 | formatting-issue-2799875-11.patch | 618 bytes | sneha_surve |
#10 | 2799875-10.patch | 643 bytes | naveenvalecha |
Comments
Comment #2
naveenvalechaComment #3
naveenvalechaComment #4
sneha_surve CreditAttribution: sneha_surve at Iksula commentedComment #5
sneha_surve CreditAttribution: sneha_surve at Iksula commentedFormatting changed.
Thanks!
Comment #6
timmillwoodI'm not sure if this was the intended fix for the strange formatting. My thought was to set
getRedirectUrl
to a variable, then use that, rather than calling it twice.Comment #7
naveenvalecha#6 +1
Comment #8
sneha_surve CreditAttribution: sneha_surve at Iksula commentedchanges made as said in #6
Comment #9
timmillwoodWhy is the variable set to high up in the file? Wouldn't it be better on the line above where it's used?
Comment #10
naveenvalechaComment #11
sneha_surve CreditAttribution: sneha_surve at Iksula commentedchanges done as said in #9.
Thanks! :)
Comment #13
timmillwoodThanks!