Issue 108860 - Validation can't work correctly.
Summary: Validation can't work correctly.
Status: CLOSED FIXED
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: lihuiibm
QA Contact: noel.power
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-03 02:43 UTC by lihuiibm
Modified: 2017-05-20 09:32 UTC (History)
3 users (show)

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

Attachments
Fix for this issue (2.11 KB, text/plain)
2010-02-03 02:45 UTC, lihuiibm 		no flags Details
Test sample for this issue. (40.00 KB, text/plain)
2010-03-12 15:50 UTC, lihuiibm 		no flags Details
Update to apply to Apache OO (1.60 KB, patch)
2012-02-07 21:44 UTC, Pedro Giffuni 		no flags Details | Diff
Show Obsolete (1) 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 02:43:26 UTC 
Steps:
    1.Open the sample file
    2.Click button "AddRangeTo10" and button "AddList" 
Defects:
    1. Cells "A2:B4" are applied with validation rule. In excel, if you input a
value( 1 <= Value <= 10 ), no error message pops up; In Symphony,  no matter
what the value is, an error alerts.

    2. Cells "A8:B9" are applied with validation list, but the list isn't
right.See the attachment.
Comment 1 lihuiibm 2010-02-03 02:44:32 UTC 
1. Root Cause:
In ScVbaValidation::Add(), the input parameter "Operator" of
Range.Validation.Add is not passed to the SC core function.
2. Resolution:
Add codes to pass the input parameter "Operator" to SC core function.
Comment 2 lihuiibm 2010-02-03 02:45:30 UTC 
Created attachment 67542 [details]
Fix for this issue
Comment 3 noel.power 2010-03-09 11:37:17 UTC 
the same document to test is missing, the patch looks ok though, please commit
and take of the upstreaming ( and add the test sample here too )
Comment 4 lihuiibm 2010-03-12 15:50:05 UTC 
Created attachment 68308 [details]
Test sample for this issue.
Comment 5 Pedro Giffuni 2012-02-07 21:44:04 UTC 
Created attachment 77192 [details]
Update to apply to Apache OO

This is yet untested but it looks good.
Comment 6 Pedro Giffuni 2012-02-10 17:09:15 UTC 
The test document didn't really work as intended: there is a command
button and nothing else(?).

I applied the patch and things seem to work ok.

Committed as revision 1242846: "Fix range validation. Patch by lihuiibm."

Thanks