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 2007/12/11 03:14:24 UTC

DO NOT REPLY [Bug 44050] New: - [PATCH] Trim function in formula cell

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=44050>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44050

           Summary: [PATCH] Trim function in formula cell
           Product: POI
           Version: unspecified
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P5
         Component: HSSF
        AssignedTo: dev@poi.apache.org
        ReportedBy: wilson@cbio.mskcc.org


I have implemented the TRIM function for formula cells.  In Excel TRIM is
defined as "Removes all spaces from text except for single spaces between words.".  

I have also filled in the TRIM "FORMULA" and "EXPECTED VALUE" rows of the file
src/scratchpad/testcases/org/apache/poi/hssf/data/FormulaEvalTestData.xls with
15 test cases.  These test cases include passing TRIM a reference to a cell with
an empty string, passing TRIM numbers, passing TRIM an area, and others.  All
tests pass.

I would like to implement more formula functions, so any feedback on this one
would be greatly appreciated.  I plan to try MID next.

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

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

DO NOT REPLY [Bug 44050] - [PATCH] Trim function in formula cell

Posted by bu...@apache.org.
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=44050>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44050





------- Additional Comments From wilson@cbio.mskcc.org  2007-12-10 18:16 -------
Created an attachment (id=21254)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=21254&action=view)
Generated using svn diff > FormulaTrimFunction.patch

This is a patch file, generated by running:

svn diff > FormulaTrimFunction.patch

After I finished implementing the TRIM function.

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

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

DO NOT REPLY [Bug 44050] - [PATCH] Trim function in formula cell

Posted by bu...@apache.org.
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=44050>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44050


nick@torchbox.com changed:

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




------- Additional Comments From nick@torchbox.com  2007-12-11 05:05 -------
Thanks, committed

One thing to note is that not all of the contents of FormulaEvalTestData.xls
isn't checked automatically. Once your function is working, you need to list the
row number in
src/scratchpad/testcases/org/apache/poi/hssf/record/formula/eval/TestEverything.java
, and then it'll get tested for you (I've added the Trim row number)

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

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

DO NOT REPLY [Bug 44050] - [PATCH] Trim function in formula cell

Posted by bu...@apache.org.
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=44050>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44050





------- Additional Comments From nick@torchbox.com  2007-12-12 02:31 -------
Ah, you're right.

I've added some comments on this to the top of the two test files, which will
hopefully clear up this confusion for the next person!

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

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

DO NOT REPLY [Bug 44050] - [PATCH] Trim function in formula cell

Posted by bu...@apache.org.
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=44050>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44050





------- Additional Comments From wilson@cbio.mskcc.org  2007-12-10 18:22 -------
Created an attachment (id=21255)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=21255&action=view)
Contains the files I modified to implement and test the TRIM function.

This tar file contains two files that I modified to implement and test the TRIM
formula function.  

The first file is:

src/scratchpad/src/org/apache/poi/hssf/record/formula/functions/Trim.java

This is where the TRIM function is implemented.

The second file is:

src/scratchpad/testcases/org/apache/poi/hssf/data/FormulaEvalTestData.xls

I modified two rows in this file (rows 1392 and 1393), the "FORMULA" and
"EXPECTED VALUE" rows for the trim function.  This row contains the test cases
for trim.  These tests run successfully and are run automatically when the ant
task "test" is executed.

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

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

DO NOT REPLY [Bug 44050] - [PATCH] Trim function in formula cell

Posted by bu...@apache.org.
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=44050>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=44050





------- Additional Comments From wilson@cbio.mskcc.org  2007-12-11 10:21 -------
Thanks Nick.  Actually, I was wondering about the testing.  Trim was being tested in:

src/scratchpad/testcases/org/apache/poi/hssf/record/formula/functions/TestEverything.java

Which appears to set up a new GenericFormulaTestCase for every function listed in 
FormulaEvalTestData.xls between rows 80 and 1481.  Row 80 marks the beginning of the Excel 
functions section.

I know it was testing trim successfully because while developing I kept running:

ant -Dtestcase=org.apache.poi.hssf.record.formula.functions.TestEverything single-scratchpad-test

Which was failing on trim for a while.

I was wondering why some functions are listed explicitly in the file you mentioned, 
src/scratchpad/testcases/org/apache/poi/hssf/record/formula/eval/TestEverything.java, when the 
same test is run in 
src/scratchpad/testcases/org/apache/poi/hssf/record/formula/functions/TestEverything.java.  So far 
the only overlaps are concatenate, int, trim and upper.  I suspect that TestEverything.java in the eval 
package was meant only to test things like "=B7<B8" and not in fact the excel functions.

(In reply to comment #3)
> Thanks, committed
> 
> One thing to note is that not all of the contents of FormulaEvalTestData.xls
> isn't checked automatically. Once your function is working, you need to list the
> row number in
> src/scratchpad/testcases/org/apache/poi/hssf/record/formula/eval/TestEverything.java
> , and then it'll get tested for you (I've added the Trim row number)




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

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