IMPR: use f-string instead of old format type

Change-Id: I1d367e53b50fb1daf5914d8d2bae6034af8623ad

Author: xqt

pywikibot/data/api/_generators.py

@@ -634,8 +634,8 @@ def generator(self):
                 previous_result_had_data = True
             else:
                 if 'query' not in self.data:
-                    pywikibot.log("%s: 'query' not found in api response." %
-                                  self.__class__.__name__)
+                    pywikibot.log(f"{type(self).__name__}: 'query' not found"
+                                  ' in api response.')
                     pywikibot.log(str(self.data))
 
                 # if (query-)continue is present, self.resultkey might not have

pywikibot/logentries.py

@@ -61,9 +61,10 @@ def __missing__(self, key: str) -> None:
         for hidden_key, hidden_types in hidden.items():
             if hidden_key in self and key in hidden_types:
                 raise HiddenKeyError(
-                    "Log entry ({}) has a hidden '{}' key and you don't have "
-                    "permission to view it due to '{}'"
-                    .format(self['type'], key, hidden_key))
+                    f'Log entry ({self["type"]}) has a hidden {key!r} key and'
+                    " you don't have permission to view it due to "
+                    f'{hidden_key!r}'
+                )
 
         raise KeyError(f"Log entry ({self['type']}) has no {key!r} key")
 

pywikibot/site/_generators.py

@@ -197,7 +197,7 @@ def preloadpages(
             for pagedata in rvgen:
                 pywikibot.debug(f'Preloading {pagedata}')
                 try:
-                    if pagedata['title'] not in cache:
+                    if (pd_title := pagedata['title']) not in cache:
                         # API always returns a "normalized" title which is
                         # usually the same as the canonical form returned by
                         # page.title(), but sometimes not (e.g.,
@@ -206,19 +206,18 @@ def preloadpages(
                         # the response that corresponds to the canonical form
                         # used in the query.
                         for key, value in cache.items():
-                            if self.sametitle(key, pagedata['title']):
-                                cache[pagedata['title']] = value
+                            if self.sametitle(key, pd_title):
+                                cache[pd_title] = value
                                 break
                         else:
-                            pywikibot.warning(
-                                'preloadpages: Query returned unexpected '
-                                "title '{}'".format(pagedata['title']))
+                            pywikibot.warning('preloadpages: Query returned '
+                                              f'unexpected title {pd_title!r}')
                             continue
 
                 except KeyError:
-                    pywikibot.debug(f"No 'title' in {pagedata}")
-                    pywikibot.debug(f'{pageids=!s}')
-                    pywikibot.debug(f'titles={list(cache.keys())}')
+                    pywikibot.debug(f"No 'title' in {pagedata}\n"
+                                    f'{pageids=!s}\n'
+                                    f'titles={list(cache.keys())}')
                     continue
 
                 priority, page = cache[pagedata['title']]