Issue 5331 - Adds has_string() global

cms/converters.py

Index: cms/converters.py
===================================================================
--- a/cms/converters.py
+++ b/cms/converters.py
@@ -376,16 +376,17 @@
         filters = {
             'translate': self.translate,
             'linkify': self.linkify,
             'toclist': self.toclist,
         }
 
         globals = {
             'get_string': self.get_string,
+            'has_string': self.has_string,
             'get_page_content': self.get_page_content,
         }
 
         for dirname, dictionary in [('filters', filters),
                                     ('globals', globals)]:
             for filename in self._params['source'].list_files(dirname):
                 root, ext = os.path.splitext(filename)
                 if ext.lower() != '.py':
@@ -437,16 +438,26 @@
 
         localedata = self._params['source'].read_locale(self._params['locale'],
                                                         page)
         default = localedata[name]

[Vasily Kuznetsov, 2017/06/20 17:38:27]

I didn't pay attention to this line when looking into this. Actually you seem to
be right and checking that the string exists is much simpler than I thought.

         return jinja2.Markup(self.localize_string(
             page, name, default, '', localedata, html_escapes
         ))
 
+    def has_string(self, name, page=None):
+        if page is None:
+            page = self._params['page']
+
+        localedata = self._params['source'].read_locale(self._params['locale'],

[Vasily Kuznetsov, 2017/06/20 17:38:27]

Maybe create a small protected method for this logic to not replicate it? For
example `_get_localedata(self, page)`

[Jon Sonesen, 2017/06/21 07:53:54]

yeah, this is good. but what about the if page is none logic? Basically, it
seems it would be redundant to put in the new method unless we remove it here.
the problem there is that we still have casees where the page will be none,
however the protected method could always return the page and the local data.
What do you think?

[Vasily Kuznetsov, 2017/06/21 08:17:00]

Yeah, all true. I was thinking to leave that part in this method (and have a bit
of duplication -- this one you can't really avoid I guess). So the new method
would just have one call to blabla.read_locale(...) in it, but there's quite
some stuff in it, so it makes sense I think.

+                                                        page)
+        if name in localedata:

[Vasily Kuznetsov, 2017/06/20 17:38:28]

`return name in localedata` would do the same in one line.

[Jon Sonesen, 2017/06/21 07:53:54]

Acknowledged.

+            return True
+        return False
+
     def get_page_content(self, page, locale=None):
         from cms.utils import get_page_params
 
         if locale is None:
             locale = self._params['locale']
         return get_page_params(self._params['source'], locale, page)
 
     def linkify(self, page, locale=None, **attrs):