Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
At DynamicPageCacheSubscriber.php, there is a long phrase wrapped in parenthesis which does not need them, plus a missing parameter name.
Comment | File | Size | Author |
---|---|---|---|
#2 | fix_typos_at-2744379-2.patch | 1.88 KB | juampynr |
Comments
Comment #2
juampynr CreditAttribution: juampynr at Lullabot commentedHere is a patch that fixes these.
Comment #3
mahtoranjeet CreditAttribution: mahtoranjeet as a volunteer and at Srijan | A Material+ Company commentedReviewed the code and tested it.
Now Parenthesis is removed that was wrapping long phase and
Added the missing parameter
Comment #4
Wim LeersThanks!
(Also: great to know that you actually read the documentation, @juampynr — hope you liked it :))
Comment #5
Wim LeersComment #6
alexpottCommitted 7e2925c and pushed to 8.1.x and 8.2.x. Thanks!
Comment #9
juampynr CreditAttribution: juampynr at Lullabot commentedIt has been the definitive way for me to understand how this works. Thanks.
Comment #10
Wim Leers#9: yay :)