Issue with DXA DefaultModelBinder for Keyword field

Question

I am migrating my old project to DXA 1.4. I am using Keyword fields in Schemas. While binding with default Tag Schema I am getting Keyword description instead of Keyword title. Reason for this is KeywordMap function with line

string displayText = String.IsNullOrEmpty(keyword.Description) ? keyword.Title : keyword.Description;

My question is what is the best way to get this behavior changed while binding a Model keeping in mind that it should always to ready to migrate to latest version? I can think of 2 way to do this:

1. Create your own Tag and ModelBuilder class, register it to modelBuilderPipeline and override the functions required.
2. Create your own Schema for Keyword field and create your own ModelBuilder class and add it also to modelBuilderPipeline to bind the model field.

Personally, I favor option 1. Please suggest, if anyone have faced similar issue. Thanks in advance.

Answer 1

Best option to override model binding methods is:

1. Implement a separate model builder class and register it along with default model builder. Write your own functions to override the default behavior in your custom model builder.
   

In this way model builder will be executed 2 times for every model.

2. Implement a custom model builder and override functions to change the default behavior. Remove Default model builder and register your custom model builder.

In this way model builder will be executed only 1 time for every model.

Here overall idea is to keep your custom code separate from default DXA code so that there will be no issue while migrating to latest version of DXA. Please share your suggestions.