Remove deprecated methods ::enable(), ::disable() and isEnabled() in SessionManager which was deprecated in #2372389: Expose session handler in container and all uses were removed in #2426031: Remove deprecated uses of SessionManager::isEnabled(), SessionManager::enable() and SessionManager::disable().

CommentFileSizeAuthor
#3 remove_deprecated-2429103-3.patch2.5 KBcpj
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

almaudoh’s picture

Issue summary: View changes
cpj’s picture

Title: Remove deprecated methods SessionHandler::enable(), SessionHandler::disable() and SessionHandler::isEnabled() » Remove deprecated methods SessionManager::enable(), SessionManager::disable() and SessionManager::isEnabled()
cpj’s picture

cpj’s picture

Status: Active » Needs review
cpj’s picture

Issue summary: View changes
almaudoh’s picture

Status: Needs review » Reviewed & tested by the community

Thanks @cpj. RTBC from me.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Methods that are already deprecated in 8.0.x are permitted to be removed under the beta policy. Committed f029fbb and pushed to 8.0.x. Thanks!

  • alexpott committed f029fbb on 8.0.x
    Issue #2429103 by cpj: Remove deprecated methods SessionManager::enable...

Status: Fixed » Closed (fixed)

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