\Drupal\entity\Access\EntityRevisionRouteAccessChecker provides basically the exact same code already so we could replace it
There might be still some references to it in other parts of the code
View original github issue - https://github.com/Jaesin/content_entity_base/issues/21

Comments

paboden created an issue. See original summary.

pashupathi nath gajawada’s picture

pashupathi nath gajawada CreditAttribution: pashupathi nath gajawada as a volunteer and at Melity commented
Assigned: Unassigned » pashupathi nath gajawada

Let me have a look at the issue.

minnur’s picture

minnur CreditAttribution: minnur as a volunteer and at Chapter Three commented
Assigned: pashupathi nath gajawada » Unassigned

The fix already exists, nothing to look at.

pashupathi nath gajawada’s picture

pashupathi nath gajawada CreditAttribution: pashupathi nath gajawada as a volunteer and at Melity commented
Status: Active » Closed (works as designed)

If this is already fixed, then why don't you close the issue.
I am closing this issue, as this is already fixed.

minnur’s picture

minnur CreditAttribution: minnur as a volunteer and at Chapter Three commented
Status: Closed (works as designed) » Active

Please do not close these issues. Several people from Chapter Three working on this module. The reason we created this tickets is only to keep track of all issues from Github. Once everything verified we will mark them as fixed one by one.

pashupathi nath gajawada’s picture

pashupathi nath gajawada CreditAttribution: pashupathi nath gajawada as a volunteer and at Melity commented

Oh sorry @Minnur,
I was not aware of it.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer and at Chapter Three commented
Status: Active » Fixed

We resolved that in the meantime by not requiring entity module.

Status: Fixed » Closed (fixed)

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

Jaesin credited Jaesin.

Jaesin’s picture

Jaesin CreditAttribution: Jaesin at Chapter Three commented

Yes, in the end, we gave up the entity module. It's seems to be working out pretty well so far.