[ckan-changes] [ckan/ckan] 25f717: Initialize JinjaOptions right after app creation.

Adrià Mercader noreply at github.com
Fri Sep 13 13:48:50 UTC 2019


  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: 25f71751aa7386651c90d1f274bb68798773d97a
      https://github.com/ckan/ckan/commit/25f71751aa7386651c90d1f274bb68798773d97a
  Author: Sergey Motornyuk <sergey.motornyuk at linkdigital.com.au>
  Date:   2019-08-28 (Wed, 28 Aug 2019)

  Changed paths:
    M ckan/config/middleware/flask_app.py

  Log Message:
  -----------
  Initialize JinjaOptions right after app creation.

Starting from Flask>=1.0, some of plain `app` properties were
rewritten as property descriptors, so setting them may accidentialy
initialize JinjaEnvironment. After this point jinja2 won't notice
any change to `jinja_options`. So I reordered assignment in app
initialization and now `jinja_options` added firstly.


  Commit: 5be464228f8896d6fcebdd40e57c80cf79eb31c8
      https://github.com/ckan/ckan/commit/5be464228f8896d6fcebdd40e57c80cf79eb31c8
  Author: Sergey Motornyuk <sergey.motornyuk at linkdigital.com.au>
  Date:   2019-08-29 (Thu, 29 Aug 2019)

  Changed paths:
    M ckan/config/middleware/flask_app.py

  Log Message:
  -----------
  remove duplicated options assignment


  Commit: 0b2a6c753713199da0ebc8afaaa49e89dee85599
      https://github.com/ckan/ckan/commit/0b2a6c753713199da0ebc8afaaa49e89dee85599
  Author: Adrià Mercader <amercadero at gmail.com>
  Date:   2019-09-13 (Fri, 13 Sep 2019)

  Changed paths:
    M ckan/config/middleware/flask_app.py

  Log Message:
  -----------
  Merge pull request #4955 from DataShades/early-jinja-options

Initialize JinjaOptions right after app creation.


Compare: https://github.com/ckan/ckan/compare/a86242a10ef0...0b2a6c753713


More information about the ckan-changes mailing list