Active
Project:
Drupal core
Version:
main
Component:
other
Priority:
Normal
Category:
Plan
Assigned:
Unassigned
Reporter:
Created:
27 Nov 2019 at 10:02 UTC
Updated:
26 Sep 2025 at 03:15 UTC
Jump to comment: Most recent
Drupal 9 still has a bunch of legacies includes files. See the previous meta #2999721: [META] Deprecate the legacy include files before Drupal 9
Here is the tracking of the deprecation status of the legacy include files in the core modules and core/includes directory:
bootstrap.inc
Done:
common.inc
module.inc
Done:
theme.inc
form.inc
errors.inc
install.core.inc
install.inc
batch.inc
theme.maintenance.inc
update.inc
utility.inc
Comments
Comment #2
volegerRemove already closed issue
Comment #3
volegerComment #4
gábor hojtsySwap parent and related issue to make it better (IMHO).
Comment #5
xjmI'm not sure this should be a child of the D10 meta. We will not block the release of Drupal 10 or change its schedule based on whether or not this is done.
Comment #6
xjmAlso, many of these issues should be retitled. The objective of the issue isn't to deprecate something -- it's to provide a modern replacement. So the title (and IS) of each should describe the replacement that's being proposed (new service, whatever).
Comment #7
volegerDefinitely this issue have not be a child of D10 meta. But this things is nice to have. We have great initiative in modernizing Batch API with a Batch service, also thing like #3038513: Move drupal_generate_test_ua() into the test system which is blocking us to easily replace TestKernel with DrupalKernel, and so on...
Thanks for advise regarding re-titling.
Comment #8
gábor hojtsyRetitled, removed parent.
Comment #9
voleger#2908079: Move some of the bootstrap.inc PHP-related constants to \Drupal and deprecate the old versions - Done
Comment #11
andypostLanded #3151118: Include bootstrap.inc using composer
So related issues may need re-roll
Comment #12
andypostCommited #2908886: Split off schema management out of schema.inc
Comment #14
sutharsan commentedLocale contains a bunch on include files and plenty of procedural code that deserves an OO upgrade too. Not sure whether use this a meta, or create a child meta dedicated to modernizing Locale module. But let me start by summarizing what I found so far:
Include files
OO-ify issues
Other locale improvement issues
Comment #15
gábor hojtsy@Sutharsan: probably locale module is the biggest offender of all that is left in terms of random global functions and twisted include files. I had a big role in it ending it up that way, so I can hopefully use the strong words :D IMHO a separate META issue would be best, since it is otherwise pretty well contained and does not have many side effects other than the installer integration.
Comment #16
andypost++ to separate meta for locale, I'm willing to help modernize it
Comment #17
sutharsan commented@Gábor, that makes two that played a big role ;) But on the positive side, I can be part of the solution too. Andpost, you offer is gladly accepted, I know my way around this (complex) code, but I should not do it alone.
I will make the issue later today.
Comment #18
sutharsan commentedCreated #3215707: [META] Modernize Locale module
Comment #19
andypostschema.incfinishedComment #20
volegerComment #21
volegerComment #22
tr commentedAdded #2010380: Deprecate module_load_install() and replace it with ModuleHandler::loadInstall to the issue summary under module.inc.
Comment #23
volegerOne more include is also done #2999961: [META] Deprecate contents of menu.inc
Comment #24
andypostmenu.incis done and should be removed in d10Comment #25
daffie commentedfile.inc is done!
Comment #26
daffie commentedfile.inc not yet completely done, we still have the function
file_build_uri()to deprecate.Comment #27
andypostAdded new issue to summary for #26 #3223016: Deprecate file_build_uri()
Comment #28
andypostsorter IS a bit also there's
FILE_STATUS_PERMANENTComment #29
andypostAdded #3021833: Move FILE_STATUS_PERMANENT to \Drupal\file\FileInterface
Comment #30
andypostOne more deprecated done #3223016: Deprecate file_build_uri()
Comment #31
daffie commentedfile.inc is done!
Comment #32
andypostone more done - #2347783: Deprecate drupal_get_path() and drupal_get_filename() and replace with ExtensionList::getPath() and ExtensionList::getPathname()
so only 3 leftovers on boostrap.inc
Comment #34
volegerMove #697946: Properly deprecate module_load_include() and move it into \Drupal::moduleHandler() service to done
Comment #35
volegerMove #2010380: Deprecate module_load_install() and replace it with ModuleHandler::loadInstall to done
Comment #36
volegerComment #37
xjmI'm removing the "before Drupal 10" in the issue title. If some of them don't get deprecated before D10, then we make the additions in D10 and deprecate the old APIs for D11 instead. :)
Comment #38
daffie commentedComment #41
andypostAdded to
theme.inc- #3348848: Deprecate theme_get_registry()
- #3348851: Deprecate drupal_theme_rebuild()
Comment #42
andypostAdded 2 remains for
module.incbut last 2 are confusingAlso clean-up stuff removed in D10
Comment #43
andypostComment #44
kim.pepperComment #46
andypostThe
drupal_common_theme()gone via #3488176: Convert system_theme to OOP and handle install time callComment #47
nicxvan commentedGlad I found this! I've been pondering working on some of this too!
Comment #48
nicxvan commentedComment #49
nicxvan commentedComment #50
nicxvan commentedComment #51
nicxvan commentedMaking good progress on theme.inc, down to 5 functions!
Comment #52
nicxvan commented