Issue 108885 - [From Symphony]Support new API, WorksheetFunction.IsNumber, WorksheetFunction.IsText, WorksheetFunction.IsNoText
Summary: [From Symphony]Support new API, WorksheetFunction.IsNumber, WorksheetFunction...
Status: CONFIRMED
Alias: None
Product: App Dev
Classification: Unclassified
Component: vba (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-03 07:17 UTC by lihuiibm
Modified: 2017-05-20 11:29 UTC (History)
1 user (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Fix for this issue (1.61 KB, text/plain)
2010-02-03 07:17 UTC, lihuiibm 		no flags Details
This patch remove the windows crlf(s), please ignore the old one and use this patch (1.60 KB, patch)
2010-02-26 07:19 UTC, lihuiibm 		no flags Details | Diff
Test sample file for this issue (43.50 KB, text/plain)
2010-03-15 14:42 UTC, lihuiibm 		no flags Details
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description lihuiibm 2010-02-03 07:17:01 UTC 
Support new API, WorksheetFunction.IsNumber, WorksheetFunction.IsText,
WorksheetFunction.IsNoText.
1. Root Cause:
Fopllowing APIs: WorksheetFunction.IsNumber, IsText, IsNoText return the wrong
result. It return a sequences.
2. Resolution:
The return value of WorksheetFunction.IsNumber, IsText, IsNoText should be
Boolean, so convert the sequences value to a boolean
Comment 1 lihuiibm 2010-02-03 07:17:25 UTC 
Created attachment 67565 [details]
Fix for this issue
Comment 2 noel.power 2010-02-22 20:33:37 UTC 
same  comments as for issue 108868
and again please if you can, try to make sure there are no windows crlf(s)
attached patch
Comment 3 lihuiibm 2010-02-26 07:11:14 UTC 
Noel, as you said, you have the same comments for this issue with the comments
for issue 108868. Your comments for issue 108868 is: "patch looks reasonable to
me"(I am not sure if you point to a wrong issue id). If you think this patch is
OK, I will deliver it.(I will update the patch to remove the windows crlf).
Comment 4 lihuiibm 2010-02-26 07:19:16 UTC 
Created attachment 68034 [details]
This patch remove the windows crlf(s), please ignore the old one and use this patch
Comment 5 noel.power 2010-02-26 09:20:16 UTC 
>(I am not sure if you point to a wrong issue id).
yes, sorry my mistake :-(  that is the wrong issue, the issue I wanted to point
to was issue 108870
Comment 6 lihuiibm 2010-03-15 14:42:04 UTC 
Created attachment 68353 [details]
Test sample file for this issue
Comment 7 noel.power 2010-03-16 12:31:37 UTC 
as discussed, issue #108870 need to use the excellent lcl_ConvertResult method
provided here
Comment 8 Marcus 2017-05-20 11:29:21 UTC 
Reset assigne to the default "issues@openoffice.apache.org".