Made the umbraco content properties order by name

[prjseal]

This change makes the Umbraco Content Properties Data List Source order by name to make the result much more usable.

As you can see, when not ordered alphabetically they are all over the place:

[prjseal]

As discussed on Twitter, we could configure a field to make this optional

Here is a link to where it has been done before in the enum data list source

Field defined here:

https://github.com/leekelleher/umbraco-contentment/blob/e190aa3b279106e375b5f31589d52f36815b0a18/src/Umbraco.Community.Contentment/DataEditors/DataList/DataSources/EnumDataListSource.cs#L95

Field used here:

https://github.com/leekelleher/umbraco-contentment/blob/e190aa3b279106e375b5f31589d52f36815b0a18/src/Umbraco.Community.Contentment/DataEditors/DataList/DataSources/EnumDataListSource.cs#L155

[prjseal]

Also discussed on Twitter, it would be good to have an option to include the Name field or not.

[prjseal]

I was able to add the option for sort alphabetical and include name in my local version. I will add it to the PR later.

[prjseal]

Hi Lee, I've pushed the configuration changes. Would like to hear your thoughts.

[leekelleher]

HI @prjseal, all looking good. I've made a couple of amends to the code (see commit 3405a76edc439885dc6244e5e1468bfc02c4f315), namely:

• Moving the include Name option above the sort order. (I dunno, I like the flow).
• Reworked the code to add the Name option at the top instead of the bottom (for default/non-alphabetic ordering). My thinking is that it's treated as a special property alias, so have it at the top. (again, dunno, just felt right to me).

If you're happy with those amends, then I'll merge it in. Thanks again! 🚀

[prjseal]

Looks spot on to me. I don't know how to change it from being a draft PR. Happy for you to merge it

[prjseal]

I've changed it from a draft to ready for review now. Hopefully that means you can merge it now.

[leekelleher]

Thanks @prjseal. I did an early dart on Friday, (hence my radio silence). All merged in now. Thanks again!

[leekelleher]

@prjseal I've just put a patch release out with this feature, Contentment v4.4.5. Thanks again!