Removed virtual from the RedirectController.Redirect() action

JeremyCaney · JeremyCaney · commit e77c7d5038cc · 2021-01-27T12:53:24.000-08:00
This implementation is so straight forward that it's likely anyone with specialized needs would just create their own controller. This is really just a convenience controller for a standard requirement; customers with more sophisticated needs have added their own redirect controllers either to replace or to complement the out-of-the-box `RedirectController`.

If we discover a specific extensibility point that we want to customize in the future, we can reevaluate the best approach for that then.
diff --git a/OnTopic.AspNetCore.Mvc/Controllers/RedirectController.cs b/OnTopic.AspNetCore.Mvc/Controllers/RedirectController.cs
@@ -49,7 +49,7 @@ public RedirectController(ITopicRepository topicRepository) : base() {
     ///   Redirect based on <see cref="Topic.Id"/>.
     /// </summary>
     /// <param name="topicId">The <see cref="Topic.Id"/> to lookup in the <see cref="ITopicRepository"/>.</param>
-    public virtual ActionResult Redirect(int topicId) {
+    public ActionResult Redirect(int topicId) {
 
       /*------------------------------------------------------------------------------------------------------------------------
       | Find the topic with the correct PageID.
