Just spinning off an issue from #2124069: Convert schema.inc to the update.update_hook_registry service (UpdateHookRegistry)

two functions that have been refactored away are still mentioned in the comments.

CommentFileSizeAuthor
#4 removedoc-2908952-4.patch1.46 KBsahil432
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

martin107 created an issue. See original summary.

martin107’s picture

martin107 CreditAttribution: martin107 as a volunteer commented
Component: base system » documentation
sahil432’s picture

sahil432 CreditAttribution: sahil432 as a volunteer and at Publicis Sapient commented
Assigned: Unassigned » sahil432
sahil432’s picture

sahil432 CreditAttribution: sahil432 as a volunteer and at Publicis Sapient commented
FileSize
removedoc-2908952-4.patch1.46 KB

Removing drupal_get_complete_schema() and drupal_get_schema() from
Drupal/Core/Config/ConfigImporter.php
Drupal/Core/Database/Driver/sqlite/Schema.php

sahil432’s picture

sahil432 CreditAttribution: sahil432 as a volunteer and at Publicis Sapient commented
Status: Active » Needs review
martin107’s picture

martin107 CreditAttribution: martin107 as a volunteer commented
Assigned: sahil432 » Unassigned
Status: Needs review » Reviewed & tested by the community

Thank you sahil432

This looks good to me.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.6.x and cherry-picked to 8.5.x. Thanks!

  • catch committed c79d74a on 8.6.x 
    Issue #2908952 by sahil432, martin107: drupal_get_complete_schema() and...

  • catch committed e6be030 on 8.5.x 
    Issue #2908952 by sahil432, martin107: drupal_get_complete_schema() and...

Status: Fixed » Closed (fixed)

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