fix(RemoteFile): Don't try to delete unavailable S3 file

press/press/doctype/remote_file/remote_file.py

@@ -1,4 +1,4 @@
 # -*- coding: utf-8 -*-
 # Copyright (c) 2020, Frappe and contributors
 # For license information, please see license.txt
 
@@ -128,7 +128,7 @@
 	"""Delete specified objects identified by keys in the backups bucket."""
 	remote_files = list(set([x for x in remote_files if x]))
 	if not remote_files:
 		return
 
 	buckets = {bucket: [] for bucket in frappe.get_all("Backup Bucket", pluck="name")}
 	buckets.update({frappe.db.get_single_value("Press Settings", "aws_s3_bucket"): []})
@@ -159,14 +159,14 @@
 	if TYPE_CHECKING:
 		from frappe.types import DF
 
 		bucket: DF.Data | None
 		file_name: DF.Data | None
 		file_path: DF.Text | None
 		file_size: DF.Data | None
 		file_type: DF.Data | None
 		site: DF.Link | None
 		status: DF.Literal["Available", "Unavailable"]
 		url: DF.Code | None
 	# end: auto-generated types
 
 	@property
@@ -174,7 +174,7 @@
 		if not self.bucket:
 			return None
 
 		elif self.bucket == frappe.db.get_single_value(
 			"Press Settings", "remote_uploads_bucket"
 		):
 			access_key_id = frappe.db.get_single_value("Press Settings", "remote_access_key_id")
@@ -231,7 +231,8 @@
 		)
 
 	def on_trash(self):
-		self.delete_remote_object()
+		if self.status != "Unavailable":
+			self.delete_remote_object()
 
 	@frappe.whitelist()
 	def get_download_link(self):