valkyrjaio / valkyrja-php / 31150269722

100%

push

github

[Container] feat(#1168): Add getAliasedId to the container contract (#1171)

# Description

The container contract could say that an alias exists, and it could
resolve one, but it could not say what an alias points to.

This adds a new method called `getAliasedId` that returns the id an
alias is bound to. For example:

```php
$container->bindAlias(ClassAFromParent::class, ClassA::class);

$container->getAliasedId(ClassAFromParent::class); // ClassA::class
```

The protected `getAlias` becomes that method, so the container keeps one
way to read the alias map instead of two. The method reads one hop, so
the caller controls how far to follow a chain.

`isService`, `isSingleton`, and `isPublished` stay literal, because a
method that followed the alias map would need an accessor that followed
it too. `ChildContainer` answers `isAlias` from the parent, so it
answers `getAliasedId` from the parent as well.

Closes #1168.

## Types of changes

- [x] Improvement _(non-breaking change which improves code)_
- [ ] Bug fix _(non-breaking change which fixes an issue)_
- [ ] New feature _(non-breaking change which adds functionality)_
- [ ] Deprecation _(breaking change which removes functionality)_
- [ ] Breaking change _(fix or feature that would cause existing
functionality to change)_
- [ ] Documentation improvement

## Changes

- **`ContainerContract`** — declares `getAliasedId`.
- **`Container`** — promotes `getAlias` to the public `getAliasedId`.
- **`ChildContainer`** — answers `getAliasedId` from the parent, to
agree with `isAlias`.
- **`NativeChildContainer`** — renames its override to match. No
behavior change.
- **`README.md`** — documents `getAliasedId`.

12 of 0 new or added lines in 0 files covered. (NaN%)

12766 of 12766 relevant lines covered (100.0%)

24.77 hits per line