Meta issue: #2205673: [meta] Remove all @deprecated functions marked "remove before 8.0"

CommentFileSizeAuthor
#1 remove-views_get_view-2315695-1.patch709 bytesa_thakur
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

a_thakur’s picture

a_thakur CreditAttribution: a_thakur commented
Status: Active » Needs review
FileSize
remove-views_get_view-2315695-1.patch709 bytes

Please find the attached patch which removes the function views_get_view()

[ashish@xeon:~/proj/self/d8/data/drupal8]$grep views_get_view -r *
core/modules/views/views.module:function views_get_views_as_options($views_only = FALSE, $filter = 'all', $exclude_view = NULL, $optgroup = FALSE, $sort = FALSE) {
core/modules/views/views.module:function views_get_view_result($name, $display_id = NULL) {
dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

Nice. Note: We should create a change record for views 8.x-3.x

leeotzu’s picture

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

Grep on the D8 code base after applying the patch. No trace of function views_get_view() found in the code base.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
CreditAttribution: jhodgdon commented
Version: 8.x-dev » 8.0.x-dev
alexpott’s picture

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

Committed 4c285fe and pushed to 8.0.x. Thanks!

Moved to views for change record.

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews as a volunteer and at City of Oaks Design commented
Project: Views (for Drupal 7) » Drupal core
Version: 8.x-3.x-dev » 8.7.x-dev
Component: Code » views.module

For more information as to why this issue was moved to the Drupal core project, please see issue #3030347: Plan to clean process issue queue

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews as a volunteer and at City of Oaks Design commented
Project: Drupal core » Views (for Drupal 7)
Version: 8.7.x-dev » 7.x-3.x-dev
Component: views.module » Code
Status: Active » Closed (outdated)

Moving back to the contributed Views issue queue and closing as outdated per https://www.drupal.org/project/views/issues/3030347#comment-13023447