From d514f9b689602c73110d5c73b8941829af4c6ec0 Mon Sep 17 00:00:00 2001
From: Nick Burch <nick@apache.org>
Date: Sun, 14 Sep 2008 22:19:06 +0000
Subject: [PATCH] Make a start on exposing the StyleRecord details into
 HSSFCellStyle, but not fully there yet

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@695303 13f79535-47bb-0310-9956-ffa450edef68
---
 .../org/apache/poi/hssf/model/Workbook.java   |  55 ++++++++++++++++++
 .../apache/poi/hssf/record/StyleRecord.java   |   5 +-
 .../poi/hssf/usermodel/HSSFCellStyle.java     |  32 ++++++++++
 .../poi/hssf/data/WithExtendedStyles.xls      | Bin 0 -> 6656 bytes
 .../poi/hssf/usermodel/TestCellStyle.java     |  36 ++++++++++++
 5 files changed, 127 insertions(+), 1 deletion(-)
 create mode 100644 src/testcases/org/apache/poi/hssf/data/WithExtendedStyles.xls

diff --git a/src/java/org/apache/poi/hssf/model/Workbook.java b/src/java/org/apache/poi/hssf/model/Workbook.java
index f8a426108a..1e37a80ca2 100644
--- a/src/java/org/apache/poi/hssf/model/Workbook.java
+++ b/src/java/org/apache/poi/hssf/model/Workbook.java
@@ -773,6 +773,61 @@ public final class Workbook implements Model {
         numxfs++;
         return xf;
     }
