Opened 4 years ago

Closed 4 years ago

#2935 closed change (fixed)

[cms] Give get_string global template function sensible default for page paramter

Reported by: kzar Assignee: kzar
Priority: Unknown Milestone:
Module: Sitescripts Keywords: cms
Cc: saroyanm, trev Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29324494/

Description

Background

In our CMS there is a get_string global template function defined that can be used from Jinja2 templates. It takes two arguments, a string name and a page name. Currently neither have default values.

What to change

It is quite comment to want to obtain strings for the current page. Therefore the page name parameter for the get_string global template function should have a default value of the current page.

Change History (2)

comment:1 Changed 4 years ago by kzar

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:2 Changed 4 years ago by kzar

  • Resolution set to fixed
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.