From 220760ae3baef38a696b293bbb1e73471b900274 Mon Sep 17 00:00:00 2001
From: Vincent Vanwaelscappel <vincent@cubedesigners.com>
Date: Fri, 28 Apr 2023 11:24:34 +0200
Subject: [PATCH] wait #5889 @0:10

---
 app/Fluidbook/Links.php | 51 ++++++++++++++++++++++-------------------
 1 file changed, 28 insertions(+), 23 deletions(-)

diff --git a/app/Fluidbook/Links.php b/app/Fluidbook/Links.php
index 95e4959a8..4b8e1a817 100644
--- a/app/Fluidbook/Links.php
+++ b/app/Fluidbook/Links.php
@@ -305,13 +305,15 @@ class Links
 
         $rulers = array();
         $links = array();
-        
+
         if ($xls->getSheetCount() === 1) {
             $s = $xls->setActiveSheetIndex(0);
-        } else if (null === $xls->getSheetByName('Links')) {
-            $s = $xls->setActiveSheetIndexByName('Links');
         } else {
-            throw new \Exception(__('Format du fichier Excel invalide (pas de feuille nommée :name)', ['name' => 'Links']));
+            try {
+                $s = $xls->setActiveSheetIndexByName('Links');
+            } catch (\Exception $e) {
+                throw new \Exception(__('Format du fichier Excel invalide (pas de feuille nommée :name)', ['name' => 'Links']));
+            }
         }
         $i = 0;
 
@@ -343,30 +345,33 @@ class Links
         self::_fixLinks($links);
 
 
-        if ($xls->getSheetCount() === 1 || null === $xls->getSheetByName('Rulers')) {
+        if ($xls->getSheetCount() === 1) {
             return;
         }
         $i = 0;
-        $s = $xls->setActiveSheetIndexByName('Rulers');
-        foreach ($s->getRowIterator() as $row) {
-            $cellIterator = $row->getCellIterator();
-            $cellIterator->setIterateOnlyExistingCells(false);
-            if ($i == 0) {
-                $cols = array();
-                foreach ($cellIterator as $cell) {
-                    $cols[] = $cell->getValue();
-                }
-            } else {
-                $link = array();
-                $j = 0;
-                foreach ($cellIterator as $cell) {
-                    $ruler[$cols[$j]] = $cell->getValue();
-                    $j++;
-                }
+        try {
+            $s = $xls->setActiveSheetIndexByName('Rulers');
+            foreach ($s->getRowIterator() as $row) {
+                $cellIterator = $row->getCellIterator();
+                $cellIterator->setIterateOnlyExistingCells(false);
+                if ($i == 0) {
+                    $cols = array();
+                    foreach ($cellIterator as $cell) {
+                        $cols[] = $cell->getValue();
+                    }
+                } else {
+                    $j = 0;
+                    foreach ($cellIterator as $cell) {
+                        $ruler[$cols[$j]] = $cell->getValue();
+                        $j++;
+                    }
 
-                $rulers[] = $ruler;
+                    $rulers[] = $ruler;
+                }
+                $i++;
             }
-            $i++;
+        } catch (\Exception $e) {
+
         }
     }
 
-- 
2.39.5