From f1f623888495facd4fb65f5cfc184eac9fffa9f0 Mon Sep 17 00:00:00 2001
From: Pierre-Yves David <pierre-yves.david@ens-lyon.org>
Date: Wed, 15 May 2024 03:01:00 +0200
Subject: [PATCH] DiskBackedContent: add a small temporary compatibility layer
There are two other package using DiskBackedContent "swh-loader-svn" and
"swh-loader-cvs". Both use it to check "DiskBackedContent.object_type"
at the same time as "model.Content.object_type".
so we do this small hack to avoid breaking these other module until
they migrate.
---
requirements-test.txt | 1 +
swh/model/from_disk.py | 11 +++++++++++
2 files changed, 12 insertions(+)
diff --git a/requirements-test.txt b/requirements-test.txt
index 593c9e62..96866990 100644
--- a/requirements-test.txt
+++ b/requirements-test.txt
@@ -5,3 +5,4 @@ pytz
types-click
types-python-dateutil
types-pytz
+types-deprecated
diff --git a/swh/model/from_disk.py b/swh/model/from_disk.py
index d5d24142..160ea739 100644
--- a/swh/model/from_disk.py
+++ b/swh/model/from_disk.py
@@ -32,6 +32,7 @@ from typing import (
import warnings
import attr
+from deprecated import deprecated
from typing_extensions import Final
from . import model
@@ -65,6 +66,16 @@ class FromDiskType(model._StringCompatibleEnum):
return hash(str(self.value))
+# There is an handful of other module that test for
+# DiskBackedContent.object_type in conjunction of Content.object_type, give
+# them a hand to let them handle compatibility in a smoother way.
+#
+# Remove this compatibility trick once this user have been migrated
+DiskBackedContent = deprecated(
+ version="v6.13.0", reason="Use model.Content.object_type instead"
+)(model.Content)
+
+
@attr.s(frozen=True, slots=True)
class DiskBackedData:
path = attr.ib(type=bytes)
--
GitLab