Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
#2849745: Replace documentation recommending db_*() wrappers is trying to fix all documentation which references db_*()
functions.
Let's split out the database.api.php
file and fix it, since it is the primary documentation for this API.
Proposed resolution
Fix the documentation in core/lib/Drupal/Core/Database/database.api.php
to exclude the deprecated db_*()
functions.
Also, this is a good documentation-only candidate for backport to 8.6.x, even during alpha.
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#7 | interdiff.txt | 1.36 KB | Mile23 |
#7 | 2987399_7.patch | 9.9 KB | Mile23 |
#5 | interdiff.txt | 1.49 KB | Mile23 |
#5 | 2987399_5.patch | 9.51 KB | Mile23 |
#3 | interdiff.txt | 5.08 KB | Mile23 |
Comments
Comment #2
Mile23This is a straight re-roll of @yogeshmpawar's work in #2849745-21: Replace documentation recommending db_*() wrappers, using only the changes for database.api.php. All credit goes to them.
Comment #3
Mile23Some edits and a few missed replacements.
Comment #4
alexpottahold of
- also ahold as far as I know is not a word.I think
As a note,
is simpler asNote:
- we've used this many times before. Whereas the are there are only 4 in core - including the 2 here.Just looking at https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Database%... - I\m not sure that @code is really meant to be used inline but that's not introduced here.
Comment #5
Mile23Good points. But I feel so provincial now that I tried to Google 'ahold.' :-)
Comment #6
alexpottThere's one more
As a note
Comment #7
Mile23Also found another inline @code.
Comment #8
volegerLooks good
Comment #9
alexpottCommitted and pushed e3d81645d3 to 8.7.x and a13f7aca22 to 8.6.x. Thanks!
As a docs fix backported to 8.6.x