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.
Comment Errors
"SiteBrandingConvertedIntoBlockUpdateTest.php" was based on a similar update test but unfortunately a number of comments in the new file were not updated and have incorrect references.
Correct comments
Provide a patch to update the comments.
Comments
Comment #2
dcrocks CreditAttribution: dcrocks as a volunteer commentedHere is a quick patch. Are there other small followups that could be added here?
Comment #3
dcrocks CreditAttribution: dcrocks as a volunteer commentedduh
Comment #4
dcrocks CreditAttribution: dcrocks as a volunteer commentedAdded one more small change.
Comment #5
RainbowArrayWe may want input from Win and/or manual testing on the upgrade path. Since Wim suggested this change, should be fine, but it would be great to have sign-off from him.
Comment #6
dcrocks CreditAttribution: dcrocks as a volunteer commentedSince 8.0 has a RC can we still fix the cache setting issue or just the documentation error?
Comment #7
joelpittetMostly documentation but but the patch removes something small in the update.
Comment #8
xjmThanks for the patch!
This patch seems to be several changes, some of which are rc eligible as docs-only changes, and one of which is not clear to me.
Can we split it into two issues?
One line summaries need to be one line, with additional information on additional paragraphs. Reference: https://www.drupal.org/node/1354#drupal
Comment #9
dcrocks CreditAttribution: dcrocks as a volunteer commentedAnother patch did part of what this patch was supposed to so restarted with a new patch. Also split so this has only the documentation fix.
Comment #10
joelpittet'site variables' I think should be 'page variables' as they were really only available at the page level? Could be wrong.
Comment #11
dcrocks CreditAttribution: dcrocks as a volunteer commentedCreated a 2nd followup, #2614506: Followup to Use a Branding Block cache fix - 2005546 and cleaned some minor IS problems.
Comment #12
dcrocks CreditAttribution: dcrocks as a volunteer commentedHow about both.
ps. @alexpott Do you know where the cache line was fixed or the @file doc fix was done?
Comment #13
Wim LeersI don't think I'm necessary here.
Comment #14
dcrocks CreditAttribution: dcrocks as a volunteer commentedThis is do trivial. Could someone RTBC it?
Comment #15
joelpittetYup thanks for the bump.
Comment #16
lauriiiRTBC++
Comment #17
xjmLet's actually just remove this. We don't need to link the issue that added code since that can be discovered via git blame. If there is some specific information from the issue that is relevant we should include it in the docs, but I don't think that's the case here.
Thanks!
Comment #18
dcrocks CreditAttribution: dcrocks as a volunteer commentedMade change.
Comment #19
joelpittetGood call, thanks @xjm and @dcrocks
Comment #20
alexpottCommitted ff12935 and pushed to 80.x and 8.1.x. Thanks!