Issue 8399 - Edit controls allow to set text longer than the max text length (in some circumstances)
Summary: Edit controls allow to set text longer than the max text length (in some circ...
Status: CLOSED FIXED
Alias: None
Product: gsl
Classification: Code
Component: code (show other issues)
Version: current
Hardware: PC All
: P3 Trivial (vote)
Target Milestone: OOo 1.1 Beta
Assignee: Frank Schönheit
QA Contact: issues@gsl
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-10-16 12:05 UTC by Frank Schönheit
Modified: 2003-03-13 11:11 UTC (History)
1 user (show)

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

Attachments
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 Frank Schönheit 2002-10-16 12:05:17 UTC 
Sometimes, the edit control (vcl/source/control/edit.cxx) does allow to set a
text which is longer than the MaxTextLen specified.
In Edit::ImplSetText, there is either ImplInsertText called, which respects
mnMaxTextLen.
But in case of "if ( mnXOffset || HasPaintEvent() )" being true, the text is set
immediately, without checking mnMaxTextLen.

Discovered this during investigating bug 6118, where after fixing the original
bug in DBA, a strange effect remained, depending on some repaint timing.
Comment 1 malte_timmermann 2002-12-19 16:22:44 UTC 
Right...
Comment 2 Frank Schönheit 2002-12-19 17:47:08 UTC 
targeting for OOo 1.1 Beta - a consequence of this bug is that under
some circumstances, wrong data is displayed in forms - this is worht
this target, IMO
Comment 3 malte_timmermann 2003-01-13 15:59:23 UTC 
...
Comment 4 malte_timmermann 2003-01-23 10:21:53 UTC 
Same behavior like in ImplInsertText now: Text will NOT be accepted 
when longer than MaxTextLen.
Comment 5 malte_timmermann 2003-02-05 10:53:22 UTC 
Reopen for sending to QA
Comment 6 malte_timmermann 2003-02-05 10:54:29 UTC 
MT->FS: Please verify.
Sorry, don't know who in QA could do that...
Comment 7 Frank Schönheit 2003-02-11 16:49:09 UTC 
verified - setting to FIXED in a first step
Comment 8 Frank Schönheit 2003-02-11 16:49:37 UTC 
marking as VERIFIED
Comment 9 michael.bemmer 2003-03-13 11:11:00 UTC 
As mentioned on the qa dev list on March 5th I will close all resolved
<wontfix/duplicate/worksforme/invalid> issues. Please see this posting for details.