+    
+    /**
+     * Returns the StyleRecord for the given
+     *  xfIndex, or null if that ExtendedFormat doesn't
+     *  have a Style set.
+     */
+    public StyleRecord getStyleRecord(int xfIndex) {
+    	// Style records always follow after 
+    	//  the ExtendedFormat records
+    	boolean done = false;
+    	for(int i=records.getXfpos(); i<records.size() &&
+    			!done; i++) {
+    		Record r = records.get(i);
+    		if(r instanceof ExtendedFormatRecord) {
+    		} else if(r instanceof StyleRecord) {
+    			StyleRecord sr = (StyleRecord)r;
+    			if(sr.getIndex() == xfIndex) {
+    				return sr;
+    			}
+    		} else {
+    			done = true;
+    		}
+    	}
+    	return null;
+    }
+    /**
+     * Creates a new StyleRecord, for the given Extended
+     *  Format index, and adds it onto the end of the
+     *  records collection
+     */
+    public StyleRecord createStyleRecord(int xfIndex) {
+    	// Style records always follow after 
+    	//  the ExtendedFormat records
+    	StyleRecord newSR = new StyleRecord();
+    	newSR.setIndex((short)xfIndex);
+    	
+    	// Find the spot
+    	int addAt = -1;
+    	for(int i=records.getXfpos(); i<records.size() &&
+    			addAt == -1; i++) {
+    		Record r = records.get(i);
+    		if(r instanceof ExtendedFormatRecord ||
+    				r instanceof StyleRecord) {
+    			// Keep going
+    		} else {
+    			addAt = i;
+    		}
+    	}
+    	if(addAt == -1) {
+    		throw new IllegalStateException("No XF Records found!");
+    	}
+    	records.add(addAt, newSR);
+    	
+    	return newSR;
+    }
 
     /**
      * Adds a string to the SST table and returns its index (if its a duplicate
diff --git a/src/java/org/apache/poi/hssf/record/StyleRecord.java b/src/java/org/apache/poi/hssf/record/StyleRecord.java
index 6990a5efdc..c6591f550d 100644
--- a/src/java/org/apache/poi/hssf/record/StyleRecord.java
+++ b/src/java/org/apache/poi/hssf/record/StyleRecord.java
@@ -160,7 +160,10 @@ public final class StyleRecord extends Record {
     public void setName(String name)
     {
         field_4_name = name;
-        //TODO set name length and string options
+        
+        // Fix up the length
+        field_2_name_length = (short)name.length();
+        //TODO set name string options
     }
 
     // end user defined
diff --git a/src/java/org/apache/poi/hssf/usermodel/HSSFCellStyle.java b/src/java/org/apache/poi/hssf/usermodel/HSSFCellStyle.java
index 6d8ae33b68..f74c38d09e 100644
--- a/src/java/org/apache/poi/hssf/usermodel/HSSFCellStyle.java
+++ b/src/java/org/apache/poi/hssf/usermodel/HSSFCellStyle.java
@@ -21,6 +21,7 @@ package org.apache.poi.hssf.usermodel;
 import org.apache.poi.hssf.model.Workbook;
 import org.apache.poi.hssf.record.ExtendedFormatRecord;
 import org.apache.poi.hssf.record.FontRecord;
+import org.apache.poi.hssf.record.StyleRecord;
 import org.apache.poi.hssf.util.HSSFColor;
 
 /**
@@ -930,6 +931,37 @@ public class HSSFCellStyle
     {
         return format.getFillForeground();
     }
+    
+    /**
+     * Gets the name of the user defined style.
+     * Returns null for built in styles, and
+     *  styles where no name has been defined
+     */
+    public String getUserStyleName() {
+    	StyleRecord sr = workbook.getStyleRecord(index);
+    	if(sr == null) {
+    		return null;
+    	}
+    	if(sr.getType() == StyleRecord.STYLE_BUILT_IN) {
+    		return null;
+    	}
+    	return sr.getName();
+    }
+    
+    /**
+     * Sets the name of the user defined style.
+     * Will complain if you try this on a built in style.
+     */
+    public void setUserStyleName(String styleName) {
+    	StyleRecord sr = workbook.getStyleRecord(index);
+    	if(sr == null) {
+    		sr = workbook.createStyleRecord(index);
+    	}
+    	if(sr.getType() == StyleRecord.STYLE_BUILT_IN) {
+    		throw new IllegalArgumentException("Unable to set user specified style names for built in styles!");
+    	}
+    	sr.setName(styleName);
+    }
 
     /**
      * Verifies that this style belongs to the supplied Workbook.
diff --git a/src/testcases/org/apache/poi/hssf/data/WithExtendedStyles.xls b/src/testcases/org/apache/poi/hssf/data/WithExtendedStyles.xls
new file mode 100644
index 0000000000000000000000000000000000000000..4a80f56f5dde69729ab1070e6d0f509cd2213622
GIT binary patch
literal 6656
zcmeHLU2IfE6h3#`ExYt@`$IvHda1Jg+alY7VhnamsX`)9Thtd4-L|*TvZbco#$sYz
z3cP88!NiwFL(&Hgi9gY3j3$=1nqV|0B<NEyhJ@(DViZHtcKyD&_qLaH*X}lrirI7X
z&CESBbI;72Gw00QZ@(#<z3}12Ym!8dN}envyi(xcYbdX^-+dx3l*wC2Boa0o#T?y5
zD==@3l{(Ca65RlGz6S6BUVwFlR(=;vEjQE*R-j9U@kB+J4mp8*L{37}nNt-k#wjL~
zo=$&}N&Ds6<m4}{z;bKFdguDnf76fW0qOQXeLnYp5l{@20HpvzfRq8{Km||<tOKfm
z^}q(;9$+J|3Ah*73{(U60Xnb+r~&Q=9ssriexMep1L}bWpb-cF9Q!saXE}&+Gtgr3
zc3Sz;isVXvw*p=dv{>Ml&x<e9<tyKR4Hof1{v~A>KZ18eUZ8EXha<iEKSg`g7{VHD
zbGKH2bA}5xI*GIdHbri^N-I?QzH0Y{N<)hCno4&o+eG7#=(~{dyz&+7jr~iIYVxV-
z^Mm39VW~5yNxX8-+7@oD4IXho!V;0BTC`iu+K(M_wZD8hp0>6+zeuJSt)J*P*wNMY
z<gD6-Q}TO4M<1J}b@DkW#s4icK36GK`*eJMHC6lnFMkSas&jf4+3VbBt^?Jk$mZ9l
z;Rn+28`AK-Dfw<#LaOyINW*9Qr!-w;&K#wPIZn0@C-Kj_-~nQiVD5|Msk|&!pz`uq
zzRD|NYgAqtb0aTAEWk-}G<q^(1o>LE_}ctDcAX|?u?Giv;6z3DF~f)k$$882LyFT(
zP7e>EcAJ)0;fsQJewIN-p?D7~r6A5|p&>+JoGO|zqS4|BS}b#$*CK^+My)S##`VZ4
z4FWvVS_o-Ks&6u`A<lX5F%ROH=MRLsIYb}!CQ#RB^oPeHQC-ar1McQ5_26QuIf0v!
zM|NO#jGBy~rl<AY;c1I7vOdg$t5K##=|iu%cW-`opr^C-ImM5GW7Ae@L9WXw)LhDM
zP(2z|I-%NDB8F~30B!fz93B`lM)iZni+b1aQ20ck%Q!j~36IqHgU$Z7b%?8ovdgsV
z3?DUgf9Gi>;EYQB7)*<z8?CjDavjR2o32hYU5&TaJIWW}Y`vJGrk)82XaUXcZ|9mk
zjc*sq%WA1ad!vwt3y_Nk4Rgs&6?$A$4OMwv<)W1ed8Uf*<e4h3Vih`+%Di=THNVX_
zR;^d5=3;=+@J2p+Zcynj(n;mS_kjb&#Hk4NL1>{=^5Wa^UMl&f>4`UPjt)!*xn!;C
zN4uo0KjtczPp~gULQE{$%<q|HbHq8D!D>(^t`*sgy{;lo9qLXULYZ0X5PcSPs6>R8
z1mS3#)1mDH?0vubI^6EmA$)7rIz*qWb?DG?Qc=T=BoFSLXKbH`u0v#6rSdrMJO>7P
zM}|j-`=fftc&`!BA9=XxaJa{aL<~kl*)!1JZyrp|K3S*m7N*dwdzu{<@aK-3ESczq
zX3xI;>+M6wd>3E$NMr3s-_zakoNBwleJC>?by~bW<P6p003+TRfcM680Hfd}!29Dn
z043g-H}8UM+cu8H=^7q2Q0DzG6_0&**9!r$&wLk=sWNj?2#CFO`I()!W*EywZp{=w
zgfffiDBb)i%XQ`JE0fzc`Od$I`3G*lw--Ej67L9jyysE~!4SZ{oLf3Rs{b>zn*V>)
z{FC-Ts);ht#b^kiXYu!GM2<_3`VV#7WM+}_7RdFc9dZw5+SO0P1ZmO^$wRo|9>N#~
zO=cD;<#RriP0FNe6B_Ne8fuUm5nBDqh{qLEvYBS`l?Ty?|7u8X>6S6fe+YM6{`=dH
zzkvNpC*(wN_Z<c&J=IzA=~6SDKmEv0)+gPv7ZQ5a7!36XGQ08YTEYBxKz7uUzZzM#
M>+k&k_HWDo6LiIw2LJ#7

literal 0
HcmV?d00001

diff --git a/src/testcases/org/apache/poi/hssf/usermodel/TestCellStyle.java b/src/testcases/org/apache/poi/hssf/usermodel/TestCellStyle.java
index 1dc826cc6f..e2783b4434 100644
--- a/src/testcases/org/apache/poi/hssf/usermodel/TestCellStyle.java
+++ b/src/testcases/org/apache/poi/hssf/usermodel/TestCellStyle.java
@@ -29,6 +29,8 @@ import java.io.*;
 import java.util.*;
 
 import junit.framework.*;
+
+import org.apache.poi.hssf.HSSFTestDataSamples;
 import org.apache.poi.util.TempFile;
 
 /**
@@ -41,6 +43,10 @@ public class TestCellStyle
     extends TestCase
 {
 
+    private static HSSFWorkbook openSample(String sampleFileName) {
+        return HSSFTestDataSamples.openSampleWorkbook(sampleFileName);
+    }
+
     /** Creates a new instance of TestCellStyle */
 
     public TestCellStyle(String name)
