Issue 7461 - Include source page into warning text for unresolved links

cms/sources.py

 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-present eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
 # but WITHOUT ANY WARRANTY; without even the implied warranty of
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ skipping 41 matching lines @@
                 locale = default_locale
         elif self.has_page(page):
             if not self.has_locale(locale, page):
                 locale = default_locale
         elif self.has_page(alternative_page):
             if not self.has_locale(locale, alternative_page):
                 locale = default_locale
         elif self.has_static(page):
             locale = None
         else:
-            logging.warning('Link to "%s" (from "%s") cannot be resolved',
-                            page, source_page)

[rhowell, 2019/04/12 20:17:12]

NIT: It might be slightly easier to read the error message the other way around.
E.g.

Link from "rel_path" to "rel_path.html" cannot be resolved
Link from "rel_path" to "myscripts.js" cannot be resolved

[Vasily Kuznetsov, 2019/04/15 16:48:45]

Yeah, makes sense. Done.

+            logging.warning('Link from "%s" to "%s" cannot be resolved',
+                            source_page, page)
 
         parts = page.split('/')
         if parts[-1] == default_page:
             page = '/'.join(parts[:-1])
         if locale:
             path = '/{}/{}'.format(locale, page)
             return locale, urlparse.urlunparse(parsed[0:2] + (path,) + parsed[3:])
         return locale, '/' + page
 
     def read_config(self):
@@ skipping 326 matching lines @@
             'read_config',
             'read_template',
             'read_locale',
             'read_file',
             'read_include',
             'exec_file',
         ]:
             setattr(source, fname, utils.memoize(getattr(source, fname)))
 
     return source