When ItemType == Link, cannot get Url about our.umbraco.umbnav

[AussieInSeattle]

Example below - cut off the bottom so I didnt have to blur out my content tree

[AussieInSeattle]

good catch - missing using statement - adding this to view fixed it:
@using UmbNav.Core.Extensions
Will buy you a beer at CG for your troubles :-)

[AaronSadlerUK]

Yes that's correct, you need to use the extension method

.Url()

[AussieInSeattle]

that fails also (this is for type=link, not type=content) - see screenshot - am I missing a using that would make .Url() work?

[AaronSadlerUK]

Hmmm .Url() is in the UmbNav.Core DLL

It's been a long time since I've touched V1, I think link is for external URLs, I'd need to get a V8 site up and running to debug

[AussieInSeattle]

I think link is for external URLs
Correct - this is only an issue for external url's - am assuming this is only coming up now as it's rare to have external url's in your top navigation where this package is likely mostly used?

[AaronSadlerUK]

Yeah I think that could be the case, I've done this and got it working in V10+ So I presume it's a bug specific to V8...

I'll do some digging...

Can you give me an example of the backoffice item, so I can be 100% sure I'm debugging it correctly

[AussieInSeattle]

When debugging I can see .Url (string) populated under Non-Public Properties - so I think it's a matter of making .Url public but only when type=link - not sure who you'd do that

[AaronSadlerUK]

That was made internal for a specific reason, it broke things with the multilingual which is why the extension method was brought in... I will debug this and get back to you

[AussieInSeattle]

That was made internal for a specific reason, it broke things with the multilingual which is why the extension method was brought in... I will debug this and get back to you

thanks - let me know if you need more info

[AaronSadlerUK]

I've just run V8 and it works for me:

These are the two usings:

@using UmbNav.Core.Extensions
@using UmbNav.Core.Models

Are you sure the Url() you are using in your code is the UmbNav one?
Your error seems to say PublishedContentExtensions.Url() whereas the UmbNav one is UmbNavItemExtensions.Url()

[AaronSadlerUK]

Just to add the UmbNav extension does a null check here:

Our.Umbraco.UmbNav/src/UmbNav.Core/Extensions/UmbNavItemExtensions.cs

Line 125 in ac2e3a9

if (item.Content != null)

and returns .Url if it is null here:

Our.Umbraco.UmbNav/src/UmbNav.Core/Extensions/UmbNavItemExtensions.cs

Line 151 in ac2e3a9

return item.Url;

[AaronSadlerUK]

Glad you got it working!