Comment to route and initiated cache array in construct.

NatLibFi · Dec 4, 2024 · 27fde89 · 27fde89
1 parent 6b63f32
commit 27fde89
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/module/Finna/config/module.config.php b/module/Finna/config/module.config.php
@@ -1160,6 +1160,7 @@
         ],
     ],
 ];
+// @deprecated, use librarycards-displaybarcode
 $config['router']['routes']['barcode-show'] = [
     'type'    => 'Laminas\Router\Http\Segment',
     'options' => [

diff --git a/module/Finna/src/Finna/Controller/LibraryCardsController.php b/module/Finna/src/Finna/Controller/LibraryCardsController.php
@@ -66,6 +66,7 @@ public function __construct(
         protected SessionContainer $session,
     ) {
         parent::__construct($sm);
+        $this->session->LibraryCards ??= [];
     }
 
     /**
@@ -920,14 +921,14 @@ public function displayBarcodeAction(): mixed
             }
             $userCardService = $this->getDbService(UserCardServiceInterface::class);
             $card = $userCardService->getOrCreateLibraryCard($user, $id);
-            if (isset($this->session->LibraryCards[$id])) {
-                $barcode = $this->session->LibraryCards[$id];
-                return $this->createViewModel(['code' => $barcode]);
-            }
             $username = $card->getCatUsername();
             if (str_contains($username, '.')) {
                 [, $username] = explode('.', $username, 2);
             }
+            if (isset($this->session->LibraryCards[$username . '|' . $id])) {
+                $barcode = $this->session->LibraryCards[$username . '|' . $id];
+                return $this->createViewModel(['code' => $barcode]);
+            }
             $catalog = $this->getILS();
             $auth = $this->getILSAuthenticator();
             if ($card->getCatUsername() === $user->getCatUsername()) {
@@ -949,7 +950,7 @@ public function displayBarcodeAction(): mixed
                 }
             }
             $barcode ??= $username;
-            $this->session->LibraryCards[$id] = $barcode;
+            $this->session->LibraryCards[$username . '|' . $id] = $barcode;
             return $this->createViewModel(['code' => $barcode]);
         } catch (\Exception) {
             $this->flashMessenger()->addErrorMessage('An error has occurred');