From 47600ff855e7855ecc8dd812b8f9a6ea08c20278 Mon Sep 17 00:00:00 2001
From: Nick Burch <nick@apache.org>
Date: Sun, 21 Sep 2008 18:56:32 +0000
Subject: [PATCH] Test to show that bug #45492 is invalid

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@697584 13f79535-47bb-0310-9956-ffa450edef68
---
 .../poi/hssf/usermodel/HSSFCellStyle.java     |   8 +++-
 .../poi/hssf/usermodel/HSSFPalette.java       |   9 +++++
 .../org/apache/poi/hssf/data/45492.xls        | Bin 0 -> 13824 bytes
 .../apache/poi/hssf/usermodel/TestBugs.java   |  35 ++++++++++++++++++
 4 files changed, 50 insertions(+), 2 deletions(-)
 create mode 100644 src/testcases/org/apache/poi/hssf/data/45492.xls

diff --git a/src/java/org/apache/poi/hssf/usermodel/HSSFCellStyle.java b/src/java/org/apache/poi/hssf/usermodel/HSSFCellStyle.java
index 2afd16d8d1..d71619aaa8 100644
--- a/src/java/org/apache/poi/hssf/usermodel/HSSFCellStyle.java
+++ b/src/java/org/apache/poi/hssf/usermodel/HSSFCellStyle.java
@@ -913,7 +913,9 @@ public class HSSFCellStyle
     }
 
     /**
-     * get the background fill color
+     * Get the background fill color.
+     * Note - many cells are actually filled with a foreground
+     *  fill, not a background fill - see {@link #getFillForegroundColor()}
      * @see org.apache.poi.hssf.usermodel.HSSFPalette#getColor(short)
      * @return fill color
      */
@@ -939,7 +941,9 @@ public class HSSFCellStyle
     }
 
     /**
-     * get the foreground fill color
+     * Get the foreground fill color.
+     * Many cells are filled with this, instead of a 
+     *  background color ({@link #getFillBackgroundColor()})
      * @see org.apache.poi.hssf.usermodel.HSSFPalette#getColor(short)
      * @return fill color
      */
diff --git a/src/java/org/apache/poi/hssf/usermodel/HSSFPalette.java b/src/java/org/apache/poi/hssf/usermodel/HSSFPalette.java
index 8674904d0c..4c9d63ebc4 100644
--- a/src/java/org/apache/poi/hssf/usermodel/HSSFPalette.java
+++ b/src/java/org/apache/poi/hssf/usermodel/HSSFPalette.java
@@ -57,6 +57,15 @@ public class HSSFPalette
     	}
         return null;
     }
