Recently there was a commit (no corresponding drupal.org issue) to remove the previous database API maintainer from MAINTAINERS.txt.

Without getting into the Drupal drama and politics (plentiful though they may be...) there was a problem with this commit. It didn't use the standard method for indicating that a component in MAINTAINERS.txt no longer has any maintainers. Instead, it basically left the "Database API" as if it were only a parent category for the three database drivers that ship with Drupal core.

This is not correct, since most of the database API code isn't driver-specific. It could definitely still use a maintainer.

Comments

David_Rothstein created an issue. See original summary.

David_Rothstein’s picture

Status: Active » Needs review
StatusFileSize
new322 bytes
new350 bytes

Here are patches for Drupal 7 and 8.

xjm’s picture

Status: Needs review » Reviewed & tested by the community

Thanks @David_Rothstein.

alexpott’s picture

Version: 8.4.x-dev » 7.x-dev
Issue tags: -Needs backport to D7

Committed and pushed 5abe312c94 to 8.5.x and e5713b3ded to 8.4.x. Thanks!

Back to 7.x for commit there.

  • alexpott committed 5abe312 on 8.5.x
    Issue #2915900 by David_Rothstein: MAINTAINERS.txt doesn't indicate that...

  • alexpott committed e5713b3 on 8.4.x
    Issue #2915900 by David_Rothstein: MAINTAINERS.txt doesn't indicate that...
David_Rothstein’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 7.x.

  • David_Rothstein committed 01ff63a on 7.x
    Issue #2915900 by David_Rothstein, alexpott: MAINTAINERS.txt doesn't...

Status: Fixed » Closed (fixed)

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