ASF Bugzilla – Attachment 33291 Details for
Bug 58636
Upgrade ClientAnchor.get/setAnchorType to use enums
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
Remember
[x]
|
Forgot Password
Login:
[x]
[patch]
Add AnchorType enum inside ClientAnchor
bug58636.patch (text/plain), 26.98 KB, created by
Javen O'Neal
on 2015-11-24 11:03:50 UTC
(
hide
)
Description:
Add AnchorType enum inside ClientAnchor
Filename:
MIME Type:
Creator:
Javen O'Neal
Created:
2015-11-24 11:03:50 UTC
Size:
26.98 KB
patch
obsolete
>Index: src/java/org/apache/poi/ss/usermodel/ClientAnchor.java >=================================================================== >--- src/java/org/apache/poi/ss/usermodel/ClientAnchor.java (revision 1716101) >+++ src/java/org/apache/poi/ss/usermodel/ClientAnchor.java (working copy) >@@ -23,46 +23,68 @@ > * @author Yegor Kozlov > */ > public interface ClientAnchor { >+ >+ public static enum AnchorType { >+ /** >+ * Move and Resize With Anchor Cells (0) >+ * <p> >+ * Specifies that the current drawing shall move and >+ * resize to maintain its row and column anchors (i.e. the >+ * object is anchored to the actual from and to row and column) >+ * </p> >+ */ >+ MOVE_AND_RESIZE(0), >+ >+ /** >+ * Don't Move but do Resize With Anchor Cells (0) >+ * <p> >+ * Specifies that the current drawing shall not move with its >+ * row and column, but should be resized. This option is not normally >+ * used, but is included for completeness. >+ * </p> >+ */ >+ DONT_MOVE_DO_RESIZE(1), >+ >+ /** >+ * Move With Cells but Do Not Resize (2) >+ * <p> >+ * Specifies that the current drawing shall move with its >+ * row and column (i.e. the object is anchored to the >+ * actual from row and column), but that the size shall remain absolute. >+ * </p> >+ * <p> >+ * If additional rows/columns are added between the from and to locations of the drawing, >+ * the drawing shall move its to anchors as needed to maintain this same absolute size. >+ * </p> >+ */ >+ MOVE_DONT_RESIZE(2), >+ >+ /** >+ * Do Not Move or Resize With Underlying Rows/Columns (3) >+ * <p> >+ * Specifies that the current start and end positions shall >+ * be maintained with respect to the distances from the >+ * absolute start point of the worksheet. >+ * </p> >+ * <p> >+ * If additional rows/columns are added before the >+ * drawing, the drawing shall move its anchors as needed >+ * to maintain this same absolute position. >+ * </p> >+ */ >+ DONT_MOVE_AND_RESIZE(3); >+ >+ public final short value; >+ AnchorType(int value) { >+ this.value = (short) value; >+ } >+ >+ public static AnchorType byId(int value) { >+ return values()[value]; >+ } >+ } >+ > /** >- * Move and Resize With Anchor Cells >- * <p> >- * Specifies that the current drawing shall move and >- * resize to maintain its row and column anchors (i.e. the >- * object is anchored to the actual from and to row and column) >- * </p> >- */ >- public static final int MOVE_AND_RESIZE = 0; >- >- /** >- * Move With Cells but Do Not Resize >- * <p> >- * Specifies that the current drawing shall move with its >- * row and column (i.e. the object is anchored to the >- * actual from row and column), but that the size shall remain absolute. >- * </p> >- * <p> >- * If additional rows/columns are added between the from and to locations of the drawing, >- * the drawing shall move its to anchors as needed to maintain this same absolute size. >- * </p> >- */ >- public static final int MOVE_DONT_RESIZE = 2; >- >- /** >- * Do Not Move or Resize With Underlying Rows/Columns >- * <p> >- * Specifies that the current start and end positions shall >- * be maintained with respect to the distances from the >- * absolute start point of the worksheet. >- * </p> >- * <p> >- * If additional rows/columns are added before the >- * drawing, the drawing shall move its anchors as needed >- * to maintain this same absolute position. >- * </p> >- */ >- public static final int DONT_MOVE_AND_RESIZE = 3; >- >- /** > * Returns the column (0 based) of the first cell. > * > * @return 0-based column of the first cell. >@@ -209,26 +231,14 @@ > > /** > * Sets the anchor type >- * <p> >- * 0 = Move and size with Cells, 2 = Move but don't size with cells, 3 = Don't move or size with cells. >- * </p> >- * @param anchorType the anchor type >- * @see #MOVE_AND_RESIZE >- * @see #MOVE_DONT_RESIZE >- * @see #DONT_MOVE_AND_RESIZE >+ * @param anchorType the anchor type to set > */ >- public void setAnchorType( int anchorType ); >+ public void setAnchorType( AnchorType anchorType ); > > /** > * Gets the anchor type >- * <p> >- * 0 = Move and size with Cells, 2 = Move but don't size with cells, 3 = Don't move or size with cells. >- * </p> > * @return the anchor type >- * @see #MOVE_AND_RESIZE >- * @see #MOVE_DONT_RESIZE >- * @see #DONT_MOVE_AND_RESIZE > */ >- public int getAnchorType(); >+ public AnchorType getAnchorType(); > > } >Index: src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFClientAnchor.java >=================================================================== >--- src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFClientAnchor.java (revision 1716101) >+++ src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFClientAnchor.java (working copy) >@@ -28,7 +28,8 @@ > * @author Yegor Kozlov > */ > public final class XSSFClientAnchor extends XSSFAnchor implements ClientAnchor { >- private int anchorType; >+ private AnchorType DEFAULT_ANCHOR_TYPE = AnchorType.MOVE_AND_RESIZE; >+ private AnchorType anchorType; > > /** > * Starting anchor point >@@ -44,6 +45,7 @@ > * Creates a new client anchor and defaults all the anchor positions to 0. > */ > public XSSFClientAnchor() { >+ anchorType = DEFAULT_ANCHOR_TYPE; > cell1 = CTMarker.Factory.newInstance(); > cell1.setCol(0); > cell1.setColOff(0); >@@ -88,6 +90,7 @@ > * @param cell2 ending anchor point > */ > protected XSSFClientAnchor(CTMarker cell1, CTMarker cell2) { >+ anchorType = DEFAULT_ANCHOR_TYPE; > this.cell1 = cell1; > this.cell2 = cell2; > } >@@ -214,10 +217,10 @@ > > /** > * Sets the anchor type >- * <p> >- * 0 = Move and size with Cells, 2 = Move but don't size with cells, 3 = Don't move or size with cells. >+ * @param anchorType the anchor type to set > */ >- public void setAnchorType( int anchorType ) >+ @Override >+ public void setAnchorType( AnchorType anchorType ) > { > this.anchorType = anchorType; > } >@@ -224,10 +227,10 @@ > > /** > * Gets the anchor type >- * <p> >- * 0 = Move and size with Cells, 2 = Move but don't size with cells, 3 = Don't move or size with cells. >+ * @return the anchor type > */ >- public int getAnchorType() >+ @Override >+ public AnchorType getAnchorType() > { > return anchorType; > } >Index: src/java/org/apache/poi/hssf/usermodel/HSSFClientAnchor.java >=================================================================== >--- src/java/org/apache/poi/hssf/usermodel/HSSFClientAnchor.java (revision 1716101) >+++ src/java/org/apache/poi/hssf/usermodel/HSSFClientAnchor.java (working copy) >@@ -250,20 +250,20 @@ > > /** > * Gets the anchor type >- * <p/> >- * 0 = Move and size with Cells, 2 = Move but don't size with cells, 3 = Don't move or size with cells. >+ * @return the anchor type > */ >- public int getAnchorType() { >- return _escherClientAnchor.getFlag(); >+ @Override >+ public AnchorType getAnchorType() { >+ return AnchorType.byId(_escherClientAnchor.getFlag()); > } > > /** > * Sets the anchor type >- * <p/> >- * 0 = Move and size with Cells, 2 = Move but don't size with cells, 3 = Don't move or size with cells. >+ * @param anchorType the anchor type to set > */ >- public void setAnchorType(int anchorType) { >- _escherClientAnchor.setFlag(Integer.valueOf(anchorType).shortValue()); >+ @Override >+ public void setAnchorType(AnchorType anchorType) { >+ _escherClientAnchor.setFlag(anchorType.value); > } > > private void checkRange(int value, int minRange, int maxRange, String varName) { >Index: src/testcases/org/apache/poi/hssf/model/TestHSSFAnchor.java >=================================================================== >--- src/testcases/org/apache/poi/hssf/model/TestHSSFAnchor.java (revision 1716101) >+++ src/testcases/org/apache/poi/hssf/model/TestHSSFAnchor.java (working copy) >@@ -21,6 +21,7 @@ > import org.apache.poi.ddf.*; > import org.apache.poi.hssf.HSSFTestDataSamples; > import org.apache.poi.hssf.usermodel.*; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > > /** > * @author Evgeniy Berlog >@@ -30,7 +31,7 @@ > > public void testDefaultValues(){ > HSSFClientAnchor clientAnchor = new HSSFClientAnchor(); >- assertEquals(clientAnchor.getAnchorType(), 0); >+ assertEquals(clientAnchor.getAnchorType(), AnchorType.MOVE_AND_RESIZE); > assertEquals(clientAnchor.getCol1(), 0); > assertEquals(clientAnchor.getCol2(), 0); > assertEquals(clientAnchor.getDx1(), 0); >@@ -41,7 +42,7 @@ > assertEquals(clientAnchor.getRow2(), 0); > > clientAnchor = new HSSFClientAnchor(new EscherClientAnchorRecord()); >- assertEquals(clientAnchor.getAnchorType(), 0); >+ assertEquals(clientAnchor.getAnchorType(), AnchorType.MOVE_AND_RESIZE); > assertEquals(clientAnchor.getCol1(), 0); > assertEquals(clientAnchor.getCol2(), 0); > assertEquals(clientAnchor.getDx1(), 0); >@@ -143,7 +144,7 @@ > > HSSFPatriarch drawing = sheet.createDrawingPatriarch(); > HSSFClientAnchor anchor = new HSSFClientAnchor(10, 10, 200, 200, (short)2, 2, (short)15, 15); >- anchor.setAnchorType(2); >+ anchor.setAnchorType(AnchorType.MOVE_DONT_RESIZE); > > HSSFSimpleShape rectangle = drawing.createSimpleShape(anchor); > rectangle.setShapeType(HSSFSimpleShape.OBJECT_TYPE_RECTANGLE); >@@ -362,9 +363,9 @@ > clientAnchor2.setRow2(7); > assertEquals(clientAnchor1, clientAnchor2); > >- clientAnchor2.setAnchorType(3); >+ clientAnchor2.setAnchorType(AnchorType.DONT_MOVE_AND_RESIZE); > assertNotSame(clientAnchor1, clientAnchor2); >- clientAnchor2.setAnchorType(0); >+ clientAnchor2.setAnchorType(AnchorType.MOVE_AND_RESIZE); > assertEquals(clientAnchor1, clientAnchor2); > > HSSFChildAnchor childAnchor1 = new HSSFChildAnchor(0, 1, 2, 3); >Index: src/java/org/apache/poi/hssf/model/ComboboxShape.java >=================================================================== >--- src/java/org/apache/poi/hssf/model/ComboboxShape.java (revision 1716101) >+++ src/java/org/apache/poi/hssf/model/ComboboxShape.java (working copy) >@@ -21,6 +21,7 @@ > import org.apache.poi.ddf.*; > import org.apache.poi.hssf.record.*; > import org.apache.poi.hssf.usermodel.*; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > > /** > * Represents a combobox shape. >@@ -92,7 +93,7 @@ > opt.addEscherProperty(new EscherSimpleProperty(EscherProperties.GROUPSHAPE__PRINT, 0x00020000)); > > HSSFClientAnchor userAnchor = (HSSFClientAnchor) shape.getAnchor(); >- userAnchor.setAnchorType(1); >+ userAnchor.setAnchorType(AnchorType.DONT_MOVE_DO_RESIZE); > EscherRecord anchor = createAnchor(userAnchor); > clientData.setRecordId(EscherClientDataRecord.RECORD_ID); > clientData.setOptions((short) 0x0000); >Index: src/java/org/apache/poi/hssf/model/ConvertAnchor.java >=================================================================== >--- src/java/org/apache/poi/hssf/model/ConvertAnchor.java (revision 1716101) >+++ src/java/org/apache/poi/hssf/model/ConvertAnchor.java (working copy) >@@ -38,7 +38,7 @@ > EscherClientAnchorRecord anchor = new EscherClientAnchorRecord(); > anchor.setRecordId( EscherClientAnchorRecord.RECORD_ID ); > anchor.setOptions( (short) 0x0000 ); >- anchor.setFlag( (short) a.getAnchorType() ); >+ anchor.setFlag( a.getAnchorType().value ); > anchor.setCol1( (short) Math.min(a.getCol1(), a.getCol2()) ); > anchor.setDx1( (short) a.getDx1() ); > anchor.setRow1( (short) Math.min(a.getRow1(), a.getRow2()) ); >Index: src/java/org/apache/poi/hssf/usermodel/HSSFCombobox.java >=================================================================== >--- src/java/org/apache/poi/hssf/usermodel/HSSFCombobox.java (revision 1716101) >+++ src/java/org/apache/poi/hssf/usermodel/HSSFCombobox.java (working copy) >@@ -19,6 +19,7 @@ > > import org.apache.poi.ddf.*; > import org.apache.poi.hssf.record.*; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > > /** > * >@@ -61,7 +62,7 @@ > opt.addEscherProperty(new EscherSimpleProperty(EscherProperties.GROUPSHAPE__PRINT, 0x00020000)); > > HSSFClientAnchor userAnchor = (HSSFClientAnchor) getAnchor(); >- userAnchor.setAnchorType(1); >+ userAnchor.setAnchorType(AnchorType.DONT_MOVE_DO_RESIZE); > EscherRecord anchor = userAnchor.getEscherAnchor(); > clientData.setRecordId(EscherClientDataRecord.RECORD_ID); > clientData.setOptions((short) 0x0000); >Index: src/java/org/apache/poi/hssf/usermodel/HSSFPicture.java >=================================================================== >--- src/java/org/apache/poi/hssf/usermodel/HSSFPicture.java (revision 1716101) >+++ src/java/org/apache/poi/hssf/usermodel/HSSFPicture.java (working copy) >@@ -34,6 +34,7 @@ > import org.apache.poi.hssf.record.CommonObjectDataSubRecord; > import org.apache.poi.hssf.record.EscherAggregate; > import org.apache.poi.hssf.record.ObjRecord; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > import org.apache.poi.ss.usermodel.Picture; > import org.apache.poi.ss.util.ImageUtils; > import org.apache.poi.util.POILogFactory; >@@ -134,7 +135,7 @@ > */ > public void resize(double scaleX, double scaleY) { > HSSFClientAnchor anchor = getClientAnchor(); >- anchor.setAnchorType(2); >+ anchor.setAnchorType(AnchorType.MOVE_DONT_RESIZE); > > HSSFClientAnchor pref = getPreferredSize(scaleX,scaleY); > >Index: src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFDrawing.java >=================================================================== >--- src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFDrawing.java (revision 1716101) >+++ src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFDrawing.java (working copy) >@@ -34,7 +34,6 @@ > import org.apache.poi.ss.usermodel.ClientAnchor; > import org.apache.poi.ss.usermodel.Drawing; > import org.apache.poi.ss.util.CellAddress; >-import org.apache.poi.ss.util.CellReference; > import org.apache.poi.util.Internal; > import org.apache.poi.util.Units; > import org.apache.poi.xssf.model.CommentsTable; >@@ -373,9 +372,9 @@ > anchor.setFrom(ctAnchor.getFrom()); > STEditAs.Enum aditAs; > switch(anchor.getAnchorType()) { >- case ClientAnchor.DONT_MOVE_AND_RESIZE: aditAs = STEditAs.ABSOLUTE; break; >- case ClientAnchor.MOVE_AND_RESIZE: aditAs = STEditAs.TWO_CELL; break; >- case ClientAnchor.MOVE_DONT_RESIZE: aditAs = STEditAs.ONE_CELL; break; >+ case DONT_MOVE_AND_RESIZE: aditAs = STEditAs.ABSOLUTE; break; >+ case MOVE_AND_RESIZE: aditAs = STEditAs.TWO_CELL; break; >+ case MOVE_DONT_RESIZE: aditAs = STEditAs.ONE_CELL; break; > default: aditAs = STEditAs.ONE_CELL; > } > ctAnchor.setEditAs(aditAs); >Index: src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFPicture.java >=================================================================== >--- src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFPicture.java (revision 1716101) >+++ src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFPicture.java (working copy) >@@ -23,7 +23,7 @@ > import java.util.List; > > import org.apache.poi.ss.usermodel.BaseTestPicture; >-import org.apache.poi.ss.usermodel.ClientAnchor; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > import org.apache.poi.util.LocaleUtil; > import org.apache.poi.xssf.XSSFITestDataProvider; > import org.apache.poi.xssf.XSSFTestDataSamples; >@@ -70,9 +70,9 @@ > assertArrayEquals(jpegData, pictures.get(jpegIdx).getData()); > > XSSFClientAnchor anchor = new XSSFClientAnchor(0, 0, 0, 0, 1, 1, 10, 30); >- assertEquals(ClientAnchor.MOVE_AND_RESIZE, anchor.getAnchorType()); >- anchor.setAnchorType(ClientAnchor.DONT_MOVE_AND_RESIZE); >- assertEquals(ClientAnchor.DONT_MOVE_AND_RESIZE, anchor.getAnchorType()); >+ assertEquals(AnchorType.MOVE_AND_RESIZE, anchor.getAnchorType()); >+ anchor.setAnchorType(AnchorType.DONT_MOVE_AND_RESIZE); >+ assertEquals(AnchorType.DONT_MOVE_AND_RESIZE, anchor.getAnchorType()); > > XSSFPicture shape = drawing.createPicture(anchor, jpegIdx); > assertTrue(anchor.equals(shape.getAnchor())); >Index: src/testcases/org/apache/poi/hssf/model/TestDrawingShapes.java >=================================================================== >--- src/testcases/org/apache/poi/hssf/model/TestDrawingShapes.java (revision 1716101) >+++ src/testcases/org/apache/poi/hssf/model/TestDrawingShapes.java (working copy) >@@ -35,6 +35,7 @@ > import org.apache.poi.hssf.record.EscherAggregate; > import org.apache.poi.hssf.record.ObjRecord; > import org.apache.poi.hssf.usermodel.*; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > import org.apache.poi.ss.usermodel.Workbook; > import org.apache.poi.util.HexDump; > >@@ -148,8 +149,8 @@ > > HSSFPatriarch drawing = sheet.createDrawingPatriarch(); > HSSFClientAnchor anchor = new HSSFClientAnchor(10, 10, 50, 50, (short) 2, 2, (short) 4, 4); >- anchor.setAnchorType(2); >- assertEquals(anchor.getAnchorType(), 2); >+ anchor.setAnchorType(AnchorType.MOVE_DONT_RESIZE); >+ assertEquals(AnchorType.MOVE_DONT_RESIZE, anchor.getAnchorType()); > > HSSFSimpleShape rectangle = drawing.createSimpleShape(anchor); > rectangle.setShapeType(HSSFSimpleShape.OBJECT_TYPE_RECTANGLE); >Index: src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java >=================================================================== >--- src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java (revision 1716101) >+++ src/testcases/org/apache/poi/hssf/usermodel/TestBugs.java (working copy) >@@ -70,6 +70,7 @@ > import org.apache.poi.ss.usermodel.BaseTestBugzillaIssues; > import org.apache.poi.ss.usermodel.Cell; > import org.apache.poi.ss.usermodel.CellStyle; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > import org.apache.poi.ss.usermodel.DataFormatter; > import org.apache.poi.ss.usermodel.FormulaEvaluator; > import org.apache.poi.ss.usermodel.Name; >@@ -2966,7 +2967,7 @@ > assertNotNull("Did not find sheet", sheet); > HSSFPatriarch patriarch = (HSSFPatriarch) sheet.createDrawingPatriarch(); > HSSFClientAnchor anchor = new HSSFClientAnchor(0, 0, 0, 0, (short) 1, 1, (short) 10, 22); >- anchor.setAnchorType(2); >+ anchor.setAnchorType(AnchorType.MOVE_DONT_RESIZE); > patriarch.createPicture(anchor, pict); > > // Write out destination file >Index: src/testcases/org/apache/poi/hssf/usermodel/TestHSSFPatriarch.java >=================================================================== >--- src/testcases/org/apache/poi/hssf/usermodel/TestHSSFPatriarch.java (revision 1716101) >+++ src/testcases/org/apache/poi/hssf/usermodel/TestHSSFPatriarch.java (working copy) >@@ -18,6 +18,7 @@ > package org.apache.poi.hssf.usermodel; > > import org.apache.poi.hssf.HSSFTestDataSamples; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > > import junit.framework.AssertionFailedError; > import junit.framework.TestCase; >@@ -56,7 +57,7 @@ > > // 3. Use patriarch > HSSFClientAnchor anchor = new HSSFClientAnchor(0, 0, 600, 245, (short) 1, 1, (short) 1, 2); >- anchor.setAnchorType(3); >+ anchor.setAnchorType(AnchorType.DONT_MOVE_AND_RESIZE); > byte[] pictureData = HSSFTestDataSamples.getTestDataFileContent("logoKarmokar4.png"); > int idx1 = wb.addPicture(pictureData, HSSFWorkbook.PICTURE_TYPE_PNG); > patr.createPicture(anchor, idx1); >Index: src/testcases/org/apache/poi/hssf/usermodel/TestOLE2Embeding.java >=================================================================== >--- src/testcases/org/apache/poi/hssf/usermodel/TestOLE2Embeding.java (revision 1716101) >+++ src/testcases/org/apache/poi/hssf/usermodel/TestOLE2Embeding.java (working copy) >@@ -31,7 +31,7 @@ > import org.apache.poi.poifs.filesystem.DirectoryNode; > import org.apache.poi.poifs.filesystem.Ole10Native; > import org.apache.poi.poifs.filesystem.POIFSFileSystem; >-import org.apache.poi.ss.usermodel.ClientAnchor; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > import org.apache.poi.ss.usermodel.CreationHelper; > import org.apache.poi.util.LocaleUtil; > import org.junit.Test; >@@ -88,25 +88,25 @@ > CreationHelper ch = wb1.getCreationHelper(); > HSSFClientAnchor anchor = (HSSFClientAnchor)ch.createClientAnchor(); > anchor.setAnchor((short)(2+coloffset), 1+rowoffset, 0, 0, (short)(3+coloffset), 5+rowoffset, 0, 0); >- anchor.setAnchorType(ClientAnchor.DONT_MOVE_AND_RESIZE); >+ anchor.setAnchorType(AnchorType.DONT_MOVE_AND_RESIZE); > > patriarch.createObjectData(anchor, pptIdx, imgPPT); > > anchor = (HSSFClientAnchor)ch.createClientAnchor(); > anchor.setAnchor((short)(5+coloffset), 1+rowoffset, 0, 0, (short)(6+coloffset), 5+rowoffset, 0, 0); >- anchor.setAnchorType(ClientAnchor.DONT_MOVE_AND_RESIZE); >+ anchor.setAnchorType(AnchorType.DONT_MOVE_AND_RESIZE); > > patriarch.createObjectData(anchor, xlsIdx, imgIdx); > > anchor = (HSSFClientAnchor)ch.createClientAnchor(); > anchor.setAnchor((short)(3+coloffset), 10+rowoffset, 0, 0, (short)(5+coloffset), 11+rowoffset, 0, 0); >- anchor.setAnchorType(ClientAnchor.DONT_MOVE_AND_RESIZE); >+ anchor.setAnchorType(AnchorType.DONT_MOVE_AND_RESIZE); > > patriarch.createObjectData(anchor, txtIdx, imgIdx); > > anchor = (HSSFClientAnchor)ch.createClientAnchor(); > anchor.setAnchor((short)(1+coloffset), -2+rowoffset, 0, 0, (short)(7+coloffset), 14+rowoffset, 0, 0); >- anchor.setAnchorType(ClientAnchor.DONT_MOVE_AND_RESIZE); >+ anchor.setAnchorType(AnchorType.DONT_MOVE_AND_RESIZE); > > HSSFSimpleShape circle = patriarch.createSimpleShape(anchor); > circle.setShapeType(HSSFSimpleShape.OBJECT_TYPE_OVAL); >Index: src/examples/src/org/apache/poi/hssf/usermodel/examples/AddDimensionedImage.java >=================================================================== >--- src/examples/src/org/apache/poi/hssf/usermodel/examples/AddDimensionedImage.java (revision 1716101) >+++ src/examples/src/org/apache/poi/hssf/usermodel/examples/AddDimensionedImage.java (working copy) >@@ -31,6 +31,7 @@ > import org.apache.poi.hssf.usermodel.HSSFClientAnchor; > import org.apache.poi.hssf.usermodel.HSSFPatriarch; > import org.apache.poi.hssf.util.CellReference; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > > > /** >@@ -291,7 +292,7 @@ > // image as the size of the row/column is adjusted. This could easilly > // become another parameter passed to the method. > //anchor.setAnchorType(HSSFClientAnchor.DONT_MOVE_AND_RESIZE); >- anchor.setAnchorType(HSSFClientAnchor.MOVE_AND_RESIZE); >+ anchor.setAnchorType(AnchorType.MOVE_AND_RESIZE); > > // Now, add the picture to the workbook. Note that the type is assumed > // to be a JPEG/JPG, this could easily (and should) be parameterised >Index: src/examples/src/org/apache/poi/hssf/usermodel/examples/OfficeDrawing.java >=================================================================== >--- src/examples/src/org/apache/poi/hssf/usermodel/examples/OfficeDrawing.java (revision 1716101) >+++ src/examples/src/org/apache/poi/hssf/usermodel/examples/OfficeDrawing.java (working copy) >@@ -18,6 +18,7 @@ > package org.apache.poi.hssf.usermodel.examples; > > import org.apache.poi.hssf.usermodel.*; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > > import java.io.*; > >@@ -152,15 +153,15 @@ > > HSSFClientAnchor anchor; > anchor = new HSSFClientAnchor(0,0,0,255,(short)2,2,(short)4,7); >- anchor.setAnchorType( 2 ); >+ anchor.setAnchorType( AnchorType.MOVE_DONT_RESIZE ); > patriarch.createPicture(anchor, loadPicture( "src/resources/logos/logoKarmokar4.png", wb )); > > anchor = new HSSFClientAnchor(0,0,0,255,(short)4,2,(short)5,7); >- anchor.setAnchorType( 2 ); >+ anchor.setAnchorType( AnchorType.MOVE_DONT_RESIZE ); > patriarch.createPicture(anchor, loadPicture( "src/resources/logos/logoKarmokar4edited.png", wb )); > > anchor = new HSSFClientAnchor(0,0,1023,255,(short)6,2,(short)8,7); >- anchor.setAnchorType( 2 ); >+ anchor.setAnchorType( AnchorType.MOVE_DONT_RESIZE ); > HSSFPicture picture = patriarch.createPicture(anchor, loadPicture( "src/resources/logos/logoKarmokar4s.png", wb )); > //Reset the image to the original size. > picture.resize(); >Index: src/examples/src/org/apache/poi/ss/examples/AddDimensionedImage.java >=================================================================== >--- src/examples/src/org/apache/poi/ss/examples/AddDimensionedImage.java (revision 1716101) >+++ src/examples/src/org/apache/poi/ss/examples/AddDimensionedImage.java (working copy) >@@ -28,6 +28,7 @@ > import org.apache.poi.hssf.usermodel.HSSFWorkbook; > import org.apache.poi.hssf.usermodel.HSSFSheet; > import org.apache.poi.ss.usermodel.ClientAnchor; >+import org.apache.poi.ss.usermodel.ClientAnchor.AnchorType; > import org.apache.poi.ss.usermodel.Drawing; > import org.apache.poi.ss.usermodel.Row; > import org.apache.poi.ss.usermodel.Sheet; >@@ -373,10 +374,10 @@ > anchor.setRow2(rowClientAnchorDetail.getToIndex()); > > // For now, set the anchor type to do not move or resize the >- // image as the size of the row/column is adjusted. This could easilly >+ // image as the size of the row/column is adjusted. This could easily > // become another parameter passed to the method. Please read the note > // above regarding the behaviour of image resizing. >- anchor.setAnchorType(ClientAnchor.MOVE_AND_RESIZE); >+ anchor.setAnchorType(AnchorType.MOVE_AND_RESIZE); > > // Now, add the picture to the workbook. Note that unlike the similar > // method in the HSSF Examples section, the image type is checked. First, >@@ -400,7 +401,7 @@ > } > > /** >- * Determines whether the sheets columns should be re-sized to accomodate >+ * Determines whether the sheets columns should be re-sized to accommodate > * the image, adjusts the columns width if necessary and creates then > * returns a ClientAnchorDetail object that facilitates construction of > * an ClientAnchor that will fix the image on the sheet and establish >@@ -407,7 +408,7 @@ > * it's size. > * > * @param sheet A reference to the sheet that will 'contain' the image. >- * @param colNumber A primtive int that contains the index number of a >+ * @param colNumber A primitive int that contains the index number of a > * column on the sheet. > * @param reqImageWidthMM A primitive double that contains the required > * width of the image in millimetres
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
Attachments on
bug 58636
:
33291
|
33292