Updated: Comment #1

Problem/Motivation

In the docs it is is not clear that the last two parameters of db_select() are optional.

Proposed resolution

Update docblock to include (optional).

Remaining tasks

Review patch.

User interface changes

API changes

Comments

elijah lynn’s picture

elijah lynn’s picture

Issue summary: View changes
Status: Active » Needs review
filijonka’s picture

StatusFileSize
new773 bytes

clean reroll

filijonka’s picture

Status: Needs review » Reviewed & tested by the community

well was reviewing pathc in #1 and was going to set it green when realising it needed a reroll. The reroll was no changes so sets this to green.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 4b7a940 and pushed to 8.0.x. Thanks!

alexpott’s picture

This issue is a prioritized change (documentation) as per https://www.drupal.org/core/beta-changes and it's benefits outweigh any disruption.

  • alexpott committed 4b7a940 on 8.0.x
    Issue #2167379 by Elijah Lynn, filijonka: db_select docblock should list...

Status: Fixed » Closed (fixed)

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