Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Syncing changes to Member Types can result in core properties getting lost #147

Closed
NikRimington opened this issue Aug 5, 2020 · 2 comments
Assignees
Labels
release/8.6.5 8.6.5 - Patch release

Comments

@NikRimington
Copy link
Contributor

Using the MemberTypeHandler to update a member type with new properties has resulted in some of the core properties being removed. As a result, it is not possible to save a member any more!

System.NullReferenceException: Object reference not set to an instance of an object.
   at Umbraco.Web.Editors.ContentControllerBase.MapPropertyValuesForPersistence[TPersisted,TSaved](TSaved contentItem, ContentPropertyCollectionDto dto, Func`3 getPropertyValue, Action`3 savePropertyValue, String culture) in d:\a\1\s\src\Umbraco.Web\Editors\ContentControllerBase.cs:line 86
   at Umbraco.Web.Editors.MemberController.MapPropertyValues(MemberSave contentItem) in d:\a\1\s\src\Umbraco.Web\Editors\MemberController.cs:line 400
   at Umbraco.Web.Editors.MemberController.PostSave(MemberSave contentItem) in d:\a\1\s\src\Umbraco.Web\Editors\MemberController.cs:line 284
   at lambda_method(Closure , Object , Object[] )
   at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ActionExecutor.<>c__DisplayClass6_2.<GetExecutor>b__2(Object instance, Object[] methodParameters)
   at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ExecuteAsync(HttpControllerContext controllerContext, IDictionary`2 arguments, CancellationToken cancellationToken)
@KevinJump
Copy link
Owner

hopefully fixed in 6ccf84e

@KevinJump KevinJump added the release/8.6.5 8.6.5 - Patch release label Aug 5, 2020
@KevinJump KevinJump self-assigned this Aug 5, 2020
@NikRimington
Copy link
Contributor Author

Can confirm it appears to resolve the issue, thanks for the Dll to test :-D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/8.6.5 8.6.5 - Patch release
Projects
None yet
Development

No branches or pull requests

2 participants