Problem/Motivation

StatementInterface::fetch() return typehint should also indicate false as a possible type, since PDO::fetch() can return false.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3312439

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

mondrake created an issue. See original summary.

mondrake opened merge request !2825

mondrake’s picture

mondrake
Location 🇮🇹
commented
Status: Active » Needs review
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented

Minor suggestion while we are here.

mondrake’s picture

mondrake
Location 🇮🇹
commented

Merged, thx

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Version: 10.1.x-dev » 9.5.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed dc3d568325 to 10.1.x and 130e46aab0 to 10.0.x and ad467d082c to 9.5.x. Thanks!

Backported to 9.5.x as this is a docs fix.

  • alexpott committed dc3d568 on 10.1.x 
    Issue #3312439 by mondrake: StatementInterface::fetch() return typehint...

  • alexpott committed 130e46a on 10.0.x 
    Issue #3312439 by mondrake: StatementInterface::fetch() return typehint...

  • alexpott committed ad467d0 on 9.5.x 
    Issue #3312439 by mondrake: StatementInterface::fetch() return typehint...

mondrake closed merge request !2825

Status: Fixed » Closed (fixed)

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