Issues when using core property value converters

[bjarnef]

When using core property value converters in Umbraco 7.6.x and 7.7.x and casting e.g. value from an Author Picker Umbraco.MultiNodeTreePicker2 to IEnumerable<IPublishedContent> it returns an exception:

Der opstod en fejl under udførelsen af den underordnede anmodning for handler 'System.Web.Mvc.HttpHandlerUtil+ServerExecuteHttpHandlerAsyncWrapper'.

foreach (BlogPost blogPost in blogPosts)
{
     Employee author = (blogPost.Author ?? Enumerable.Empty<IPublishedContent>()).FirstOrDefault() as Employee;
}

When I use same logic just in a regular partial view there are no issues.

[bjarnef]

I had switcher from Umbraco.MultiNodeTreePicker to Umbraco.MultiNodeTreePicker2, but I also needed to re-select the items and recycle the application pool.

[Jeavon]

Line 49 & 50 in the below file should be removed

LeBlender/Src/Lecoati.LeBlender.Extension/Models/LeBlenderPropertyModel.cs

Line 49 in e11905f

if (value2 is T)

Source should never be returned, it should always be Object that is returned

[bjarnef]

It would be awesome if you submit a PR for that @Jeavon

I also think it would be great to have this overload method as in Umbraco core 👍
#81

[bjarnef]

@soreng could you have a look at this one please? 😁

I think you should be able to reproduce if you add a LeBlender property using Multi Url Picker.
rasmusjp/umbraco-multi-url-picker#63

E.g. create two datatype instances of Multi Url Picker:

• Single Url Picker (max items = 1).
• Multi Url Picker (max items = empty).

Single Url Picker returns Link, so Model.Items.First().GetValue<Link>("link") should work.
Multi Url Picker returns IEnumerable<Link>, so Model.Items.First().GetValue<IEnumerable<Link>>("links") should work.

[soreng]

hi @bjarnef I can have a look at it later, but I can't really say when it will be :-/

I will be looking into a few thing (maybe a v2) in the not so distant future

[bjarnef]

Okay, hopefully you get some time to look at this issue.
In a project I am working on, it works with Model.Items.First().GetValue<Link>("link") which use multiple LeBlender items, but not when item a single LeBlender item (default if no min/max it set).

I have also had some issues when using MNTP for some menu items in a megamenu, where it sometimes returned IPublishedContent and sometimes IEnumerable<IPublishedContent> or just when casting Model.Items.First().GetValue<string>("link"), where I had to add a bit of a hack to make it work 😁
Jeavon/Umbraco-Core-Property-Value-Converters#24 (comment)

[bjarnef]

@soreng have you tried LeBlender with ModelsBuilder and property value converters enabled.

I have often see then returned value not is they expected inside LeBlender, while it works correct when the properties are directly on document types. It happens for e.g. Multinode Tree Picker, RJP MultiUrlPicker and others.

[soreng]

hi @bjarnef

after reading the documentation, I can see the LeBlender impl. is wrong, as per @Jeavon's reply above.

this need a test and a fix :)

[bjarnef]

@soreng looking forward a fix for this since it has really bothered me now and then, where it was returning different than expected.

Hopefully in next release and both a release for Our and NuGet.
https://our.umbraco.com/projects/backoffice-extensions/leblender/

At the moment I think the latest NuGet version is broken, and doesn't contain some of the fixes which was merged to master branch before the release of the latest NuGet package.

[soreng]

Maybe a 1.10 is comming out sometime soon - who knows

Just need access to upload stuff to the page on our. Will see it I can get that.