This branch add request.redirect on all requests.
In case of a front end request, we do an url_for to the location.

We removed redirect_with_hash that was only for retro compatibility

local_redirect has been renamed to redirect_query, and param keep_hash has been
removed and moved.

Default code for redirect is 303 now instead of 302.

Now redirect and redirect_query make local redirect by default, you need to
pass local=False to make external redirect.

All werkeug.utils.redirect has been replaced by request.redirect.

Http.redirect now use an http.Response type, and it become easy to add an
override like 'set_cookies' e.g.

Dispatch of a website.page return an http.response too, so we first need to
check if it is a cached version before to check if it is an Odoo Response.

Migrate your code:

http.redirect -> request.redirect(location, code, local)
http.local_redirect -> request.redirect_query(location, query, code, local)
http.redirect_with_hash -> request.redirect

Courtesy of odony for help and review ;)

See commit 478068c82999eaf77fd2b68659bb3fb3d3c2872f

The Subscriptions tree view shows the field `invoice_count` it is a computed field
counting invoices related with subscriptions

This PR is changing the way to get all them using read_group in order to
get all the result using only one faster query with all ids:

    SELECT min("account_move_line".id) AS id,
        count("account_move_line".id) AS "subscription_id_count" ,
        COUNT(DISTINCT "account_move_line"."move_id") AS "move_id",
        "account_move_line"."subscription_id" as "subscription_id"
    FROM "account_move_line"
        LEFT JOIN "sale_subscription" AS "account_move_line__subscription_id"
          ON ("account_move_line"."subscription_id" = "account_move_line__subscription_id"."id")
    WHERE ("account_move_line"."subscription_id" in (...))
    GROUP BY "account_move_line"."subscription_id","account_move_line__subscription_id"."id"
    ORDER BY  "account_move_line__subscription_id"."id"

It is showing the following line profile result using 1000 records
in real database:

    Total time: 4.20824 s
    Line #      Hits         Time  Per Hit   % Time  Line Contents
    ==============================================================
    250                                               def _compute_invoice_count(self):
    251         1        123.0    123.0      0.0          can_read = self.env['account.move'].check_access_rights('read', raise_exception=False)
    252         1          1.0      1.0      0.0          if not can_read:
    253                                                       self.update({'invoice_count': 0})
    254                                                       return
    255         2     750501.0 375250.5     17.8          res = self.env['account.move.line'].read_group(
    256         1        216.0    216.0      0.0              [('subscription_id', 'in', self.ids)], ['move_id:count_distinct'], ['subscription_id'], lazy=False)
    257         1        165.0    165.0      0.0          invoice_count_dict = {r['subscription_id'][0]: r['move_id'] for r in res}
    258      1001       3721.0      3.7      0.1          for subscription in self:
    259      1000    3453515.0   3453.5     82.1              subscription.invoice_count = invoice_count_dict.get(subscription.id, 0)

Instead of running too much queries with only one id like original way:

    SELECT count(1)
    FROM "account_move"
    WHERE "account_move"."id" in (SELECT "account_move_line"."move_id"
            FROM "account_move_line"
            WHERE "account_move_line"."subscription_id" = ID1
            ORDER BY  "account_move_line"."id")

    SELECT count(1)
    FROM "account_move"
    WHERE "account_move"."id" in (SELECT "account_move_line"."move_id"
            FROM "account_move_line"
            WHERE "account_move_line"."subscription_id" = ID2
            ORDER BY  "account_move_line"."id")

    ... (other N times running these kind of queries for each ID)

It was showing the following line profile result using 1000 records
in real database:

    Total time: 404.346 s
    Line #      Hits         Time  Per Hit   % Time  Line Contents
    ==============================================================
    250                                               def _compute_invoice_count(self):
    251         1         23.0     23.0      0.0          Invoice = self.env['account.move']
    252         1        102.0    102.0      0.0          can_read = Invoice.check_access_rights('read', raise_exception=False)
    253      1001       4651.0      4.6      0.0          for subscription in self:
    254      1000  404340821.0 404340.8    100.0              subscription.invoice_count = can_read and Invoice.search_count([('invoice_line_ids.subscription_id', '=', subscription.id)]) or 0

It means, speed-up 100x faster

Forward-Port-Of: odoo/enterprise#19511