Update the XSLF slide master text extraction to be optional, as HSLF already is, plus fix some indents

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

src/documentation/content/xdocs/status.xml

@@ -34,7 +34,7 @@
 
     <changes>
         <release version="3.8-beta5" date="2011-??-??">
-           <action dev="poi-developers" type="add">51804 - include Master Slide text in XSLF text extraction</action>
+           <action dev="poi-developers" type="add">51804 - optionally include Master Slide text in XSLF text extraction, as HSLF already offers</action>
            <action dev="poi-developers" type="add">New PackagePart method getRelatedPart(PackageRelationship) to simplify navigation of relations between OPC Parts</action>
            <action dev="poi-developers" type="fix">51832 - handle XLS files where the WRITEPROTECT record preceeds the FILEPASS one, rather than following as normal</action>
            <action dev="poi-developers" type="fix">51809 - correct GTE handling in COUNTIF</action>

src/ooxml/java/org/apache/poi/xslf/extractor/XSLFPowerPointExtractor.java

@@ -45,6 +45,7 @@ public class XSLFPowerPointExtractor extends POIXMLTextExtractor {
 	private XMLSlideShow slideshow;
 	private boolean slidesByDefault = true;
 	private boolean notesByDefault = false;
+   private boolean masterByDefault = false;
 	
 	public XSLFPowerPointExtractor(XMLSlideShow slideshow) {
 		super(slideshow);
@@ -84,6 +85,13 @@ public class XSLFPowerPointExtractor extends POIXMLTextExtractor {
 		this.notesByDefault = notesByDefault;
 	}
 	
+   /**
+    * Should a call to getText() return text from master? Default is no
+    */
+   public void setMasterByDefault(boolean masterByDefault) {
+       this.masterByDefault = masterByDefault;
+   }
+	
 	/**
 	 * Gets the slide text, but not the notes text
 	 */
@@ -97,6 +105,16 @@ public class XSLFPowerPointExtractor extends POIXMLTextExtractor {
     * @param notesText Should we retrieve text from notes?
     */
    public String getText(boolean slideText, boolean notesText) {
+      return getText(slideText, notesText, masterByDefault);
+   }
+   
+   /**
+    * Gets the requested text from the file
+    * @param slideText Should we retrieve text from slides?
+    * @param notesText Should we retrieve text from notes?
+    * @param masterText Should we retrieve text from master slides?
+    */
+   public String getText(boolean slideText, boolean notesText, boolean masterText) {
       StringBuffer text = new StringBuffer();
 
       XSLFSlide[] slides = slideshow.getSlides();
@@ -115,8 +133,8 @@ public class XSLFPowerPointExtractor extends POIXMLTextExtractor {
             if (slideText) {
                extractText(slide.getCommonSlideData(), text);
                
-               // If there's a master sheet, grab text from there
+               // If there's a master sheet and it's requested, grab text from there
-               if(master != null) {
+               if(masterText && master != null) {
                   extractText(master.getCommonSlideData(), text);
                }
 

src/ooxml/testcases/org/apache/poi/xslf/extractor/TestXSLFPowerPointExtractor.java

@@ -68,7 +68,45 @@ public class TestXSLFPowerPointExtractor extends TestCase {
          "Fifth level\n";
 		
 		// Just slides, no notes
-		text = extractor.getText(true, false);
+		text = extractor.getText(true, false, false);
+		assertEquals(
+				"Lorem ipsum dolor sit amet\n" +
+				"Nunc at risus vel erat tempus posuere. Aenean non ante.\n" +
+				"\n" +
+				"Lorem ipsum dolor sit amet\n" +
+				"Lorem\n" +
+				"ipsum\n" +
+				"dolor\n" +
+				"sit\n" +
+				"amet\n" +
+				"\n"
+				, text
+		);
+		
+		// Just notes, no slides
+		text = extractor.getText(false, true);
+		assertEquals(
+				"\n\n\n\n", text
+		);
+		
+		// Both
+		text = extractor.getText(true, true, false);
+		assertEquals(
+				"Lorem ipsum dolor sit amet\n" +
+				"Nunc at risus vel erat tempus posuere. Aenean non ante.\n" +
+            "\n\n\n" +
+				"Lorem ipsum dolor sit amet\n" +
+				"Lorem\n" +
+				"ipsum\n" +
+				"dolor\n" +
+				"sit\n" +
+				"amet\n" +
+				"\n\n\n"
+				, text
+		);
+		
+		// With Slides and Master Text
+      text = extractor.getText(true, false, true);
       assertEquals(
             "Lorem ipsum dolor sit amet\n" +
             "Nunc at risus vel erat tempus posuere. Aenean non ante.\n" +
@@ -87,14 +125,8 @@ public class TestXSLFPowerPointExtractor extends TestCase {
             , text
       );
 		
-		// Just notes, no slides
+		// With Slides, Notes and Master Text
-		text = extractor.getText(false, true);
+      text = extractor.getText(true, true, true);
-		assertEquals(
-				"\n\n\n\n", text
-		);
-		
-		// Both
-		text = extractor.getText(true, true);
       assertEquals(
             "Lorem ipsum dolor sit amet\n" +
             "Nunc at risus vel erat tempus posuere. Aenean non ante.\n" +