From bdd647d98f3f93f2d2b992dc9fec6f8614364881 Mon Sep 17 00:00:00 2001 From: Keiron Liddle Date: Tue, 12 Dec 2000 23:48:36 +0000 Subject: [PATCH] Fixed colspanning so that the cell state keeps track of the width so far properly Also some cleanup, jstyle Code changes sponsored by Dresdner Bank, Germany git-svn-id: https://svn.apache.org/repos/asf/xmlgraphics/fop/trunk@193867 13f79535-47bb-0310-9956-ffa450edef68 --- src/org/apache/fop/fo/flow/TableRow.java | 948 +++++++++++------------ 1 file changed, 460 insertions(+), 488 deletions(-) diff --git a/src/org/apache/fop/fo/flow/TableRow.java b/src/org/apache/fop/fo/flow/TableRow.java index 8c6e0b5d4..7ee5ebb96 100644 --- a/src/org/apache/fop/fo/flow/TableRow.java +++ b/src/org/apache/fop/fo/flow/TableRow.java @@ -1,36 +1,36 @@ -/*-- $Id$ -- +/*-- $Id$ -- ============================================================================ The Apache Software License, Version 1.1 ============================================================================ - + Copyright (C) 1999 The Apache Software Foundation. All rights reserved. - + Redistribution and use in source and binary forms, with or without modifica- tion, are permitted provided that the following conditions are met: - + 1. Redistributions of source code must retain the above copyright notice, this list of conditions and the following disclaimer. - + 2. Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer in the documentation and/or other materials provided with the distribution. - + 3. The end-user documentation included with the redistribution, if any, must include the following acknowledgment: "This product includes software developed by the Apache Software Foundation (http://www.apache.org/)." Alternately, this acknowledgment may appear in the software itself, if and wherever such third-party acknowledgments normally appear. - + 4. The names "FOP" and "Apache Software Foundation" must not be used to endorse or promote products derived from this software without prior written permission. For written permission, please contact apache@apache.org. - + 5. Products derived from this software may not be called "Apache", nor may "Apache" appear in their name, without prior written permission of the Apache Software Foundation. - + THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESSED OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE @@ -41,12 +41,12 @@ ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. - + This software consists of voluntary contributions made by many individuals on behalf of the Apache Software Foundation and was originally created by - James Tauber . For more information on the Apache + James Tauber . For more information on the Apache Software Foundation, please see . - + */ package org.apache.fop.fo.flow; @@ -63,16 +63,16 @@ import org.apache.fop.apps.FOPException; import java.util.Vector; public class TableRow extends FObj { - + public static class Maker extends FObj.Maker { - public FObj make(FObj parent, PropertyList propertyList) - throws FOPException { - return new TableRow(parent, propertyList); - } + public FObj make(FObj parent, + PropertyList propertyList) throws FOPException { + return new TableRow(parent, propertyList); + } } public static FObj.Maker maker() { - return new TableRow.Maker(); + return new TableRow.Maker(); } boolean setup = false; @@ -81,8 +81,8 @@ public class TableRow extends FObj { int spaceBefore; int spaceAfter; ColorType backgroundColor; - String id; - + String id; + ColorType borderTopColor; int borderTopWidth; int borderTopStyle; @@ -109,505 +109,477 @@ public class TableRow extends FObj { AreaContainer areaContainer; - // added by Dresdner Bank, Germany + // added by Dresdner Bank, Germany DisplaySpace spacer = null; - /** - * The list of cell states for this row. This is the location of - * where I will be storing the state of each cell so that I can - * spread a cell over multiple pages if I have to. This is part - * of fixing the TableRow larger than a single page bug. - * Hani Elabed, 11/22/2000. - */ - public Vector cells = null; - - /** - * CellState
- * - * Copyright @ 2000 Circuit Court Automation Program. - * state of Wisconsin. - * All Rights Reserved. - *

