From 6f321081c49bc0f96165b30afe823f2534f0df4a Mon Sep 17 00:00:00 2001 From: Miltiadis Vasilakis Date: Thu, 9 May 2024 10:11:24 +0300 Subject: [PATCH] Refactor PDFReaderState --- Zotero/Scenes/Detail/PDF/Models/PDFReaderState.swift | 6 +++--- Zotero/Scenes/Detail/PDF/PDFCoordinator.swift | 2 +- .../Scenes/Detail/PDF/Views/PDFReaderViewController.swift | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/Zotero/Scenes/Detail/PDF/Models/PDFReaderState.swift b/Zotero/Scenes/Detail/PDF/Models/PDFReaderState.swift index d5d70d0aa..77fc3e6e5 100644 --- a/Zotero/Scenes/Detail/PDF/Models/PDFReaderState.swift +++ b/Zotero/Scenes/Detail/PDF/Models/PDFReaderState.swift @@ -67,7 +67,7 @@ struct PDFReaderState: ViewModelState { let key: String let parentKey: String? let document: PSPDFKit.Document - let activityTitle: String? + let title: String? let previewCache: NSCache let commentFont: UIFont let userId: Int @@ -129,7 +129,7 @@ struct PDFReaderState: ViewModelState { url: URL, key: String, parentKey: String?, - activityTitle: String?, + title: String?, libraryId: LibraryIdentifier, initialPage: Int?, preselectedAnnotationKey: String?, @@ -143,7 +143,7 @@ struct PDFReaderState: ViewModelState { self.key = key self.parentKey = parentKey self.document = Document(url: url) - self.activityTitle = activityTitle + self.title = title self.previewCache = NSCache() self.commentFont = PDFReaderLayout.annotationLayout.font self.userId = userId diff --git a/Zotero/Scenes/Detail/PDF/PDFCoordinator.swift b/Zotero/Scenes/Detail/PDF/PDFCoordinator.swift index 7f0989441..c7ebb925e 100644 --- a/Zotero/Scenes/Detail/PDF/PDFCoordinator.swift +++ b/Zotero/Scenes/Detail/PDF/PDFCoordinator.swift @@ -134,7 +134,7 @@ final class PDFCoordinator: Coordinator { url: self.url, key: self.key, parentKey: self.parentKey, - activityTitle: try? controllers.userControllers?.dbStorage.perform(request: ReadFilenameDbRequest(libraryId: libraryId, key: key), on: .main), + title: try? controllers.userControllers?.dbStorage.perform(request: ReadFilenameDbRequest(libraryId: libraryId, key: key), on: .main), libraryId: self.libraryId, initialPage: self.page, preselectedAnnotationKey: self.preselectedAnnotationKey, diff --git a/Zotero/Scenes/Detail/PDF/Views/PDFReaderViewController.swift b/Zotero/Scenes/Detail/PDF/Views/PDFReaderViewController.swift index 13f0aeff4..412fc2609 100644 --- a/Zotero/Scenes/Detail/PDF/Views/PDFReaderViewController.swift +++ b/Zotero/Scenes/Detail/PDF/Views/PDFReaderViewController.swift @@ -121,7 +121,7 @@ class PDFReaderViewController: UIViewController { if changedCurrentItem { close(dismiss: false) } else if openItemsChanged { - openItemsController.setOpenItemsUserActivity(from: self, libraryId: viewModel.state.library.identifier, title: viewModel.state.activityTitle) + openItemsController.setOpenItemsUserActivity(from: self, libraryId: viewModel.state.library.identifier, title: viewModel.state.title) } } ) @@ -225,7 +225,7 @@ class PDFReaderViewController: UIViewController { override func viewDidLoad() { super.viewDidLoad() - openItemsController.setOpenItemsUserActivity(from: self, libraryId: viewModel.state.library.identifier, title: viewModel.state.activityTitle) + openItemsController.setOpenItemsUserActivity(from: self, libraryId: viewModel.state.library.identifier, title: viewModel.state.title) view.backgroundColor = .systemGray6 // Create intraDocumentNavigationHandler before setting up views, as it may be called by a child view controller, before view has finished loading. intraDocumentNavigationHandler = IntraDocumentNavigationButtonsHandler(