From c87413e8add2eb558b98d76c4378a01106d7ee9a Mon Sep 17 00:00:00 2001
From: Debao Zhang <hello@debao.me>
Date: Wed, 30 Oct 2013 13:37:44 +0800
Subject: [PATCH] Fix defineName bug

sheetId is used in application inside, when save to/load from xml file,
which will convert to/from localSheetId
---
 src/xlsx/xlsxworkbook.cpp | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/xlsx/xlsxworkbook.cpp b/src/xlsx/xlsxworkbook.cpp
index 168351e..fb0e161 100755
--- a/src/xlsx/xlsxworkbook.cpp
+++ b/src/xlsx/xlsxworkbook.cpp
@@ -142,10 +142,12 @@ bool Workbook::defineName(const QString &name, const QString &formula, const QSt
         formulaString = formula.mid(1);
 
     int id=-1;
-    for (int i=0; i<d->worksheets.size(); ++i) {
-        if (d->worksheets[i]->sheetName() == scope) {
-            id = i;
-            break;
+    if (!scope.isEmpty()) {
+        for (int i=0; i<d->worksheets.size(); ++i) {
+            if (d->worksheets[i]->sheetName() == scope) {
+                id = d->worksheets[i]->sheetId();
+                break;
+            }
         }
     }