From 523370909d040403f47ebac4ec9d0ed870b3ac02 Mon Sep 17 00:00:00 2001
From: Beatrycze Volk <beatrycze.volk@slub-dresden.de>
Date: Wed, 12 Apr 2023 10:57:31 +0200
Subject: [PATCH] Remove TODO for passing state

It looks like the page and simultaneousPages can be undefined, so better to avoid overwrite data with undefined
---
 Resources/Public/JavaScript/PageView/Controller.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Resources/Public/JavaScript/PageView/Controller.js b/Resources/Public/JavaScript/PageView/Controller.js
index 60391fefd..49496dfe7 100644
--- a/Resources/Public/JavaScript/PageView/Controller.js
+++ b/Resources/Public/JavaScript/PageView/Controller.js
@@ -124,7 +124,6 @@ class dlfController {
      * @param {dlf.StateChangeDetail} detail
      */
     changeState(detail) {
-        // TODO(client-side): Consider passing full new state in stateChanged event, then reduce usage of currentPageNo and simultaneousPages properties
         if (detail.page !== undefined) {
             this.doc.state.page = detail.page;
         }