Closed (fixed)
Project:
Drupal core
Version:
8.3.x-dev
Component:
documentation
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
29 Aug 2016 at 05:32 UTC
Updated:
8 Nov 2016 at 18:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
shashikant_chauhan commentedreplaced the DatabaseConnection_pgsql->queryTableInformation with \Drupal\Core\Database\Driver\pgsql\Schema::queryTableInformation(). Adding patch for it.
Comment #3
daffie commentedLooks good to me.
Comment #4
xjmThanks for working on this!
There's another outdated reference in this pattern in
Schema.php:(The last three references are in a Drupal 7 fixture and so do not need to be changed.)
Comment #5
alexpottSince we've got the @see we don't to use the fully qualified class name because it is redundant. So changing this to
Schema::queryTableInformation().Comment #6
shashikant_chauhan commentedadding updated patch as suggested.
Comment #7
shashikant_chauhan commentedComment #8
daffie commentedI think that this is how @xjm and @alexpott would like to have it.
Comment #9
alexpottSince this is docs it can go in both 8.3.x and 8.2.x.
Committed and pushed 178cbb9 to 8.3.x and a51cf37 to 8.2.x. Thanks!