Make book navigation block cacheable

Created on 5 May 2015, over 9 years ago
Updated 21 July 2023, over 1 year ago

Problem/Motivation

From BookNavigationBlock:

  /**
   * {@inheritdoc}
   *
   * @todo Make cacheable as part of https://drupal.org/node/1805054
   */
  public function getCacheMaxAge() {
    return 0;
  }

Proposed resolution

Once #1805054: Cache localized, access filtered, URL resolved, and rendered menu trees β†’ lands, doing this becomes possible.

But, it doesn't make sense to fix it as part of that issue, because it requires updating quite a (non-public) code in BookManager. Sadly, BookManager basically reimplements a lot of the menu tree building/manipulating handling. If it'd use the menu tree building instead, then we'd get it for free.

Remaining tasks

See #4.

User interface changes

None.

API changes

TBD

πŸ“Œ Task
Status

Postponed

Version

11.0 πŸ”₯

Component
BookΒ  β†’

Last updated 1 day ago

  • Maintained by
  • πŸ‡ΊπŸ‡ΈUnited States @pwolanin
Created by

πŸ‡§πŸ‡ͺBelgium wim leers Ghent πŸ‡§πŸ‡ͺπŸ‡ͺπŸ‡Ί

Live updates comments and jobs are added and updated live.
  • Performance

    It affects performance. It is often combined with the Needs profiling tag.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024