[Improvement] Override Umbraco's HasValue() and GetValue() instead of adding new methods HasVortoValue() and GetVortoValue() about umbraco-vorto HOT 5 CLOSED

I believe this would just result in an ambiguous reference as the runtime wouldn't know which method to use, hence why we have vorto prefixed versions.

from umbraco-vorto.

You can always add global reference to required DLL in Views/web.config to avail such library across all views 👍

from umbraco-vorto.

Thanks for the suggestions, but I think I'd prefer to leave it as is as I don't want to cause any confusion with people not knowing if vorto is being used or the default Umbraco GetValue methods. If this is something you want personally then you can always make your own convenience methods to wrap the GetVortoValue methods, however I think from a general perspective, it is much clearer using the explicit GetVortoValue method what the intention is and thus where to look if problems do arise.

from umbraco-vorto.

The reason this is requested is probably related to the Models Builder. See this for more info: modelsbuilder/ModelsBuilder.Original#72 (comment)

from umbraco-vorto.

I still don't think this would help though as Models Builder would be coded to use the Umbraco implementation so by having the same method name, it wouldn't automatically just use it, Models Builder would have to make an explicit attempt to use it.

from umbraco-vorto.