You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by bu...@apache.org on 2009/01/22 05:20:01 UTC

DO NOT REPLY [Bug 46581] New: [PATCH] excel text extraction by cell type

https://issues.apache.org/bugzilla/show_bug.cgi?id=46581

           Summary: [PATCH] excel text extraction by cell type
           Product: POI
           Version: 3.5-dev
          Platform: Macintosh
        OS/Version: Mac OS X 10.4
            Status: NEW
          Severity: normal
          Priority: P2
         Component: POI Overall
        AssignedTo: dev@poi.apache.org
        ReportedBy: woody@xoopit.com


Created an attachment (id=23154)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23154)
patch for bug 46576

addresses bug: 46576

affects these files:
M      src/java/org/apache/poi/hssf/usermodel/HSSFCell.java
M      src/java/org/apache/poi/hssf/extractor/ExcelExtractor.java
M      src/java/org/apache/poi/hssf/extractor/EventBasedExcelExtractor.java
M     
src/ooxml/interfaces-jdk15/org/apache/poi/ss/extractor/ExcelExtractor.java
M      src/ooxml/java/org/apache/poi/xssf/extractor/XSSFExcelExtractor.java


augment ExcelExtractor interface to support extracting cells by type.
EventBasedExcelExtractor implements ExcelExtractor interface.
ensures HSSFCell constants match those of Cell.
cleanup some stringbuffer->stringbuilder code.

bz2, standard patch format: svn diff > mydiff.txt ; bzip2 mydiff.txt


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org

DO NOT REPLY [Bug 46581] [PATCH] excel text extraction by cell type

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46581


woody <wo...@xoopit.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #23154|0                           |1
        is obsolete|                            |




--- Comment #1 from woody <wo...@xoopit.com>  2009-01-21 21:44:14 PST ---
Created an attachment (id=23156)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=23156)
bzip2 patch for bug 46576

prev patch is no good.


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org

[Bug 46581] [PATCH] excel text extraction by cell type

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46581

Nick Burch <ap...@gagravarr.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |WONTFIX

--- Comment #2 from Nick Burch <ap...@gagravarr.org> ---
It isn't clear to me what the use-case is for this.

Given how long it has been since this patch was proposed, the fact that the use
isn't clear, and that the patch no longer applies cleanly, I'm going to close
this as Won't Fix

If someone can explain the use case, and why this logic belongs in the text
extractor, please re-open the bug and comment

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org

[Bug 46581] [PATCH] excel text extraction by cell type

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46581

Dominik Stadler <do...@gmx.at> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |PatchAvailable

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org

DO NOT REPLY [Bug 46581] [PATCH] excel text extraction by cell type

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=46581


woody <wo...@xoopit.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #23154|patch for bug 46576         |BAD PATCH
        description|                            |




-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org