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_plugin_list -r *
core/modules/views/views.module:function views_plugin_list() {
[ashish@xeon:~/proj/self/d8/data/drupal8]$
CommentFileSizeAuthor
#1 remove-views_plugin_list-2306545-1.patch1003 bytesa_thakur

Comments

a_thakur’s picture

a_thakur commented
Status: Active » Needs review
StatusFileSize
new remove-views_plugin_list-2306545-1.patch1003 bytes

Please find the attached patch.

holyfire’s picture

holyfire commented

Reviewing issue...

amunir’s picture

amunir commented
Status: Needs review » Reviewed & tested by the community

looks good and it applies clean-- reviewed while other person was doing at the same time

holyfire’s picture

holyfire commented

Also reviewed... looks good. Patch applied, found no evidence of function in the code.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Project: Drupal core » Views (for Drupal 7)
Version: 8.x-dev » 8.x-3.x-dev
Component: views.module » Code
Status: Reviewed & tested by the community » Active
Issue tags: +Needs change record

Committed 5554273 and pushed to 8.x. Thanks!

andypost’s picture

andypost
he/him
Primary language Russian
commented

Any reason o move the issue to views?

dawehner’s picture

dawehner
Primary language German
commented

Change record, see the tag :)

ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
commented
Status: Active » Needs review

I have posted a draft change record

dorficus’s picture

dorficus commented
Status: Needs review » Reviewed & tested by the community

Looks good!

ianthomas_uk’s picture

ianthomas_uk
he/him
Primary language English
Location Brighton, UK
commented
Status: Reviewed & tested by the community » Fixed
Issue tags: -Needs change record

Published the change record. Given it was just a change record review, I think I'm allowed to mark this fixed.

Status: Fixed » Closed (fixed)

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