Problem/Motivation

During porting to 8.x, much of the functionality that has been deprecated or outdated should be removed.

Proposed resolution

Remove them.

CommentFileSizeAuthor
#3 remove_deprecated_and-2641250-3.patch1.71 KBhussainweb
#2 remove_deprecated_and-2641250-2.patch1.31 KBhussainweb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented
Status: Active » Needs review
FileSize
remove_deprecated_and-2641250-2.patch1.31 KB

Removing token_type_load and token_element_validate_token_context for now.

hussainweb’s picture

hussainweb
Primary language English
CreditAttribution: hussainweb at Axelerant commented
FileSize
remove_deprecated_and-2641250-3.patch1.71 KB

Also removing token_admin_menu_output_alter as per comment in #2640612-14: Move token information functions to a service.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Reviewed & tested by the community

Looks good to me. Lets wait a bit before commit in case Dave disagrees with removing those but I don't think they should be kept.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

And they're gone. We can easily add something back if we need it again.

  • Berdir committed f6db793 on 8.x-1.x authored by hussainweb 
    Issue #2641250 by hussainweb: Remove deprecated and outdated functions

Status: Fixed » Closed (fixed)

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