From ba565758c6e9039b424fab97ce5b5de090d14fe8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9my=20Voet=20=28ryv=29?= <ryv@odoo.com>
Date: Wed, 15 Feb 2023 16:11:58 +0000
Subject: [PATCH] [REM] core: remove deprecated methods of models.py

closes odoo/odoo#112817

Signed-off-by: Vincent Schippefilt (vsc) <vsc@odoo.com>
---
 odoo/models.py | 84 --------------------------------------------------
 1 file changed, 84 deletions(-)

diff --git a/odoo/models.py b/odoo/models.py
index ce1b41d4134..4bdd985b206 100644
--- a/odoo/models.py
+++ b/odoo/models.py
@@ -1406,14 +1406,6 @@ class BaseModel(metaclass=MetaModel):
 
         return defaults
 
-    @api.model
-    def fields_get_keys(self):
-        warnings.warn(
-            'fields_get_keys() method is deprecated, use `_fields` or `get_views` instead',
-            DeprecationWarning, stacklevel=2,
-        )
-        return list(self._fields)
-
     @api.model
     def _rec_name_fallback(self):
         # if self._rec_name is set, it belongs to self._fields
@@ -4915,14 +4907,6 @@ class BaseModel(metaclass=MetaModel):
         return {key: val[0] if val else ''
                 for key, val in results.items()}
 
-    def get_xml_id(self):
-        warnings.warn(
-            'get_xml_id() is deprecated method, use get_external_id() instead',
-            DeprecationWarning, stacklevel=2,
-        )
-        return self.get_external_id()
-
-    # Transience
     @classmethod
     def is_transient(cls):
         """ Return whether the model is transient.
@@ -5510,27 +5494,6 @@ class BaseModel(metaclass=MetaModel):
         for name, value in values.items():
             self[name] = value
 
-    @api.model
-    def flush(self, fnames=None, records=None):
-        """ Process all the pending computations (on all models), and flush all
-        the pending updates to the database.
-
-        :param list[str] fnames: list of field names to flush.  If given,
-            limit the processing to the given fields of the current model.
-        :param Model records: if given (together with ``fnames``), limit the
-            processing to the given records.
-        """
-        warnings.warn(
-            "Deprecated method flush(), use flush_model(), flush_recordset() or env.flush_all() instead",
-            DeprecationWarning, stacklevel=2,
-        )
-        if fnames is None:
-            self.env.flush_all()
-        elif records is None:
-            self.flush_model(fnames)
-        else:
-            records.flush_recordset(fnames)
-
     def flush_model(self, fnames=None):
         """ Process the pending computations and database updates on ``self``'s
         model.  When the parameter is given, the method guarantees that at least
@@ -5875,36 +5838,6 @@ class BaseModel(metaclass=MetaModel):
         # the sake of code simplicity.
         return self.browse(ids)
 
-    @api.model
-    def refresh(self):
-        """ Clear the records cache.
-
-            .. deprecated:: 8.0
-                The record cache is automatically invalidated.
-        """
-        warnings.warn('refresh() is deprecated method, use invalidate_cache() instead',
-                      DeprecationWarning, stacklevel=2)
-        self.env.invalidate_all()
-
-    @api.model
-    def invalidate_cache(self, fnames=None, ids=None):
-        """ Invalidate the record caches after some records have been modified.
-            If both ``fnames`` and ``ids`` are ``None``, the whole cache is cleared.
-
-            :param fnames: the list of modified fields, or ``None`` for all fields
-            :param ids: the list of modified record ids, or ``None`` for all
-        """
-        warnings.warn(
-            "Deprecated method invalidate_cache(), use invalidate_model(), invalidate_recordset() or env.invalidate_all() instead",
-            DeprecationWarning, stacklevel=2
-        )
-        if ids is not None:
-            self.browse(ids).invalidate_recordset(fnames)
-        elif fnames is not None:
-            self.invalidate_model(fnames)
-        else:
-            self.env.invalidate_all()
-
     def invalidate_model(self, fnames=None, flush=True):
         """ Invalidate the cache of all records of ``self``'s model, when the
         cached values no longer correspond to the database values.  If the
@@ -6084,23 +6017,6 @@ class BaseModel(metaclass=MetaModel):
 
             yield from records._modified_triggers(subtree)
 
-    @api.model
-    def recompute(self, fnames=None, records=None):
-        """ Recompute all function fields (or the given ``fnames`` if present).
-            The fields and records to recompute have been determined by method
-            :meth:`modified`.
-        """
-        warnings.warn(
-            "Deprecated method recompute(), use flush_model(), flush_recordset() or env.flush_all() instead",
-            DeprecationWarning, stacklevel=2,
-        )
-        if fnames is None:
-            self.env._recompute_all()
-        elif records is None:
-            self._recompute_model(fnames)
-        else:
-            records._recompute_recordset(fnames)
-
     def _recompute_model(self, fnames=None):
         """ Process the pending computations of the fields of ``self``'s model.