Use FS path for {{{PageUrl}}} on local PDFs

- For uploaded local PDFs it would prev resolve the upload locator and thus the memex.cloud/ct/ URL
WorldBrain · Feb 27, 2024 · 3f7dd6f · 3f7dd6f
1 parent 91881f9
commit 3f7dd6f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/external/@worldbrain/memex-common b/external/@worldbrain/memex-common
diff --git a/src/copy-paster/background/template-data-fetchers.ts b/src/copy-paster/background/template-data-fetchers.ts
@@ -28,6 +28,7 @@ import type { ImageSupportInterface } from 'src/image-support/background/types'
 import type { CustomList } from '@worldbrain/memex-common/lib/types/core-data-types/client'
 import type { FollowedListEntry } from 'src/page-activity-indicator/background/types'
 import type { AutoPk } from '@worldbrain/memex-common/lib/storage/types'
+import { ContentLocatorType } from '@worldbrain/memex-common/lib/personal-cloud/storage/types'
 
 export function getTemplateDataFetchers({
  storageManager,
@@ -117,7 +118,10 @@ export function getTemplateDataFetchers({
  const pageLocators = allLocators.filter(
  (l) => l.normalizedUrl === page.url,
  )
- const mainLocator = pickBestLocator(pageLocators)
+ const mainLocator = pickBestLocator(pageLocators, {
+ ignoreUploadLocators: true,
+ priority: ContentLocatorType.Remote,
+ })
  fullUrl = mainLocator?.originalLocation
  ? encodeURI(mainLocator.originalLocation)
  : page.fullUrl