Closed (fixed)
Project:
Drupal core
Version:
9.2.x-dev
Component:
database system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
31 May 2021 at 12:25 UTC
Updated:
10 Jul 2021 at 23:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
mondrakeComment #3
andypostLooks much better
Comment #4
daffie commentedTwo nitpicks:
Not outlined correctly.
Double space.
The documentation text is +1 for me.
Comment #5
mondrakeComment #6
sudiptadas19 commented@daffie
Added the changes as suggested. Please review it.
Comment #7
daffie commented@sudipta: You have fixed #4.2 correctly, only #4.1 still needs to be done.
Comment #8
sudiptadas19 commented@daffie:
You want some text change or how it is, not getting it clearly.
Comment #9
daffie commentedTo be more clear. This text does not need to start on a new line. Look at the other change. "If a SelectInterface object ..." also does not start on a new line.
Comment #10
sudiptadas19 commentedGiven feedback addresses. Please review it.
As it is a small change, not added an interdiff.
Comment #11
andypoststill not aligned properly
Comment #12
mondrakenot a native speaker, but this is incorrect grammar. How about "If a SelectInterface object, it will be treated as a subquery."
Comment #13
longwaveHow about: The base table name or subquery for this query, used in the FROM clause. If a string, the table specified will also be used as the "base" table for query_alter hook implementations.
How about just: The table name or subquery that is being queried.
Comment #14
guilhermevp commentedMade changes suggested in #13.
Please review.
Comment #15
mondrakeBrilliant!
Comment #16
xjmComment #19
xjmGood find. Committed to9.3.x and cherry-picked to 9.2.x. Thanks everyone!