Skip to content

Commit 2925038

Browse files
committed
Refactor PDFReaderState
1 parent 68c3a60 commit 2925038

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

Zotero/Scenes/Detail/PDF/Models/PDFReaderState.swift

+3-3
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ struct PDFReaderState: ViewModelState {
6868
let key: String
6969
let parentKey: String?
7070
let document: PSPDFKit.Document
71-
let activityTitle: String?
71+
let title: String?
7272
let previewCache: NSCache<NSString, UIImage>
7373
let commentFont: UIFont
7474
let userId: Int
@@ -131,7 +131,7 @@ struct PDFReaderState: ViewModelState {
131131
url: URL,
132132
key: String,
133133
parentKey: String?,
134-
activityTitle: String?,
134+
title: String?,
135135
libraryId: LibraryIdentifier,
136136
initialPage: Int?,
137137
preselectedAnnotationKey: String?,
@@ -145,7 +145,7 @@ struct PDFReaderState: ViewModelState {
145145
self.key = key
146146
self.parentKey = parentKey
147147
self.document = Document(url: url)
148-
self.activityTitle = activityTitle
148+
self.title = title
149149
self.previewCache = NSCache()
150150
self.commentFont = PDFReaderLayout.annotationLayout.font
151151
self.userId = userId

Zotero/Scenes/Detail/PDF/PDFCoordinator.swift

+1-1
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ final class PDFCoordinator: Coordinator {
135135
url: self.url,
136136
key: self.key,
137137
parentKey: self.parentKey,
138-
activityTitle: try? controllers.userControllers?.dbStorage.perform(request: ReadFilenameDbRequest(libraryId: libraryId, key: key), on: .main),
138+
title: try? controllers.userControllers?.dbStorage.perform(request: ReadFilenameDbRequest(libraryId: libraryId, key: key), on: .main),
139139
libraryId: self.libraryId,
140140
initialPage: self.page,
141141
preselectedAnnotationKey: self.preselectedAnnotationKey,

Zotero/Scenes/Detail/PDF/Views/PDFReaderViewController.swift

+2-2
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ class PDFReaderViewController: UIViewController {
121121
if changedCurrentItem {
122122
close(dismiss: false)
123123
} else if openItemsChanged {
124-
openItemsController.setOpenItemsUserActivity(from: self, libraryId: viewModel.state.library.identifier, title: viewModel.state.activityTitle)
124+
openItemsController.setOpenItemsUserActivity(from: self, libraryId: viewModel.state.library.identifier, title: viewModel.state.title)
125125
}
126126
}
127127
)
@@ -225,7 +225,7 @@ class PDFReaderViewController: UIViewController {
225225
override func viewDidLoad() {
226226
super.viewDidLoad()
227227

228-
openItemsController.setOpenItemsUserActivity(from: self, libraryId: viewModel.state.library.identifier, title: viewModel.state.activityTitle)
228+
openItemsController.setOpenItemsUserActivity(from: self, libraryId: viewModel.state.library.identifier, title: viewModel.state.title)
229229
view.backgroundColor = .systemGray6
230230
// Create intraDocumentNavigationHandler before setting up views, as it may be called by a child view controller, before view has finished loading.
231231
intraDocumentNavigationHandler = IntraDocumentNavigationButtonsHandler(

0 commit comments

Comments
 (0)