DarthFubuMVC/fubucore — comment on commit 2466eddb2e066ab285b69349493ff985567fbf72
File:  line 0
By emiaj:

This change means, that I can't no longer rely on what LocalizationExtensions gives me as LocalizationKey from a PropertyToken instance?
I had this helper somewhere in my code base:

public static void Append(this ILocaleCache cache, Expression> property, string value)
{
cache.Append(PropertyToken.For(property).ToLocalizationKey(), value);

// Added by me
Debug.Assert(LocalizationManager.GetHeader(property) == value);
}
And now the assertion fails.

In order to make it work as before, I have changed the code to:

cache.Append(new LocalizationKey(PropertyToken.For(property).StringTokenKey), value);

Should then LocalizationExtensions be removed? Or updated?

Just found this issue today.

Kind regards,
Jaime.