- Issue created by @catch
- Status changed to Needs work
8 months ago 8:52pm 25 April 2024 - π¬π§United Kingdom catch
Removing the code first to see if the test coverage still passes, I think it probably will.
- Merge request !7749Remove the support first and see what happens to the unit test. β (Open) created by catch
- π¬π§United Kingdom catch
No wait I'm wrong, or at least, it's not definitely dead/redundant.
EntityBase::toUrl() still does this:
// Pass the entity data through as options, so that alter functions do not // need to look up this entity again. $uri ->setOption('entity_type', $this->getEntityTypeId()) ->setOption('entity', $this);
I think this eventually gets passed down as undocumented keys in the $options array in
OutboundPathProcessorInterface::processOutbound()
If that's the case, it might be valid after all, or... we might want to remove entity and entity_type from options in EntityBase::toUrl() instead.