The status message advising to remove the deprecated configuration for the temporary files directory is missing a close code element, resulting in a nice monospace font for the rest of the status page.

It was added here: #3039026: Deprecate file_directory_temp() and move to FileSystem service

CommentFileSizeAuthor
#2 3092595-2.patch786 bytesimclean
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

imclean created an issue. See original summary.

imclean’s picture

imclean CreditAttribution: imclean commented
Status: Active » Needs review
FileSize
3092595-2.patch786 bytes
imclean’s picture

imclean CreditAttribution: imclean commented
Title: Deprecated temp file path config status message missing closing code element » Deprecated temp file path message missing closing code element
kim.pepper’s picture

kim.pepper
Primary language English
Location 🏄‍♂️🇦🇺Sydney, Australia
CreditAttribution: kim.pepper at PreviousNext commented
Status: Needs review » Reviewed & tested by the community

Looks good to me!

  • catch committed c85e3f7 on 9.0.x 
    Issue #3092595 by imclean: Deprecated temp file path message missing...

  • catch committed f61a2d5 on 8.8.x 
    Issue #3092595 by imclean: Deprecated temp file path message missing...
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Good spot. Committed/pushed to 9.0.x, 8.9.x, 8.8.x, thanks!

  • catch committed c429e78 on 8.9.x 
    Issue #3092595 by imclean: Deprecated temp file path message missing...

Status: Fixed » Closed (fixed)

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