+    /**
+     * Retrieves the color at a given index
+     *
+     * @param index the palette index, between 0x8 to 0x40 inclusive
+     * @return the color, or null if the index is not populated
+     */
+    public HSSFColor getColor(int index) {
+    	return getColor((short)index);
+    }
     
     /**
      * Finds the first occurance of a given color
diff --git a/src/testcases/org/apache/poi/hssf/data/45492.xls b/src/testcases/org/apache/poi/hssf/data/45492.xls
new file mode 100644
index 0000000000000000000000000000000000000000..7e044e8977281260534631253073191a8727c436
GIT binary patch
literal 13824
zcmeHOU1(fI6#nk!?q)ZC-6S>r!}YRJVv`se6k3rs8?|DQU}(@5RFchRYu2P$*=%eC
z!Bt!PQXx{omw*<OwhCGi^u>qdtq&G|posdC_RSan1q*H0@0^=!vfaH&6bjnTEa%Lf
zGiPSbd~@f{oVj<u{GxH;#7CPi;F_=tRk&J=Ahb+w@p_Y&j)M8cq9na|Bd;kk;eUic
zB+P*f1@T+kx$5}@FFD8;5WpFGo*cMHdJ@lK9q@SenS44m>hxtMlDW*7b6vyzYD3Zz
z8|G-!PF`7<rL=+fph%mPH!J^8^*&R+lR4A0_xqtA?%6>-u-yJYLiJu#9#`H#ZsAic
z^PNh%u^DHnUwm;!Zgu>Kpw){UGDu<^*Dp6_Z%2^5vRcuJR{pjTatCodf_f{C`<S|m
z(n@(TyO&cm=3n#)>_r+QcxuH;9zZp2TV6>QZCKfga*deXhcIqmtr|azDphz0?YI}#
zAp74=L@Hw-xh_Vl>3H?TSt4G7v0I1m1cb-zL)hxy^dN_3kb~nEli?|Ex>QFrMQ5q$
z@_eu9qa5nqO1&<J$j<;j%~CUZ(6b5(N71pGDW&}=$70EpQq9Vyl<T;tEMUD%N&HcC
zG<MmXBXXInD*xTfT3Ienk4pM}a;deF{*5fRc&{!656=!4hd;d>PrlVh-svN6_mM{`
z%B3~6C$ZSB-J;`OF0b*C*ZIh6o#O!;(Lz;G;>JQq<EDbGadV+sV-%qA`a-Zcr0c~I
zu0rotOZy=VjHc81?qjfN<2%;86~;Vo$f1md%eb4*Rq)vkM1bT}K6@~k&!i}8O=+kC
za_JEp7#^QWQ|;+oIz3TD4WFZ<)T(2_8Y!%cbhKoVt_`>og4sF&l40N{QoSx~b>t&l
z1Z9b9BnP6b^YZCmzU~|D?-^8iyUIJ1zpRo1b-Zu~0pU`JmzXneOIx>)UQzAC%1`iN
zCK{IhMB57sJ?@pnk5QMk1{zc(QxytIIKYjEUh5u`omht}+FQtGgTYN%Y*$LTsQ_BE
zZ8ueg4cw-Bc{Vn35WG~dq?XWP>KhK&bJPKL1)A`Q?rsyBcG<1elNuRZQwb`I0NY72
z6ebu13<3rLgMdNcMnK>ySKpxSX-az|tKY(rw|>3!=xFr)*Td*&`}o^!l${~To<{N$
zBw27DCdtNOh}1-yCdt-ihEz|QB{h>ykz^}!k%eN6Wdm@{p~9;2zDz2Yoy;D{JG-Y-
z>2bB8@SeTt<j-fV1nWcFt0}o&T*O6Uf<eF_U=T0}7z7Lg1_6VBLBJqj5HJWB1a5i+
zR`UM?=g*y=?b;GO{wDqZ&PyMN|Nl9O?=sZ??`JGW_4`R1NaLhhQl2Ef{R<@V`wx>^
zNXJNW9`FW9&IeACmY)Ym***OO&SY1vE8B%`@!=b`M6^y``o=LfCg%}yF5$k7K^;X>
zwHz}@wfnNUu}OJRGpWb>-q}8LOfU!-1PlTO0fT@+z#w1{FbEg~3<3rLgTPIIfOx{<
zD2uNx@ABnaMfu)Md};Y#hxq#P-J3Yk;<JnYE}!y=-!J)%jOCrb`1#&Pg5vATd4S|!
zAxUh)e-nYd$nuwmgWZ^5oWt`RxwVHDem&y#OztBmEF73qj)&aH<uh+LeRYc8X2|{E
z$@j7K9L6xLc8q!AHOKrH$lvXY9cwNq2UH9}3}6Ze`Acfw_Yj2xYLiIv{O?DW68~@~
o+m|^c^JlH?%eOLmK*lJD2iZbOZAj|~{4;xY{EPl;*BAT$7l(~4@Bjb+

literal 0
HcmV?d00001

diff --git a/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java b/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java
index cc3bf573d2..ea807f0542 100644
--- a/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java
+++ b/src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java
@@ -1482,6 +1482,41 @@ public final class TestBugs extends TestCase {
     	// This used to break
         HSSFWorkbook wb = openSample("45784.xls");
         assertEquals(1, wb.getNumberOfSheets());
+    }
+    
+   /**
+     * Cell background colours
+     */
+    public void test45492() {
+    	HSSFWorkbook wb = openSample("45492.xls");
+    	HSSFSheet s = wb.getSheetAt(0);
+    	HSSFRow r = s.getRow(0);
+    	HSSFPalette p = wb.getCustomPalette();
+    	
+    	HSSFCell auto = r.getCell(0);
+    	HSSFCell grey = r.getCell(1);
+    	HSSFCell red = r.getCell(2);
+    	HSSFCell blue = r.getCell(3);
+    	HSSFCell green = r.getCell(4);
+    	
+    	assertEquals(64, auto.getCellStyle().getFillForegroundColor());
+    	assertEquals(64, auto.getCellStyle().getFillBackgroundColor());
+    	assertEquals("0:0:0", p.getColor(64).getHexString());
+    	
+    	assertEquals(22, grey.getCellStyle().getFillForegroundColor());
+    	assertEquals(64, grey.getCellStyle().getFillBackgroundColor());
+    	assertEquals("C0C0:C0C0:C0C0", p.getColor(22).getHexString());
+    	
+    	assertEquals(10, red.getCellStyle().getFillForegroundColor());
+    	assertEquals(64, red.getCellStyle().getFillBackgroundColor());
+    	assertEquals("FFFF:0:0", p.getColor(10).getHexString());
+    	
+    	assertEquals(12, blue.getCellStyle().getFillForegroundColor());
+    	assertEquals(64, blue.getCellStyle().getFillBackgroundColor());
+    	assertEquals("0:0:FFFF", p.getColor(12).getHexString());
     	
+    	assertEquals(11, green.getCellStyle().getFillForegroundColor());
+    	assertEquals(64, green.getCellStyle().getFillBackgroundColor());
+    	assertEquals("0:FFFF:0", p.getColor(11).getHexString());
     }
 }
-- 
2.39.5