Merge pull request #18547 from mvdbeek/fix_displayable_deleted_request

[24.1] Also check dataset.deleted when determining if data can be displayed
This commit is contained in:
Marius van den Beek
2024-07-17 12:13:35 +02:00
committed by GitHub
3 changed files with 9 additions and 2 deletions
+2 -1
View File
@@ -193,7 +193,8 @@ class Interval(Tabular):
def displayable(self, dataset: DatasetProtocol) -> bool:
try:
return (
not dataset.dataset.purged
not dataset.deleted
and not dataset.dataset.purged
and dataset.has_data()
and dataset.state == dataset.states.OK
and dataset.metadata.columns > 0
+5
View File
@@ -16,6 +16,10 @@ class HasCreatingJob(Protocol):
def creating_job(self): ...
class HasDeleted(Protocol):
deleted: bool
class HasExt(Protocol):
@property
def ext(self): ...
@@ -55,6 +59,7 @@ class HasExtraFilesAndMetadata(HasExtraFilesPath, HasMetadata, Protocol): ...
class DatasetProtocol(
HasCreatingJob,
HasDeleted,
HasExt,
HasExtraFilesPath,
HasFileName,
+2 -1
View File
@@ -134,7 +134,8 @@ class TabularData(Text):
def displayable(self, dataset: DatasetProtocol) -> bool:
try:
return (
not dataset.dataset.purged
not dataset.deleted
and not dataset.dataset.purged
and dataset.has_data()
and dataset.state == dataset.states.OK
and dataset.metadata.columns > 0