mirror of https://github.com/Sonarr/Sonarr
55 lines
2.4 KiB
Plaintext
55 lines
2.4 KiB
Plaintext
================================================================================================
|
|
change - Removed WebLogger and WebLoggerFactory
|
|
|
|
impact - low
|
|
fixability - medium
|
|
revision -
|
|
|
|
description - To minimize management overhead the classes were removed so that only single
|
|
Client Profile version of Castle.Core can be distributed.
|
|
|
|
fix - You can use NLog or Log4Net web logger integration, or reuse implementation of existing
|
|
web logger and use it as a custom logger.
|
|
|
|
================================================================================================
|
|
change - Removed obsolete overload of ProxyGenerator.CreateClassProxy
|
|
|
|
impact - low
|
|
fixability - trivial
|
|
revision -
|
|
|
|
description - Deprecated overload of ProxyGenerator.CreateClassProxy was removed to keep the
|
|
method consistent with other methods and to remove confusion
|
|
|
|
fix - whenever removed overload was used, use one of the other overloads.
|
|
|
|
================================================================================================
|
|
change - IProxyGenerationHook.NonVirtualMemberNotification method was renamed
|
|
|
|
impact - high
|
|
fixability - easy
|
|
revision -
|
|
|
|
description - to accommodate class proxies with target method NonVirtualMemberNotification on
|
|
IProxyGenerationHook type was renamed to more accurate NonProxyableMemberNotification
|
|
since for class proxies with target not just methods but also fields and other member that
|
|
break the abstraction will be passed to this method.
|
|
|
|
fix - whenever NonVirtualMemberNotification is used/implemented change the method name to
|
|
NonProxyableMemberNotification. Implementors should also accommodate possibility that not
|
|
only MethodInfos will be passed as method's second parameter.
|
|
|
|
================================================================================================
|
|
change - DynamicProxy will now allow to intercept members of System.Object
|
|
|
|
impact - very low
|
|
fixability - easy
|
|
revision -
|
|
|
|
description - to allow scenarios like mocking of System.Object members, DynamicProxy will not
|
|
disallow proxying of these methods anymore. AllMethodsHook (default IProxyGenerationHook)
|
|
will still filter them out though.
|
|
|
|
fix - whenever custom IProxyGenerationHook is used, user should account for System.Object's
|
|
members being now passed to ShouldInterceptMethod and NonVirtualMemberNotification methods
|
|
and if neccessary update the code to handle them appropriately. |