Display source entities hidden by the Trash module

Ouch, thanks for raising this. It's indeed very unfortunate that the Trash module is so aggressive when hiding trashed entities... This means that our code in the controller:

        $source_entity = $type_storage->load($source_id);
        if (!$source_entity) {
          // If for some reason this record is broken, just skip it.
          continue;
        }

will just think the source entity is broken since it can't be loaded, and skip it entirely from the table. I actually think this behavior from the Trash module could produce other unintended effects on sites using that module, because "the entity was moved into the trash" doesn't necessarily mean IMO "no other piece of code should be able to load this entity".

I'd like to give this a little more thought, but at this point I'm leaning towards just displaying a general warning message at the top of the usage page if the Trash module is enabled, or something along those lines. Thoughts?

I looked a bit deeper and I see that there seems to be a way to execute code bypassing Trash's hiding mechanisms. I don't really love the idea of special-casing Trash's behavior in our controller, but at this point maybe it's a good compromise?
So instead of having:

        $source_entity = $type_storage->load($source_id);
        if (!$source_entity) {
          // If for some reason this record is broken, just skip it.
          continue;
        }

we could have something along the lines of (untested):

        // Special-case the Trash module behavior, which hides the entities in
        // trash from all entity_loads.
        if (\Drupal::getContainer()->has('trash.manager')) {
          $trash_manager = \Drupal::getContainer()->get('trash.manager');
          $source_entity = $trash_manager->executeInTrashContext('inactive', function () use ($type_storage, $source_id) {
            return $type_storage->load($source_id);
          });
        }
        else {
          $source_entity = $type_storage->load($source_id);
        }
        if (!$source_entity) {
          // If for some reason this record is broken, just skip it.
          continue;
        }

and similarly to other places we may be checking things on the source entities (for example access checks, getting the labels, etc). If we know the entity is in trash, we may even add a suffix of type " (in trash)" to the label...

Thoughts?

@bkosborne sorry just re-read the issue summary and I am realizing you had already suggested something along those lines...

An alternative approach would be if this module detected if the Trash module was enabled, then it could deactivate trash's hiding feature on the usage page to it continues to show that usage, but then also indicates that the source entity is in the trash. All of that adds a pretty big coupling to the Trash module though.

So yes I believe we should go this route, even though I think it's suboptimal from a maintenance perspective... 🤷‍♂️

I tested the MR on a site using trash module 3.0.21 and entity_usage 8.x-2.0-beta24. Works and marking RTBC.

I have rerolled with -dev and all tests are indeed green.
Thanks everyone for working on this!

Hi maintainers! I appreciate the work you do on this useful module. Please cut a new release incorporating the trash module integration asap. Thanks!

@daddison thanks for reaching out. I plan to tag a new release later this week 👍

Hi @marcoscano - Are you still planning to tag a new release soon?

Just tagged https://www.drupal.org/project/entity_usage/releases/8.x-2.0-beta25 with this fix.
I was hopeful we could include other improvements in the same tag, but they are not ready yet. No big deal, I'll cut another release when that happens.
Thanks for your patience.