@@ -303,6 +309,36 @@ public class TestCellStyle
     	assertFalse(fmtClone.getFormat("Test##") == fmt.getFormat("Test##"));
     	assertEquals(5, wbClone.getNumberOfFonts());
     }
+    
+    public void testStyleNames() throws Exception {
+        HSSFWorkbook wb = openSample("WithExtendedStyles.xls");
+    	HSSFSheet s = wb.getSheetAt(0);
+    	HSSFCell c1 = s.getRow(0).getCell(0);
+    	HSSFCell c2 = s.getRow(1).getCell(0);
+    	HSSFCell c3 = s.getRow(2).getCell(0);
+    	
+    	HSSFCellStyle cs1 = c1.getCellStyle();
+    	HSSFCellStyle cs2 = c2.getCellStyle();
+    	HSSFCellStyle cs3 = c3.getCellStyle();
+    	
+    	assertNotNull(cs1);
+    	assertNotNull(cs2);
+    	assertNotNull(cs3);
+    	
+    	// Check we got the styles we'd expect
+    	assertEquals(10, cs1.getFont(wb).getFontHeightInPoints());
+    	assertEquals(9,  cs2.getFont(wb).getFontHeightInPoints());
+    	assertEquals(12, cs3.getFont(wb).getFontHeightInPoints());
+    	
+    	assertEquals(15, cs1.getIndex());
+    	assertEquals(23, cs2.getIndex());
+    	assertEquals(24, cs3.getIndex());
+    	
+    	// Now check the style names
+//    	assertEquals(null, cs1.getUserStyleName());
+//    	assertEquals("style1", cs2.getUserStyleName());
+//    	assertEquals("style2", cs3.getUserStyleName());
+    }
 
     public static void main(String [] ignored_args)
     {