- * This class is a container class for encapsulating a the - * state of a cell - * - * - * - * - * - * - * - *
Name: CellState
Purpose: a helpful container class
Description: This class is a container class for - * encapsulating the state of a - * cell belonging to a TableRow class - *
- * - * @author Hani Elabed - * @version 0.14.0, 11/22/2000 - * @since JDK1.1 - */ - public final class CellState - { - /** the cell location or index starting at 0.*/ - private int location; - - /** true if the layout of the cell was complete, false otherwise.*/ - private boolean layoutCompleted; - - /** the width of the cell so far.*/ - private int widthOfCellSoFar; - - private int column = 0; - - /** - * simple no args constructor. - */ - public CellState() - { - this( 0, false, 0 ); - } - - /** - * three argument fill everything constructor. - * @param int the location(index) of the cell. - * @param boolean flag of wether the cell was completely laid out or not. - * @param int the horizontal offset(width so far) of the cell. - */ - public CellState( int aLocation, boolean completed, int aWidth ) - { - - location = aLocation; - layoutCompleted = completed; - widthOfCellSoFar = aWidth; - } - - /** - * returns the index of the cell starting at 0. - * @return int the location of the cell. - */ - public final int getLocation() - { return location; } - - /** - * sets the location of the cell. - * @param int, the location of the cell. - */ - public final void setLocation(int aLocation) - { location = aLocation; } - - - /** - * returns true if the cell was completely laid out. - * @return false if cell was partially laid out. - */ - public final boolean isLayoutComplete() - { return layoutCompleted; } - - /** - * sets the layoutCompleted flag. - * @param boolean, the layout Complete state of the cell. - */ - public final void setLayoutComplete(boolean completed ) - { layoutCompleted = completed; } - - - /** - * returns the horizontal offset of the cell. - * @return int the horizontal offset of the cell, also known as width - * of the cell so far. - */ - public final int getWidthOfCellSoFar() - { return widthOfCellSoFar; } - - /** - * sets the width of the Cell So Far, i.e the cell's offset. - * @param int, the horizontal offset of the cell. - */ - public final void setWidthOfCellSoFar(int aWidth) - { widthOfCellSoFar = aWidth; } - - public int getColumn() - { return column; } - - public void setColumn(int col) - { column = col; } - } + /** + * The list of cell states for this row. This is the location of + * where I will be storing the state of each cell so that I can + * spread a cell over multiple pages if I have to. This is part + * of fixing the TableRow larger than a single page bug. + * Hani Elabed, 11/22/2000. + */ + public Vector cells = null; + + /** + * CellState
+ * + * Copyright @ 2000 Circuit Court Automation Program. + * state of Wisconsin. + * All Rights Reserved. + *

+ * This class is a container class for encapsulating a the + * state of a cell + * + * + * + * + * + * + * + *
Name: CellState
Purpose: a helpful container class
Description: This class is a container class for + * encapsulating the state of a + * cell belonging to a TableRow class + *
+ * + * @author Hani Elabed + * @version 0.14.0, 11/22/2000 + * @since JDK1.1 + */ + public final class CellState { + /** the cell location or index starting at 0.*/ + private int location; + + /** true if the layout of the cell was complete, false otherwise.*/ + private boolean layoutCompleted; + + /** the width of the cell so far.*/ + private int widthOfCellSoFar; + + private int column = 0; + + /** + * simple no args constructor. + */ + public CellState() { + this(0, false, 0); + } + + /** + * three argument fill everything constructor. + * @param int the location(index) of the cell. + * @param boolean flag of wether the cell was completely laid out or not. + * @param int the horizontal offset(width so far) of the cell. + */ + public CellState(int aLocation, boolean completed, int aWidth) { + + location = aLocation; + layoutCompleted = completed; + widthOfCellSoFar = aWidth; + } + + /** + * returns the index of the cell starting at 0. + * @return int the location of the cell. + */ + public final int getLocation() { + return location; + } + + /** + * sets the location of the cell. + * @param int, the location of the cell. + */ + public final void setLocation(int aLocation) { + location = aLocation; + } + + + /** + * returns true if the cell was completely laid out. + * @return false if cell was partially laid out. + */ + public final boolean isLayoutComplete() { + return layoutCompleted; + } + + /** + * sets the layoutCompleted flag. + * @param boolean, the layout Complete state of the cell. + */ + public final void setLayoutComplete(boolean completed) { + layoutCompleted = completed; + } + + + /** + * returns the horizontal offset of the cell. + * @return int the horizontal offset of the cell, also known as width + * of the cell so far. + */ + public final int getWidthOfCellSoFar() { + return widthOfCellSoFar; + } + + /** + * sets the width of the Cell So Far, i.e the cell's offset. + * @param int, the horizontal offset of the cell. + */ + public final void setWidthOfCellSoFar(int aWidth) { + widthOfCellSoFar = aWidth; + } + + public int getColumn() { + return column; + } + + public void setColumn(int col) { + column = col; + } + } public TableRow(FObj parent, PropertyList propertyList) { - super(parent, propertyList); - this.name = "fo:table-row"; + super(parent, propertyList); + this.name = "fo:table-row"; } public void setColumns(Vector columns) { - this.columns = columns; + this.columns = columns; } - public int getKeepWithPrevious() - { + public int getKeepWithPrevious() { return keepWithPrevious; } - public void doSetup(Area area) throws FOPException - { - String fontFamily = - this.properties.get("font-family").getString(); - String fontStyle = - this.properties.get("font-style").getString(); - String fontWeight = - this.properties.get("font-weight").getString(); - int fontSize = - this.properties.get("font-size").getLength().mvalue(); - - this.fs = new FontState(area.getFontInfo(), fontFamily, - fontStyle, fontWeight, fontSize); - this.spaceBefore = - this.properties.get("space-before.optimum").getLength().mvalue(); - this.spaceAfter = - this.properties.get("space-after.optimum").getLength().mvalue(); - this.backgroundColor = - this.properties.get("background-color").getColorType(); - this.borderTopColor = - this.properties.get("border-color").getColorType(); - this.borderBottomColor = this.borderTopColor; - this.borderLeftColor = this.borderTopColor; - this.borderRightColor = this.borderTopColor; - if (this.borderTopColor == null) { - this.borderTopColor = - this.properties.get("border-top-color").getColorType(); - this.borderBottomColor = - this.properties.get("border-bottom-color").getColorType(); - this.borderLeftColor = - this.properties.get("border-left-color").getColorType(); - this.borderRightColor = - this.properties.get("border-right-color").getColorType(); - } - this.borderTopWidth = - this.properties.get("border-width").getLength().mvalue(); - this.borderBottomWidth = this.borderTopWidth; - this.borderLeftWidth = this.borderTopWidth; - this.borderRightWidth = this.borderTopWidth; - if (this.borderTopWidth == 0) { - this.borderTopWidth = - this.properties.get("border-top-width").getLength().mvalue(); - this.borderBottomWidth = - this.properties.get("border-bottom-width").getLength().mvalue(); - this.borderLeftWidth = - this.properties.get("border-left-width").getLength().mvalue(); - this.borderRightWidth = - this.properties.get("border-right-width").getLength().mvalue(); - } - this.borderTopStyle = - this.properties.get("border-style").getEnum(); - this.borderBottomStyle = this.borderTopStyle; - this.borderLeftStyle = this.borderTopStyle; - this.borderRightStyle = this.borderTopStyle; - if (this.borderTopStyle == 0) { - this.borderTopStyle = - this.properties.get("border-top-style").getEnum(); - this.borderBottomStyle = - this.properties.get("border-bottom-style").getEnum(); - this.borderLeftStyle = - this.properties.get("border-left-style").getEnum(); - this.borderRightStyle = - this.properties.get("border-right-style").getEnum(); - } - this.keepWithNext = - this.properties.get("keep-with-next").getNumber().intValue(); - this.keepWithPrevious = - this.properties.get("keep-with-previous").getNumber().intValue(); - this.paddingTop = - this.properties.get("padding").getLength().mvalue(); - this.paddingLeft = this.paddingTop; - this.paddingRight = this.paddingTop; - this.paddingBottom = this.paddingTop; - if (this.paddingTop == 0) { - this.paddingTop = - this.properties.get("padding-top").getLength().mvalue(); - this.paddingLeft = - this.properties.get("padding-left").getLength().mvalue(); - this.paddingBottom = - this.properties.get("padding-bottom").getLength().mvalue(); - this.paddingRight = - this.properties.get("padding-right").getLength().mvalue(); - } - this.id= - this.properties.get("id").getString(); - setup = true; + public void doSetup(Area area) throws FOPException { + String fontFamily = this.properties.get("font-family").getString(); + String fontStyle = this.properties.get("font-style").getString(); + String fontWeight = this.properties.get("font-weight").getString(); + int fontSize = + this.properties.get("font-size").getLength().mvalue(); + + this.fs = new FontState(area.getFontInfo(), fontFamily, + fontStyle, fontWeight, fontSize); + this.spaceBefore = this.properties.get( + "space-before.optimum").getLength().mvalue(); + this.spaceAfter = this.properties.get( + "space-after.optimum").getLength().mvalue(); + this.backgroundColor = + this.properties.get("background-color").getColorType(); + this.borderTopColor = + this.properties.get("border-color").getColorType(); + this.borderBottomColor = this.borderTopColor; + this.borderLeftColor = this.borderTopColor; + this.borderRightColor = this.borderTopColor; + if (this.borderTopColor == null) { + this.borderTopColor = this.properties.get( + "border-top-color").getColorType(); + this.borderBottomColor = this.properties.get( + "border-bottom-color").getColorType(); + this.borderLeftColor = this.properties.get( + "border-left-color").getColorType(); + this.borderRightColor = this.properties.get( + "border-right-color").getColorType(); + } + this.borderTopWidth = + this.properties.get("border-width").getLength().mvalue(); + this.borderBottomWidth = this.borderTopWidth; + this.borderLeftWidth = this.borderTopWidth; + this.borderRightWidth = this.borderTopWidth; + if (this.borderTopWidth == 0) { + this.borderTopWidth = this.properties.get( + "border-top-width").getLength().mvalue(); + this.borderBottomWidth = this.properties.get( + "border-bottom-width").getLength().mvalue(); + this.borderLeftWidth = this.properties.get( + "border-left-width").getLength().mvalue(); + this.borderRightWidth = this.properties.get( + "border-right-width").getLength().mvalue(); + } + this.borderTopStyle = this.properties.get("border-style").getEnum(); + this.borderBottomStyle = this.borderTopStyle; + this.borderLeftStyle = this.borderTopStyle; + this.borderRightStyle = this.borderTopStyle; + if (this.borderTopStyle == 0) { + this.borderTopStyle = + this.properties.get("border-top-style").getEnum(); + this.borderBottomStyle = + this.properties.get("border-bottom-style").getEnum(); + this.borderLeftStyle = + this.properties.get("border-left-style").getEnum(); + this.borderRightStyle = + this.properties.get("border-right-style").getEnum(); + } + this.keepWithNext = this.properties.get( + "keep-with-next").getNumber().intValue(); + this.keepWithPrevious = this.properties.get( + "keep-with-previous").getNumber().intValue(); + this.paddingTop = + this.properties.get("padding").getLength().mvalue(); + this.paddingLeft = this.paddingTop; + this.paddingRight = this.paddingTop; + this.paddingBottom = this.paddingTop; + if (this.paddingTop == 0) { + this.paddingTop = this.properties.get( + "padding-top").getLength().mvalue(); + this.paddingLeft = this.properties.get( + "padding-left").getLength().mvalue(); + this.paddingBottom = this.properties.get( + "padding-bottom").getLength().mvalue(); + this.paddingRight = this.properties.get( + "padding-right").getLength().mvalue(); + } + this.id = this.properties.get("id").getString(); + setup = true; } public Status layout(Area area) throws FOPException { - if (this.marker == BREAK_AFTER) { - return new Status(Status.OK); - } - - if (this.marker == START) { - if(!setup) - doSetup(area); - - if (area instanceof BlockArea) { - area.end(); - } - - area.getIDReferences().createID(id); - - this.marker = 0; - - - // cells is The list of cell states for this row. This is the location of - // where I will be storing the state of each cell so that I can - // spread a cell over multiple pages if I have to. This is part - // of fixing the TableRow larger than a single page bug. - // Hani Elabed, 11/22/2000. - if( cells == null ) // do it once.. - { - int numOfCols = columns.size(); - cells = new Vector( numOfCols ); - int cumulativeWidth = 0; - // populating the cells vector with initial states. - for (int i = 0; i < numOfCols; i++) - { - int width = ((TableColumn) columns.elementAt(i)).getColumnWidth(); - - CellState state = new CellState( i, false, cumulativeWidth ); - // add the state of a cell. - cells.insertElementAt( state,i); - // cumulative width, i.e start offset of cell - // also known as widthOfCellSoFar... - cumulativeWidth += width; - } - - } - - - - } - - if ((spaceBefore != 0) && (this.marker ==0)) { - spacer = new DisplaySpace(spaceBefore); - area.increaseHeight(spaceBefore); - } - - if ( marker==0 ) { - // configure id - area.getIDReferences().configureID(id,area); + if (this.marker == BREAK_AFTER) { + return new Status(Status.OK); } - this.areaContainer = - new AreaContainer(fs, -area.borderWidthLeft, -area.borderWidthTop, - area.getAllocationWidth(), - area.spaceLeft(), Position.RELATIVE); - areaContainer.setPage(area.getPage()); - areaContainer.setPadding(paddingTop, paddingLeft, paddingBottom, - paddingRight); - areaContainer.setBackgroundColor(backgroundColor); - areaContainer.setBorderStyle(borderTopStyle, borderLeftStyle, borderBottomStyle, borderRightStyle); - areaContainer.setBorderWidth(borderTopWidth, borderLeftWidth, borderBottomWidth, borderRightWidth); - areaContainer.setBorderColor(borderTopColor, borderLeftColor, borderBottomColor, borderRightColor); - areaContainer.start(); + if (this.marker == START) { + if (!setup) + doSetup(area); + + if (area instanceof BlockArea) { + area.end(); + } + + area.getIDReferences().createID(id); + + this.marker = 0; + } + + + if ((spaceBefore != 0) && (this.marker == 0)) { + spacer = new DisplaySpace(spaceBefore); + area.increaseHeight(spaceBefore); + } + + if (marker == 0) { + // configure id + area.getIDReferences().configureID(id, area); + } + + this.areaContainer = new AreaContainer(fs, -area.borderWidthLeft, + -area.borderWidthTop, area.getAllocationWidth(), + area.spaceLeft(), Position.RELATIVE); + areaContainer.setPage(area.getPage()); + areaContainer.setPadding(paddingTop, paddingLeft, + paddingBottom, paddingRight); + areaContainer.setBackgroundColor(backgroundColor); + areaContainer.setBorderStyle(borderTopStyle, borderLeftStyle, + borderBottomStyle, borderRightStyle); + areaContainer.setBorderWidth(borderTopWidth, borderLeftWidth, + borderBottomWidth, borderRightWidth); + areaContainer.setBorderColor(borderTopColor, borderLeftColor, + borderBottomColor, borderRightColor); + areaContainer.start(); areaContainer.setAbsoluteHeight(area.getAbsoluteHeight()); areaContainer.setIDReferences(area.getIDReferences()); - int numChildren = this.children.size(); -// if (numChildren != columns.size()) { -// MessageHandler.errorln("WARNING: Number of children under table-row not equal to number of table-columns"); -// return new Status(Status.OK); -// } - - // added by Eric Schaeffer - widthOfCellsSoFar = 0; - largestCellHeight = 0; - - // added by Hani Elabed 11/27/2000 - boolean someCellDidNotLayoutCompletely = false; - - int colCount = -1; - for (int i = this.marker; i < numChildren; i++) { - TableCell cell = (TableCell) children.elementAt(i); - - // added by Hani Elabed 11/22/2000 - CellState cellState = (CellState) cells.elementAt( i ); - - if(colCount == -1) { - colCount = cellState.getColumn(); - } - cell.doSetup(areaContainer); - int numCols = cell.getNumColumnsSpanned(); - int numRows = cell.getNumRowsSpanned(); - - - //if (this.isInListBody) { - //fo.setIsInListBody(); - //fo.setDistanceBetweenStarts(this.distanceBetweenStarts); - //fo.setBodyIndent(this.bodyIndent); - //} - - - //--- this is modified to preserve the state of start - //--- offset of the cell. - //--- change by Hani Elabed 11/22/2000 - //cell.setStartOffset(widthOfCellsSoFar); - cell.setStartOffset( cellState.getWidthOfCellSoFar() ); - - int width = 0; - cellState.setColumn(colCount); - for(int count = 0; count < numCols && count < columns.size(); count++) { - width += ((TableColumn) columns.elementAt(colCount)).getColumnWidth(); - colCount++; - } - - cell.setWidth(width); - widthOfCellsSoFar += width; - - Status status; - if ((status = cell.layout(areaContainer)).isIncomplete()) - { - this.marker = i; -/* if ((i != 0) && (status.getCode() == Status.AREA_FULL_NONE)) - { - status = new Status(Status.AREA_FULL_SOME); - }*/ - - - if( status.getCode() == Status.AREA_FULL_SOME ) - { - // this whole block added by - // Hani Elabed 11/27/2000 - - cellState.setLayoutComplete( false ); - - // locate the first cell - // that need to be laid out further - - for (int j = 0; j < numChildren; j++) - { - CellState state = (CellState) cells.elementAt( j ); - - if( ! state.isLayoutComplete() ) - { - this.marker = j; - break; // out of for loop - } - } - } - else - { - // added on 11/28/2000, by Dresdner Bank, Germany - if(spacer != null) - area.removeChild(spacer); - - // removing something that was added by succession - // of cell.layout() - // just to keep my sanity here, Hani - area.increaseHeight(areaContainer.getHeight()); - area.removeChild(areaContainer); - this.resetMarker(); - this.removeID(area.getIDReferences()); - - // hani elabed 11/27/2000 - cellState.setLayoutComplete( false ); - - return status; - } - } - else // layout was complete for a particular cell - { // Hani Elabed - cellState.setLayoutComplete( true ); - } - - - - int h = cell.getHeight(); - if (h > largestCellHeight) - { - largestCellHeight = h; - } - } - - for (int i = 0; i < numChildren; i++) - { - TableCell cell = (TableCell)children.elementAt(i); - cell.setHeight(largestCellHeight); - } - - // added by Dresdner Bank, Germany - if(spacer != null) { - area.addChild(spacer); - spacer = null; - } + // cells is The list of cell states for this row. This is the location of + // where I will be storing the state of each cell so that I can + // spread a cell over multiple pages if I have to. This is part + // of fixing the TableRow larger than a single page bug. + // Hani Elabed, 11/22/2000. + if (cells == null)// do it once.. + { + widthOfCellsSoFar = 0; + cells = new Vector(); + int colCount = 0; + int numChildren = this.children.size(); + for (int i = 0; i < numChildren; i++) { + TableCell cell = (TableCell) children.elementAt(i); + cell.doSetup(areaContainer); + int numCols = cell.getNumColumnsSpanned(); + int numRows = cell.getNumRowsSpanned(); + int width = 0; + CellState state = + new CellState(i, false, widthOfCellsSoFar); + state.setColumn(colCount); + // add the state of a cell. + cells.insertElementAt(state, i); + for (int count = 0; + count < numCols && count < columns.size(); + count++) { + width += ((TableColumn) columns.elementAt(colCount)). + getColumnWidth(); + colCount++; + } + + cell.setWidth(width); + widthOfCellsSoFar += width; + + } + } + + int numChildren = this.children.size(); + // if (numChildren != columns.size()) { + // MessageHandler.errorln("WARNING: Number of children under table-row not equal to number of table-columns"); + // return new Status(Status.OK); + // } + + // added by Eric Schaeffer + largestCellHeight = 0; + + // added by Hani Elabed 11/27/2000 + boolean someCellDidNotLayoutCompletely = false; + + for (int i = this.marker; i < numChildren; i++) { + TableCell cell = (TableCell) children.elementAt(i); + + // added by Hani Elabed 11/22/2000 + CellState cellState = (CellState) cells.elementAt(i); + + //if (this.isInListBody) { + //fo.setIsInListBody(); + //fo.setDistanceBetweenStarts(this.distanceBetweenStarts); + //fo.setBodyIndent(this.bodyIndent); + //} + + + //--- this is modified to preserve the state of start + //--- offset of the cell. + //--- change by Hani Elabed 11/22/2000 + cell.setStartOffset(cellState.getWidthOfCellSoFar()); + + Status status; + if ((status = cell.layout(areaContainer)).isIncomplete()) { + this.marker = i; + /* if ((i != 0) && (status.getCode() == Status.AREA_FULL_NONE)) + { + status = new Status(Status.AREA_FULL_SOME); + }*/ + + + if (status.getCode() == Status.AREA_FULL_SOME) { + // this whole block added by + // Hani Elabed 11/27/2000 + + cellState.setLayoutComplete(false); + + // locate the first cell + // that need to be laid out further + + for (int j = 0; j < numChildren; j++) { + CellState state = (CellState) cells.elementAt(j); + + if (! state.isLayoutComplete()) { + this.marker = j; + break; // out of for loop + } + } + } else { + // added on 11/28/2000, by Dresdner Bank, Germany + if (spacer != null) + area.removeChild(spacer); + + // removing something that was added by succession + // of cell.layout() + // just to keep my sanity here, Hani + area.increaseHeight(areaContainer.getHeight()); + area.removeChild(areaContainer); + this.resetMarker(); + this.removeID(area.getIDReferences()); + + // hani elabed 11/27/2000 + cellState.setLayoutComplete(false); + + return status; + } + } else // layout was complete for a particular cell + { // Hani Elabed + cellState.setLayoutComplete(true); + } + + + + int h = cell.getHeight(); + if (h > largestCellHeight) { + largestCellHeight = h; + } + } + + for (int i = 0; i < numChildren; i++) { + TableCell cell = (TableCell) children.elementAt(i); + cell.setHeight(largestCellHeight); + } - area.addChild(areaContainer); - areaContainer.end(); - area.addDisplaySpace(largestCellHeight - + areaContainer.getPaddingTop() - + areaContainer.borderWidthTop - + areaContainer.getPaddingBottom() - + areaContainer.borderWidthBottom); - - // bug fix from Eric Schaeffer - //area.increaseHeight(largestCellHeight); - - if (spaceAfter != 0) { - area.addDisplaySpace(spaceAfter); - } - - if (area instanceof BlockArea) { - area.start(); - } - - - // test to see if some cells are not - // completely laid out. - // Hani Elabed 11/22/2000 - for (int i = 0; i < numChildren; i++) - { - CellState cellState = (CellState) cells.elementAt( i ); - - if( ! cellState.isLayoutComplete() ) - { - someCellDidNotLayoutCompletely = true; - break; // out of for loop - } - } - - - - - // replaced by Hani Elabed 11/27/2000 - //return new Status(Status.OK); - - if( someCellDidNotLayoutCompletely ) - { - return new Status(Status.AREA_FULL_SOME); - } - else - { - if (keepWithNext != 0) { - return new Status(Status.KEEP_WITH_NEXT); - } - return new Status(Status.OK); - } - - } + // added by Dresdner Bank, Germany + if (spacer != null) { + area.addChild(spacer); + spacer = null; + } + + area.addChild(areaContainer); + areaContainer.end(); + area.addDisplaySpace(largestCellHeight + + areaContainer.getPaddingTop() + + areaContainer.borderWidthTop + + areaContainer.getPaddingBottom() + + areaContainer.borderWidthBottom); + + // bug fix from Eric Schaeffer + //area.increaseHeight(largestCellHeight); + + if (spaceAfter != 0) { + area.addDisplaySpace(spaceAfter); + } + + if (area instanceof BlockArea) { + area.start(); + } + + + // test to see if some cells are not + // completely laid out. + // Hani Elabed 11/22/2000 + for (int i = 0; i < numChildren; i++) { + CellState cellState = (CellState) cells.elementAt(i); + + if (! cellState.isLayoutComplete()) { + someCellDidNotLayoutCompletely = true; + break; // out of for loop + } + } + + + + + // replaced by Hani Elabed 11/27/2000 + //return new Status(Status.OK); + + if (someCellDidNotLayoutCompletely) { + return new Status(Status.AREA_FULL_SOME); + } else { + if (keepWithNext != 0) { + return new Status(Status.KEEP_WITH_NEXT); + } + return new Status(Status.OK); + } + + } public int getAreaHeight() { - return areaContainer.getHeight(); + return areaContainer.getHeight(); } - public void removeLayout(Area area) - { - if(spacer != null) - area.removeChild(spacer); + public void removeLayout(Area area) { + if (spacer != null) + area.removeChild(spacer); - // removing something that was added by succession - // of cell.layout() - // just to keep my sanity here, Hani -// area.increaseHeight(areaContainer.getHeight()); + // removing something that was added by succession + // of cell.layout() + // just to keep my sanity here, Hani + // area.increaseHeight(areaContainer.getHeight()); area.removeChild(areaContainer); - this.resetMarker(); + this.resetMarker(); this.removeID(area.getIDReferences()); marker = 0; } -- 2.39.5