API page: http://api.drupal.org/api/drupal/includes--database--database.inc/functi...

Describe the problem you have found:

Please fix the link: instead of:

See http://www.php.net/PDOStatement-fetch
-- there should be --
See http://www.php.net/manual/en/pdostatement.fetch.php

CommentFileSizeAuthor
#2 fixed_link-1122880-2.patch1.16 KBjn2
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Title: Documentation problem with DatabaseConnection::defaultOptions » DatabaseConnection::defaultOptions has incorrect php.net link
Version: 7.x-dev » 8.x-dev

Actually, please use http://php.net/manual/pdostatement.fetch.php which will redirect to a locale-appropriate page.

Also, needs to be fixed in 8.x first, then ported to 7.x

Thanks for catching this problem and pointing out the correct link!

jn2’s picture

jn2 CreditAttribution: jn2 commented
Status: Active » Needs review
FileSize
fixed_link-1122880-2.patch1.16 KB

Here's a patch.

Dries’s picture

Dries CreditAttribution: Dries commented
Status: Needs review » Fixed

Good catch. Committed to 7.x and 8.x. Thanks.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented

Go Nancy! Thanks jn2...

Status: Fixed » Closed (fixed)

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