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

Comments

martin107 created an issue. See original summary.

martin107’s picture

martin107 commented
Component: base system » documentation
sahil432’s picture

sahil432 commented
Assigned: Unassigned » sahil432
sahil432’s picture

sahil432 commented
StatusFileSize
new 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 commented
Status: Active » Needs review
martin107’s picture

martin107 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
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.