From bbde79818fb07b807f448871e8e4305c513e1068 Mon Sep 17 00:00:00 2001
From: Abe Jellinek <jellinek@berkeley.edu>
Date: Fri, 24 May 2024 11:25:08 -0400
Subject: [PATCH] Snapshot resource handler: Use attachmentReaderType

isSnapshotAttachment() checks the link mode (needs to be IMPORTED_URL),
but we actually allow viewing all attachments with text/html MIME types
as "snapshots" in the reader. Don't throw on any HTML attachment that
the reader is willing to open.
---
 chrome/content/zotero/ZoteroProtocolHandler.jsm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/chrome/content/zotero/ZoteroProtocolHandler.jsm b/chrome/content/zotero/ZoteroProtocolHandler.jsm
index 1fb8f6e3cc..78c9018156 100644
--- a/chrome/content/zotero/ZoteroProtocolHandler.jsm
+++ b/chrome/content/zotero/ZoteroProtocolHandler.jsm
@@ -107,7 +107,7 @@ function ZoteroProtocolHandler() {
 						.slice(params.groupID !== undefined ? 4 : 3)
 						.filter(Boolean);
 					if (resourcePathParts.length) {
-						if (!item.isSnapshotAttachment()) {
+						if (item.attachmentReaderType !== 'snapshot') {
 							return this._errorChannel(`Item for ${uriPath} is not a snapshot attachment -- cannot access resources`);
 						}