Better handling of [ApiController] usage

[bkoelman]

Enhancements:

• Log a warning when [ApiController] is used on a JSON:API controller.
• Better error response on invalid ModelState when [ApiController] is used on a JSON:API controller.

Based on the work in #1225.
Closes #1224.

QUALITY CHECKLIST

• Changes implemented in code
• Complies with our contributing guidelines
• Adapted tests
• N/A: Documentation updated

[codecov]

Codecov Report

Merging #1246 (79cfb3d) into master (be99234) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1246      +/-   ##
==========================================
+ Coverage   92.62%   92.63%   +0.01%     
==========================================
  Files         243      243              
  Lines        7795     7806      +11     
==========================================
+ Hits         7220     7231      +11     
  Misses        575      575              

Impacted Files	Coverage Δ
...etCore/Errors/UnsuccessfulActionResultException.cs	96.00% <100.00%> (+1.00%)	⬆️
...iDotNetCore/Middleware/JsonApiRoutingConvention.cs	92.75% <100.00%> (+0.69%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.