tidy up API for font index
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1825409 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
4221ccebda
commit
fef8c69085
@ -138,7 +138,7 @@ public class SVTableCellEditor extends AbstractCellEditor implements TableCellEd
|
|||||||
HSSFCell cell = (HSSFCell) value;
|
HSSFCell cell = (HSSFCell) value;
|
||||||
if (cell != null) {
|
if (cell != null) {
|
||||||
HSSFCellStyle style = cell.getCellStyle();
|
HSSFCellStyle style = cell.getCellStyle();
|
||||||
HSSFFont f = wb.getFontAt(style.getFontIntIndex());
|
HSSFFont f = wb.getFontAt(style.getFontIndexAsInt());
|
||||||
boolean isbold = f.getBold();
|
boolean isbold = f.getBold();
|
||||||
boolean isitalics = f.getItalic();
|
boolean isitalics = f.getItalic();
|
||||||
|
|
||||||
|
@ -148,7 +148,7 @@ public class SVTableCellRenderer extends JLabel
|
|||||||
|
|
||||||
if (c != null) {
|
if (c != null) {
|
||||||
HSSFCellStyle s = c.getCellStyle();
|
HSSFCellStyle s = c.getCellStyle();
|
||||||
HSSFFont f = wb.getFontAt(s.getFontIntIndex());
|
HSSFFont f = wb.getFontAt(s.getFontIndexAsInt());
|
||||||
setFont(SVTableUtils.makeFont(f));
|
setFont(SVTableUtils.makeFont(f));
|
||||||
|
|
||||||
if (s.getFillPattern() == FillPatternType.SOLID_FOREGROUND) {
|
if (s.getFillPattern() == FillPatternType.SOLID_FOREGROUND) {
|
||||||
|
@ -63,7 +63,7 @@ public class CellStyleDetails {
|
|||||||
System.out.print("FG=" + renderColor(style.getFillForegroundColorColor()) + " ");
|
System.out.print("FG=" + renderColor(style.getFillForegroundColorColor()) + " ");
|
||||||
System.out.print("BG=" + renderColor(style.getFillBackgroundColorColor()) + " ");
|
System.out.print("BG=" + renderColor(style.getFillBackgroundColorColor()) + " ");
|
||||||
|
|
||||||
Font font = wb.getFontAt(style.getFontIntIndex());
|
Font font = wb.getFontAt(style.getFontIndexAsInt());
|
||||||
System.out.print("Font=" + font.getFontName() + " ");
|
System.out.print("Font=" + font.getFontName() + " ");
|
||||||
System.out.print("FontColor=");
|
System.out.print("FontColor=");
|
||||||
if (font instanceof HSSFFont) {
|
if (font instanceof HSSFFont) {
|
||||||
|
@ -300,7 +300,7 @@ public class ToHtml {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private void fontStyle(CellStyle style) {
|
private void fontStyle(CellStyle style) {
|
||||||
Font font = wb.getFontAt(style.getFontIntIndex());
|
Font font = wb.getFontAt(style.getFontIndexAsInt());
|
||||||
|
|
||||||
if (font.getBold()) {
|
if (font.getBold()) {
|
||||||
out.format(" font-weight: bold;%n");
|
out.format(" font-weight: bold;%n");
|
||||||
|
@ -198,7 +198,7 @@ public final class HSSFCellStyle implements CellStyle {
|
|||||||
* @since 4.0.0
|
* @since 4.0.0
|
||||||
*/
|
*/
|
||||||
@Override
|
@Override
|
||||||
public int getFontIntIndex()
|
public int getFontIndexAsInt()
|
||||||
{
|
{
|
||||||
return _format.getFontIndex();
|
return _format.getFontIndex();
|
||||||
}
|
}
|
||||||
@ -206,11 +206,11 @@ public final class HSSFCellStyle implements CellStyle {
|
|||||||
/**
|
/**
|
||||||
* gets the font for this style
|
* gets the font for this style
|
||||||
* @param parentWorkbook The HSSFWorkbook that this style belongs to
|
* @param parentWorkbook The HSSFWorkbook that this style belongs to
|
||||||
* @see org.apache.poi.hssf.usermodel.HSSFCellStyle#getFontIntIndex()
|
* @see org.apache.poi.hssf.usermodel.HSSFCellStyle#getFontIndexAsInt()
|
||||||
* @see org.apache.poi.hssf.usermodel.HSSFWorkbook#getFontAt(int)
|
* @see org.apache.poi.hssf.usermodel.HSSFWorkbook#getFontAt(int)
|
||||||
*/
|
*/
|
||||||
public HSSFFont getFont(org.apache.poi.ss.usermodel.Workbook parentWorkbook) {
|
public HSSFFont getFont(org.apache.poi.ss.usermodel.Workbook parentWorkbook) {
|
||||||
return ((HSSFWorkbook) parentWorkbook).getFontAt(getFontIntIndex());
|
return ((HSSFWorkbook) parentWorkbook).getFontAt(getFontIndexAsInt());
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@ -849,7 +849,7 @@ public final class HSSFCellStyle implements CellStyle {
|
|||||||
FontRecord fr = _workbook.createNewFont();
|
FontRecord fr = _workbook.createNewFont();
|
||||||
fr.cloneStyleFrom(
|
fr.cloneStyleFrom(
|
||||||
source._workbook.getFontRecordAt(
|
source._workbook.getFontRecordAt(
|
||||||
source.getFontIntIndex()
|
source.getFontIndexAsInt()
|
||||||
)
|
)
|
||||||
);
|
);
|
||||||
|
|
||||||
|
@ -56,7 +56,7 @@ public interface CellStyle {
|
|||||||
/**
|
/**
|
||||||
* gets the index of the font for this style
|
* gets the index of the font for this style
|
||||||
* @see Workbook#getFontAt(short)
|
* @see Workbook#getFontAt(short)
|
||||||
* @deprecated use <code>getFontIntIndex()</code> instead
|
* @deprecated use <code>getFontIndexAsInt()</code> instead
|
||||||
*/
|
*/
|
||||||
@Removal(version = "4.2")
|
@Removal(version = "4.2")
|
||||||
short getFontIndex();
|
short getFontIndex();
|
||||||
@ -66,7 +66,7 @@ public interface CellStyle {
|
|||||||
* @see Workbook#getFontAt(int)
|
* @see Workbook#getFontAt(int)
|
||||||
* @since 4.0.0
|
* @since 4.0.0
|
||||||
*/
|
*/
|
||||||
int getFontIntIndex();
|
int getFontIndexAsInt();
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* set the cell's using this style to be hidden
|
* set the cell's using this style to be hidden
|
||||||
|
@ -346,7 +346,7 @@ public final class CellUtil {
|
|||||||
put(properties, FILL_PATTERN, style.getFillPattern());
|
put(properties, FILL_PATTERN, style.getFillPattern());
|
||||||
put(properties, FILL_FOREGROUND_COLOR, style.getFillForegroundColor());
|
put(properties, FILL_FOREGROUND_COLOR, style.getFillForegroundColor());
|
||||||
put(properties, FILL_BACKGROUND_COLOR, style.getFillBackgroundColor());
|
put(properties, FILL_BACKGROUND_COLOR, style.getFillBackgroundColor());
|
||||||
put(properties, FONT, style.getFontIntIndex());
|
put(properties, FONT, style.getFontIndexAsInt());
|
||||||
put(properties, HIDDEN, style.getHidden());
|
put(properties, HIDDEN, style.getHidden());
|
||||||
put(properties, INDENTION, style.getIndention());
|
put(properties, INDENTION, style.getIndention());
|
||||||
put(properties, LEFT_BORDER_COLOR, style.getLeftBorderColor());
|
put(properties, LEFT_BORDER_COLOR, style.getLeftBorderColor());
|
||||||
|
@ -144,7 +144,7 @@ public class SheetUtil {
|
|||||||
if (cellType == CellType.FORMULA)
|
if (cellType == CellType.FORMULA)
|
||||||
cellType = cell.getCachedFormulaResultType();
|
cellType = cell.getCachedFormulaResultType();
|
||||||
|
|
||||||
Font font = wb.getFontAt(style.getFontIntIndex());
|
Font font = wb.getFontAt(style.getFontIndexAsInt());
|
||||||
|
|
||||||
double width = -1;
|
double width = -1;
|
||||||
if (cellType == CellType.STRING) {
|
if (cellType == CellType.STRING) {
|
||||||
|
@ -482,7 +482,7 @@ public class XSSFCellStyle implements CellStyle {
|
|||||||
* @since 4.0.0
|
* @since 4.0.0
|
||||||
*/
|
*/
|
||||||
@Override
|
@Override
|
||||||
public int getFontIntIndex() {
|
public int getFontIndexAsInt() {
|
||||||
return getFontId();
|
return getFontId();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -210,7 +210,7 @@ public class TestAutoSizeColumnTracker {
|
|||||||
|
|
||||||
private static void assumeRequiredFontsAreInstalled(final Workbook workbook, final Cell cell) {
|
private static void assumeRequiredFontsAreInstalled(final Workbook workbook, final Cell cell) {
|
||||||
// autoSize will fail if required fonts are not installed, skip this test then
|
// autoSize will fail if required fonts are not installed, skip this test then
|
||||||
Font font = workbook.getFontAt(cell.getCellStyle().getFontIntIndex());
|
Font font = workbook.getFontAt(cell.getCellStyle().getFontIndexAsInt());
|
||||||
Assume.assumeTrue("Cannot verify autoSizeColumn() because the necessary Fonts are not installed on this machine: " + font,
|
Assume.assumeTrue("Cannot verify autoSizeColumn() because the necessary Fonts are not installed on this machine: " + font,
|
||||||
SheetUtil.canComputeColumnWidth(font));
|
SheetUtil.canComputeColumnWidth(font));
|
||||||
}
|
}
|
||||||
|
@ -350,7 +350,7 @@ public class TestSXSSFSheetAutoSizeColumn {
|
|||||||
|
|
||||||
private static void assumeRequiredFontsAreInstalled(final Workbook workbook, final Cell cell) {
|
private static void assumeRequiredFontsAreInstalled(final Workbook workbook, final Cell cell) {
|
||||||
// autoSize will fail if required fonts are not installed, skip this test then
|
// autoSize will fail if required fonts are not installed, skip this test then
|
||||||
Font font = workbook.getFontAt(cell.getCellStyle().getFontIntIndex());
|
Font font = workbook.getFontAt(cell.getCellStyle().getFontIndexAsInt());
|
||||||
Assume.assumeTrue("Cannot verify autoSizeColumn() because the necessary Fonts are not installed on this machine: " + font,
|
Assume.assumeTrue("Cannot verify autoSizeColumn() because the necessary Fonts are not installed on this machine: " + font,
|
||||||
SheetUtil.canComputeColumnWidth(font));
|
SheetUtil.canComputeColumnWidth(font));
|
||||||
}
|
}
|
||||||
|
@ -324,7 +324,7 @@ public final class TestXSSFSheet extends BaseTestXSheet {
|
|||||||
cellXf.setXfId(1);
|
cellXf.setXfId(1);
|
||||||
stylesTable.putCellXf(cellXf);
|
stylesTable.putCellXf(cellXf);
|
||||||
XSSFCellStyle cellStyle = new XSSFCellStyle(1, 1, stylesTable, null);
|
XSSFCellStyle cellStyle = new XSSFCellStyle(1, 1, stylesTable, null);
|
||||||
assertEquals(1, cellStyle.getFontIntIndex());
|
assertEquals(1, cellStyle.getFontIndexAsInt());
|
||||||
|
|
||||||
sheet.setDefaultColumnStyle(3, cellStyle);
|
sheet.setDefaultColumnStyle(3, cellStyle);
|
||||||
assertEquals(1, ctWorksheet.getColsArray(0).getColArray(0).getStyle());
|
assertEquals(1, ctWorksheet.getColsArray(0).getColArray(0).getStyle());
|
||||||
|
@ -3130,12 +3130,12 @@ public final class TestBugs extends BaseTestBugzillaIssues {
|
|||||||
|
|
||||||
Cell cell = row.getCell(1);
|
Cell cell = row.getCell(1);
|
||||||
CellStyle style = cell.getCellStyle();
|
CellStyle style = cell.getCellStyle();
|
||||||
assertEquals(26, style.getFontIntIndex());
|
assertEquals(26, style.getFontIndexAsInt());
|
||||||
|
|
||||||
row = sheet.getRow(3);
|
row = sheet.getRow(3);
|
||||||
cell = row.getCell(1);
|
cell = row.getCell(1);
|
||||||
style = cell.getCellStyle();
|
style = cell.getCellStyle();
|
||||||
assertEquals(28, style.getFontIntIndex());
|
assertEquals(28, style.getFontIndexAsInt());
|
||||||
|
|
||||||
// check the two fonts
|
// check the two fonts
|
||||||
HSSFFont font = wb.getFontAt(26);
|
HSSFFont font = wb.getFontAt(26);
|
||||||
|
@ -115,7 +115,7 @@ public abstract class BaseTestFont {
|
|||||||
s1 = wb2.getSheetAt(0);
|
s1 = wb2.getSheetAt(0);
|
||||||
|
|
||||||
assertEquals(num0 + 1, wb2.getNumberOfFontsAsInt());
|
assertEquals(num0 + 1, wb2.getNumberOfFontsAsInt());
|
||||||
int idx = s1.getRow(0).getCell(0).getCellStyle().getFontIntIndex();
|
int idx = s1.getRow(0).getCell(0).getCellStyle().getFontIndexAsInt();
|
||||||
Font fnt = wb2.getFontAt(idx);
|
Font fnt = wb2.getFontAt(idx);
|
||||||
assertNotNull(fnt);
|
assertNotNull(fnt);
|
||||||
assertEquals(IndexedColors.YELLOW.getIndex(), fnt.getColor());
|
assertEquals(IndexedColors.YELLOW.getIndex(), fnt.getColor());
|
||||||
|
Loading…
x
Reference in New Issue
Block a user