Follow-up to clean-up #3072431: \Drupal database use dependency injection instead

CommentFileSizeAuthor
#4 3072440-4.patch2.11 KBandypost
#2 examples-d9-compatibility-3072440-1.patch4.08 KBElena Yurchenko

Comments

Olena Yurchenko created an issue. See original summary.

Elena Yurchenko’s picture

Elena Yurchenko commented
Status: Active » Needs review
StatusFileSize
new examples-d9-compatibility-3072440-1.patch4.08 KB
Elena Yurchenko’s picture

Elena Yurchenko commented
andypost’s picture

andypost
he/him
Primary language Russian
commented
Title: Fix call to deprecated function db_query() » Follow-up for Fix call to deprecated function db_query()
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new 3072440-4.patch2.11 KB

Still makes sense to get rid of useless intermediate variables

  • valthebald committed 755d40f on 8.x-1.x 
    Issue #3072440 by Olena Yurchenko, andypost: Follow-up for Fix call to...
valthebald’s picture

valthebald
Location Sofia
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x-1.x, thank you all!

Status: Fixed » Closed (fixed)

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

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented
Issue tags: +Drupal 9 compatibility

Per a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing tag cleanup here based on that discussion.