Skip to content
This repository has been archived by the owner on Dec 19, 2023. It is now read-only.

RegisterDecorator extension only available on ITargetContainerOwner interface #25

Closed
LordZoltan opened this issue Mar 30, 2017 · 1 comment
Closed

RegisterDecorator extension only available on ITargetContainerOwner interface #25

LordZoltan opened this issue Mar 30, 2017 · 1 comment
Assignees
@LordZoltan
Labels
bug
Milestone
1.2

Comments

@LordZoltan
Copy link
Member

Whilst it's correct that the RegisterDecorator API only works on target containers which also implement ITargetContainerOwner; it should still be available on ITargetContainer types.

If the target container on which it is called is not an ITargetContainerOwner, then a NotSupportedException should be thrown.
@LordZoltan LordZoltan added the bug label Mar 30, 2017
@LordZoltan LordZoltan added this to the 1.2 milestone Mar 30, 2017
@LordZoltan LordZoltan self-assigned this Mar 30, 2017
@LordZoltan LordZoltan added ready and removed ready labels Mar 30, 2017
LordZoltan added a commit that referenced this issue Apr 1, 2017
@LordZoltan
#25 on the way
deb4db2
@LordZoltan LordZoltan added ready and removed ready labels Apr 2, 2017
@LordZoltan
Copy link
Member Author

ITargetContainerOwnerinterface has been removed and unified into ITargetContainer.

This is obviously a breaking change for anyone who has directly used that interface in their code - but it's unlikely they have.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@LordZoltan LordZoltan

Labels
bug
Projects
None yet
Milestone
1.2
Development

No branches or pull requests
1 participant
@LordZoltan