Problem/Motivation

CONTRIBUTED PROJECTS
--------------------------------------------------------------------------------
Cache Tools 8.x-1.5
Scanned on Tue, 07/12/2022 - 13:50.

2 warnings found.

web/modules/contrib/cache_tools/src/Plugin/views/cache/SanitizedCacheTag.php:
┌──────────┬──────┬────────────────────────────────────────────────────────────┐
│  STATUS  │ LINE │                          MESSAGE                           │
├──────────┼──────┼────────────────────────────────────────────────────────────┤
│ Check    │ 92   │ Call to deprecated method getPublishedEntityTypeCacheTag() │
│ manually │      │ of class Drupal\cache_tools\Service\CacheSanitizer.        │
│          │      │                                                            │
└──────────┴──────┴────────────────────────────────────────────────────────────┘

web/modules/contrib/cache_tools/cache_tools.info.yml:
┌──────────┬──────┬────────────────────────────────────────────────────────────┐
│  STATUS  │ LINE │                          MESSAGE                           │
├──────────┼──────┼────────────────────────────────────────────────────────────┤
│ Check    │ 0    │ Value of core_version_requirement: ^8 || ^9 is not         │
│ manually │      │ compatible with the next major version of Drupal core. See │
│          │      │ https://drupal.org/node/3070687.                           │
│          │      │                                                            │
└──────────┴──────┴────────────────────────────────────────────────────────────┘

Steps to reproduce

Run upgrade_status check on the module https://www.drupal.org/project/upgrade_status

Issue fork cache_tools-3295523

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

petar_basic created an issue. See original summary.

petar_basic opened merge request !4

petar_basic’s picture

petar_basic commented
Assigned: petar_basic » mostepaniukvm
Priority: Normal » Critical

The warnings from the report are resolved in the MR.

petar_basic’s picture

petar_basic commented
Status: Active » Needs review

mostepaniukvm made their first commit to this issue’s fork.

mostepaniukvm’s picture

mostepaniukvm
he/him
commented
Status: Needs review » Reviewed & tested by the community

I tested PR on drupal 10. Works well but I had to add small change to fix dependencies in child SanitizedCacheFieldTag. You can see my commit in PR. Please just continue, merge it and tag new d10 compatible release.

  • petar_basic committed ed3d170 on 8.x-1.x 
    Issue #3295523 by petar_basic, mostepaniukvm: Drupal 10 compatibility
petar_basic’s picture

petar_basic commented
Assigned: mostepaniukvm » petar_basic
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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