Rewrite url stackoverflow

How does SO perform url rewriting if we only set the question id?

questions/{id}/{whatever}

      

to

questions/{id}/{question-slug}

      

I've been working with MVC for a while and I have a job (routes, action, everything) to make it pick up the correct content based on the ID provided.

However, the part after {id}

(slug part) remains the same as typed. Therefore, if someone typed content/5/foobar

, it will display the desired content, but leave it there foobar

.

In the controller (or somewhere else, please suggest where) I would need to go into the DB and pull out the correct slug, put it in the route data and then redirect to the same action with that correct data, I guess?

This is an attempt at overriding the execution result. It works, but doesn't redirect or replace / display the correct url in the browser:

    protected override void Execute(System.Web.Routing.RequestContext requestContext) {

        if (requestContext.RouteData.Values["id"] != null) {
            string currentSlug = _repository.Find(int.Parse(requestContext.RouteData.Values["id"].ToString())).Slug;
            if (requestContext.RouteData.Values["slug"] != null) {
                requestContext.RouteData.Values.Remove("slug");
            }
            requestContext.RouteData.Values.Add("slug", currentSlug);
        }

        base.Execute(requestContext);
    }

      

This is another version of the action Display

that works well , so you can see what it does and get an idea of ​​what I want:

    //
    // GET: {culture}/directory/5/{slug}
    public virtual ActionResult Display(int id, string slug)
    {
        var model = _repository.Find(id);
        if (model != null) {
            if (!model.Slug.Equals(slug, System.StringComparison.OrdinalIgnoreCase)) {
                return RedirectToActionPermanent(pndng.DirectoryEntry.ActionNames.Display, pndng.DirectoryEntry.Name, new { id = model.Id, slug = model.Slug });
            }
            return View(model);
        }
        // no model found
        return InvokeHttp404(HttpContext);
    }

      

This does a permanent redirect (it does what I want), but is it true? I guess I need to redirect refreshing the browser url, right?

+3


source to share


2 answers


  public ActionResult Details(int id, string slug)
        {
            var session = MvcApplication.CurrentRavenSession;

            var blogPostRelations = session
                .Query<BlogPost, BlogPosts_WithRelatedData>()
                .Where(x => x.IntId == id)
                .As<BlogPostRelations>()
                .FirstOrDefault()
                ;


            if (blogPostRelations == null)
                return HttpNotFound();

            if (blogPostRelations.BlogPost.DisplayData.Slug.Value != slug)
                return RedirectToActionPermanent("Details", new { id = id, slug = blogPostRelations.BlogPost.DisplayData.Slug.Value });

            return View(blogPostRelations);
        }

      

Note:



if (blogPostRelations.BlogPost.DisplayData.Slug.Value != slug)
                    return RedirectToActionPermanent("Details", new { id = id, slug = blogPostRelations.BlogPost.DisplayData.Slug.Value });

      

So, your # 2 approach is correct.

+2


source


You can write your own route for this:

public class QuestionsRoute : Route
{
    public QuestionsRoute()
        : base(
            "questions/{id}/{slug}",
            new RouteValueDictionary(new
            {
                controller = "questions",
                action = "index",
                slug = UrlParameter.Optional
            }),
            new RouteValueDictionary(new
            {
                id = @"\d+"
            }),
            new MvcRouteHandler()
        )
    { }

    public override RouteData GetRouteData(HttpContextBase httpContext)
    {
        var rd = base.GetRouteData(httpContext);
        if (rd == null)
        {
            return null;
        }

        var id = rd.GetRequiredString("id");
        var slug = rd.Values["slug"] as string;
        if (string.IsNullOrEmpty(slug))
        {
            slug = GoFetchSlugFromDb(id);
            if (string.IsNullOrEmpty(slug))
            {
                return null;
            }

            httpContext.Response.RedirectToRoutePermanent(new
            {
                action = "index",
                controller = "questions",
                id = id,
                slug = slug
            });
            return null;
        }
        return rd;
    }

    private string GoFetchSlugFromDb(string id)
    {
        // TODO: you know what to do here
        throw new NotImplementedException();
    }
}

      

which will be registered in Application_Start

:



public static void RegisterRoutes(RouteCollection routes)
{
    routes.IgnoreRoute("{resource}.axd/{*pathInfo}");
    routes.Add("questions", new QuestionsRoute());
}

      

Your QuestionController should now be pretty simple:

public class QuestionsController: Controller
{
    public ActionResult Index(int id, string slug)
    {
        ...
    }
}

      

+2


source







All Articles