• 
      

    Add a central function for returning the server URL.

    Review Request #6307 — Created Sept. 9, 2014 and submitted — Latest diff uploaded

    Information

    Review Board
    release-2.0.x
    bdc92c2...

    Reviewers

    We had a lot of code that attempted to build a server URL by fetching a
    Site, SiteConfiguration, doing a URL lookup, and building a string. This
    was getting silly, so I've added two central functions for this purpose:
    get_server_url() and build_server_url().

    get_server_url() returns the URL to the root of the server (or a Local
    Site, if one of the identifying arguments are provided).

    build_server_url() builds a URL off of the server URL. It's a
    convenience around calling urljoin(get_server_url(), ...).

    Many call sites have been updated to use these new functions. Many still
    remain, as we have templates that are building their own URLs, and some
    code that's doing fancier URL building, and I didn't want to risk
    breaking them right now.

    This also fixes a bug with the e-mail headers. We were always referring
    to the root of the site, rather than the Local Site, which made it harder
    to filter when working on multiple Local Sites.

    Unit tests pass.

    Tested each affected call site manually, with and without Local Sites.