Issue 119403 - AOO 3.4 - GridControl not refreshing after deleting rows
Summary: AOO 3.4 - GridControl not refreshing after deleting rows
Status: CLOSED FIXED
Alias: None
Product: App Dev
Classification: Unclassified
Component: api (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: PC Windows 7
: P3 Normal
Target Milestone: ---
Assignee: jsc
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-22 16:57 UTC by wolfgang-becker
Modified: 2017-05-20 09:32 UTC (History)
6 users (show)

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


Attachments
CALC macro to display GridContol. Buttons to add / remove rows. (11.04 KB, application/vnd.oasis.opendocument.text)
2012-05-22 16:57 UTC, wolfgang-becker
no flags Details
Proposed patch to invalidate for transparency background (702 bytes, patch)
2012-06-09 09:17 UTC, hanya
hanya.runo: review?
Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description wolfgang-becker 2012-05-22 16:57:51 UTC
Created attachment 77580 [details]
CALC macro to display GridContol. Buttons to add / remove rows.

I use Windows 7 and AOO 3.4.

GridControl in user-dialog is not refreshing when rows are removed with BASIC-command "DeleteRow" within CALC-Macro. 

To check the problem see Calc-file with attached macro. Grid is only updated after minimizing und maximizing the Window on the desktop. 

With the same macro there is no repaint problem in Libre Office 3.5.

Procedure to reproduce the bug on the attached document:
- open the attached document and push button to show the dialog,
=> dialog having a grid control with two buttons is shown
- push "AddRow" button five times,
=> 5 rows are added
- push "RemoveRow" button three times,
=> the removed 3 rows are still displayed within grid.
Comment 1 Oliver Brinzing 2012-05-23 10:59:35 UTC
.
Comment 2 Du Jing 2012-05-25 09:51:40 UTC
Can reproduce in Aoo3.4.
Comment 3 hanya 2012-06-09 08:10:01 UTC
The grid window filled by background color on 3.3 but current grid uses 
transparent background, so it should be invalidated with INVALIDATE_TRANSPARENT 
in TableControl_Impl::invalidateRowRange, svtools/source/table/tablecontrol_impl.cxx .
Comment 4 hanya 2012-06-09 09:17:54 UTC
Created attachment 78209 [details]
Proposed patch to invalidate for transparency background

Described above.
Comment 5 jsc 2012-06-11 08:49:34 UTC
reviewed, built and tested on trunk, patch applied

trunk -> revision 1348750
Comment 6 Li Lin Yi 2012-06-18 09:53:25 UTC
Changed to VERIFIED. Verified on trunk r1350879.
Comment 7 Li Lin Yi 2012-06-19 07:19:45 UTC
Not fixed on branch, r1332941.
So I reopened it.
Comment 8 Li Lin Yi 2012-06-20 03:20:50 UTC
It is not a release blocker. Do not need to commit to branch.
So I changed to resolved again.
Comment 9 Li Lin Yi 2012-06-20 03:21:12 UTC
Changed to VERIFIED. Verified on trunk r1350879.
Comment 10 wolfgang-becker 2012-08-14 02:19:47 UTC
NOT FIXED with last developer build 1372282.
Comment 11 hanya 2012-08-14 02:59:49 UTC
(In reply to comment #10)
> NOT FIXED with last developer build 1372282.

the patch applied only trunk, not for branch AOO34
Comment 12 wolfgang-becker 2012-08-14 12:06:19 UTC
(In reply to comment #11)
> (In reply to comment #10)
> > NOT FIXED with last developer build 1372282.
> 
> the patch applied only trunk, not for branch AOO34

I wasn't familiar with the concept of trunk and branch yet.

Though it's not a release blocker, I think most people expect from next release (3.4.1) to fix atmost all bugs, that are detected; they do not want to wait until major release 3.5 or 4.0.

So I hope, Your patch for trunk will be soon applied to branch too - Sorry, the user is always impatient.
Comment 13 jsc 2012-08-14 15:19:17 UTC
every trivial bug fix contains a potential risk to break something. In a micro release as we have defined it, only show stopper and security fixes should be integrated. We have many open issues and many are already fixed on trunk. But we don't include them in 3.4.1 because of our definition what goes in a micro release and what not.

It will be fixed in 3.5 or 4.0, the next minor or major version depending on our decision what will be next.