Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Meta issue: #2205673: [meta] Remove all @deprecated functions marked "remove before 8.0"
Only the function exists, the usage of the function is already removed.
[ashish@xeon:~/proj/self/d8/data/drupal8]$grep views_get_plugin_definitions -r *
core/modules/views/views.module:function views_get_plugin_definitions() {
[ashish@xeon:~/proj/self/d8/data/drupal8]$
Comment | File | Size | Author |
---|---|---|---|
#6 | remove-views_get_plugin_definitions-2306471-6.patch | 699 bytes | amitgoyal |
Comments
Comment #1
a_thakur CreditAttribution: a_thakur commentedPlease find the attached patch.
Comment #2
dawehnerNice!
Comment #3
KarlKedrovsky CreditAttribution: KarlKedrovsky commentedI looked at the patch and ensured that all of the comments and code for this function were removed and that no other lines were added or removed.
Did a find/grep on the D8 code base to ensure that no other mentions of this function exist.
Checked to ensure that the patch still applies cleanly.
Looks goo to me!
Comment #4
alexpottCan we add this issue to relevant change record. Thanks.
Comment #5
alexpottActually there will not be a change record - the process is to write one against views.
This patch now needs a reroll.
Comment #6
amitgoyal CreditAttribution: amitgoyal commentedRe-rolled.
Comment #7
leeotzu CreditAttribution: leeotzu commentedDid a grep on the D8 code base after applying the rerolled patch. No trace of function views_get_plugin_definitions found in the code base.
Comment #8
alexpottCommitted 2a806cc and pushed to 8.x. Thanks!
Comment #11
a_thakur CreditAttribution: a_thakur commentedComment #12
a_thakur CreditAttribution: a_thakur commentedComment #13
a_thakur CreditAttribution: a_thakur commentedPlease ignore. Accidentally edited the summary.