In drupal 8 we should not use the depreciated db_queries.

CommentFileSizeAuthor
#4 removed_depreciated-2869345-3.png67.89 KBamit.drupal
#2 removed_depreciated-2869345-2.patch620 bytesMunavijayalakshmi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Munavijayalakshmi created an issue. See original summary.

Munavijayalakshmi’s picture

Munavijayalakshmi CreditAttribution: Munavijayalakshmi at Valuebound commented
Assigned: Munavijayalakshmi » Unassigned
Status: Active » Needs review
FileSize
removed_depreciated-2869345-2.patch620 bytes
amit.drupal’s picture

amit.drupal CreditAttribution: amit.drupal as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented

Documentation for how/when to use this new style?

https://api.drupal.org/api/drupal/core!includes!database.inc/function/db...

Get a database connection injected into your service from the container and call query() on it.

For example, $injected_database->query($query, $args, $options);

we are going to change this again and inject the service instead to use \Drupal::database()

That makes me wonder if this is worth it or if it would better to wait until move this into a service and do the change there?

amit.drupal’s picture

amit.drupal CreditAttribution: amit.drupal as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented
Status: Needs review » Reviewed & tested by the community
FileSize
removed_depreciated-2869345-3.png67.89 KB

  • Nitebreed committed ca52b86 on 8.x-1.x 
    Issue #2869345 by Munavijayalakshmi,amit.drupal : Removed depreciated...
Nitebreed’s picture

Nitebreed
Primary language Dutch
CreditAttribution: Nitebreed at Synetic commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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