From: Dominik Stadler <centic@apache.org>
Date: Sun, 28 Dec 2014 09:08:58 +0000 (+0000)
Subject: Ensure that XSSFWorkbooks contain a BookView, some corrupt ones do not contain this... 
X-Git-Tag: REL_3_12_BETA1~67
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=9c41ad8b677edd78c5d92aee8aba7acbf6c44b1a;p=poi.git

Ensure that XSSFWorkbooks contain a BookView, some corrupt ones do not contain this, e.g. ones created by movescound.com

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1648158 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java b/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
index f3efe94663..ec23f209be 100644
--- a/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
+++ b/src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java
@@ -238,6 +238,13 @@ public class XSSFWorkbook extends POIXMLDocument implements Workbook, Iterable<X
         
         // Build a tree of POIXMLDocumentParts, this workbook being the root
         load(XSSFFactory.getInstance());
+        
+        // some broken Workbooks miss this...
+        if(!workbook.isSetBookViews()) {
+            CTBookViews bvs = workbook.addNewBookViews();
+            CTBookView bv = bvs.addNewWorkbookView();
+            bv.setActiveTab(0);
+        }
     }
 
     /**
@@ -261,6 +268,13 @@ public class XSSFWorkbook extends POIXMLDocument implements Workbook, Iterable<X
         
         // Build a tree of POIXMLDocumentParts, this workbook being the root
         load(XSSFFactory.getInstance());
+
+        // some broken Workbooks miss this...
+        if(!workbook.isSetBookViews()) {
+            CTBookViews bvs = workbook.addNewBookViews();
+            CTBookView bv = bvs.addNewWorkbookView();
+            bv.setActiveTab(0);
+        }
     }
 
     /**