]> source.dussan.org Git - poi.git/commitdiff
rolling back r781622
authorJosh Micich <josh@apache.org>
Thu, 4 Jun 2009 00:06:31 +0000 (00:06 +0000)
committerJosh Micich <josh@apache.org>
Thu, 4 Jun 2009 00:06:31 +0000 (00:06 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@781623 13f79535-47bb-0310-9956-ffa450edef68

src/java/org/apache/poi/hssf/dev/BiffViewer.java
src/java/org/apache/poi/hssf/usermodel/HSSFName.java
src/java/org/apache/poi/ss/formula/FormulaParser.java
src/java/org/apache/poi/ss/usermodel/Cell.java
src/java/org/apache/poi/ss/usermodel/Sheet.java
src/testcases/org/apache/poi/hssf/usermodel/TestHSSFName.java

index 484dd58143ac861a28a1f713c36e85c3275484b2..264aa6d7f811c623fd82d82b63ed84ec9a93df85 100644 (file)
@@ -155,7 +155,6 @@ public final class BiffViewer {
                        case EndRecord.sid:            return new EndRecord(in);
                        case ExtSSTRecord.sid:         return new ExtSSTRecord(in);
                        case ExtendedFormatRecord.sid: return new ExtendedFormatRecord(in);
-                       case ExternalNameRecord.sid:   return new ExternalNameRecord(in);
                        case ExternSheetRecord.sid:    return new ExternSheetRecord(in);
                        case FilePassRecord.sid:       return new FilePassRecord(in);
                        case FileSharingRecord.sid:    return new FileSharingRecord(in);
@@ -359,7 +358,6 @@ public final class BiffViewer {
         *
         */
        public static void main(String[] args) {
-               args = new String[] { "--out", "c:/josh/temp/ex47199-23710-twoPSBs-x.xls", };
 
                CommandArgs cmdArgs;
                try {
index 1c21c0f5b126aa3f32c3ec6726517ad8bf84d36f..bceaf340adf9e120854752a93d56dea5a0bf3e1f 100644 (file)
@@ -21,10 +21,8 @@ import org.apache.poi.hssf.model.HSSFFormulaParser;
 import org.apache.poi.hssf.model.Workbook;
 import org.apache.poi.hssf.record.NameRecord;
 import org.apache.poi.hssf.record.formula.Ptg;
-import org.apache.poi.ss.SpreadsheetVersion;
 import org.apache.poi.ss.formula.FormulaType;
 import org.apache.poi.ss.usermodel.Name;
-import org.apache.poi.ss.util.CellReference;
 
 /**
  * High Level Representation of a 'defined name' which could be a 'built-in' name,
@@ -135,37 +133,13 @@ public final class HSSFName implements Name {
         }
     }
 
-    static void validateName(String name){
-        if(name.length() == 0) {
-                       throw new IllegalArgumentException("Name cannot be blank");
-               }
+    private static void validateName(String name){
+        if(name.length() == 0)  throw new IllegalArgumentException("Name cannot be blank");
         
         char c = name.charAt(0);
         if(!(c == '_' || Character.isLetter(c)) || name.indexOf(' ') != -1) {
             throw new IllegalArgumentException("Invalid name: '"+name+"'; Names must begin with a letter or underscore and not contain spaces");
         }
-        if (true) {
-               return;
-        }
-        int looksLikeType = CellReference.classifyCellReference(name, SpreadsheetVersion.EXCEL97);
-        String looksLikeStr;
-        switch (looksLikeType) {
-               case CellReference.NameType.NAMED_RANGE:
-                       // all OK
-                       return;
-               case CellReference.NameType.CELL:
-                       looksLikeStr = "cell reference";
-                       break;
-               case CellReference.NameType.COLUMN:
-                       looksLikeStr = "column reference";
-                       break;
-               default:
-                       looksLikeStr = null;
-        }
-               if (looksLikeType != CellReference.NameType.NAMED_RANGE) {
-               throw new IllegalArgumentException("Specified name '" + name + "' is invalid"
-                               + (looksLikeStr == null ? "" : " because it looks like a " + looksLikeStr));
-        }
     }
 
     /**
index 5366b6028dfcf7d68705e964d0035e88d562473a..ec2252d5e054523d20f2c3fb0b23bb533129aee1 100644 (file)
@@ -557,7 +557,7 @@ public final class FormulaParser {
                // which will either be named ranges or functions
                StringBuilder sb = new StringBuilder();
 
-               if (!isValidDefinedNameChar(look)) {
+               if (!Character.isLetter(look)) {
                        throw expected("number, string, or defined name");
                }
                while (isValidDefinedNameChar(look)) {
index 0a76061e45901a481abb8b2e7fe51df270fbd046..aa1e8c39552e0b5b8797837a11de41c97d739960 100644 (file)
@@ -149,19 +149,11 @@ public interface Cell {
     void setCellValue(double value);
 
     /**
-     * Converts the supplied date to its equivalent Excel numeric value and sets
-     * that into the cell.
-     * <p/>
-     * <b>Note</b> - There is actually no 'DATE' cell type in Excel. In many
-     * cases (when entering date values), Excel automatically adjusts the
-     * <i>cell style</i> to some date format, creating the illusion that the cell
-     * data type is now something besides {@link Cell#CELL_TYPE_NUMERIC}.  POI
-     * does not attempt to replicate this behaviour.  To make a numeric cell
-     * display as a date, use {@link #setCellStyle(CellStyle)} etc.
-     * 
-     * @param value the numeric value to set this cell to.  For formulas we'll set the
-     *        precalculated value, for numerics we'll set its value. For other types we
-     *        will change the cell to a numerics cell and set its value.
+     * Set a boolean value for the cell
+     *
+     * @param value the boolean value to set this cell to.  For formulas we'll set the
+     *        precalculated value, for booleans we'll set its value. For other types we
+     *        will change the cell to a boolean cell and set its value.
      */
     void setCellValue(Date value);
 
index 9ee99a4f2acfed088f19737bfbe5d0cc29e80a87..ffeb96abda8d80f7b2f3e740a832d8bf2c7387e4 100644 (file)
@@ -406,8 +406,8 @@ public interface Sheet extends Iterable<Row> {
     PrintSetup getPrintSetup();
 
     /**
-     * Gets the user model for the default document header.  If it does not exist, the footer will
-     * be created. See {@link #hasHeader()} and {@link #removeHeader()}. <br/>
+     * Gets the user model for the default document header.
+     * <p>
      * Note that XSSF offers more kinds of document headers than HSSF does
      * </p>
      * @return the document header.
@@ -415,33 +415,12 @@ public interface Sheet extends Iterable<Row> {
     Header getHeader();
 
     /**
-     * @return <code>true</code> if this sheet has a header 
-     */
-//    boolean hasHeader();
-
-    /**
-     * Removes the header if it exists. If it does not exist, this method does nothing.
-     */
-//    void removeHeader();
-
-    /**
-     * Gets the user model for the default document footer.  If it does not exist, the footer will
-     * be created. See {@link #hasFooter()} and {@link #removeFooter()}. <br/>
+     * Gets the user model for the default document footer.
      * Note that XSSF offers more kinds of document footers than HSSF does.
      *
      * @return the document footer.
      */
     Footer getFooter();
-    
-    /**
-     * @return <code>true</code> if this sheet has a footer 
-     */
-//    boolean hasFooter();
-
-    /**
-     * Removes the footer if it exists. If it does not exist, this method does nothing.
-     */
-//    void removeFooter();
 
     /**
      * Sets a flag indicating whether this sheet is selected.
index ee97c35d469912eb631c20a54d6e01c6223b6f20..88a2c0cf16953939ccb1b9ac61de3ff070b016fe 100755 (executable)
@@ -29,7 +29,6 @@ import org.apache.poi.hssf.record.NameRecord;
 import org.apache.poi.hssf.record.formula.Ptg;
 import org.apache.poi.ss.formula.FormulaType;
 import org.apache.poi.ss.usermodel.BaseTestNamedRange;
-import org.apache.poi.ss.usermodel.Name;
 import org.apache.poi.ss.util.AreaReference;
 
 /**
@@ -251,11 +250,6 @@ public final class TestHSSFName extends BaseTestNamedRange {
         for (int i = 0; i < ptgs.length; i++) {
             assertEquals('R', ptgs[i].getRVAType());
         }
+
     }
-    
-    public void testValidNames() {
-       Name n = getTestDataProvider().createWorkbook().createName();
-               n.setNameName("B00000001");
-               
-       }
 }
\ No newline at end of file