Issue 78664 - event "item status changed" is fired twice
Summary: event "item status changed" is fired twice
Status: CLOSED FIXED
Alias: None
Product: App Dev
Classification: Unclassified
Component: api (show other issues)
Version: 3.3.0 or older (OOo)
Hardware: All All
: P3 Trivial
Target Milestone: ---
Assignee: marc.neumann
QA Contact: issues@api
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-19 18:15 UTC by mroe
Modified: 2013-02-24 21:07 UTC (History)
3 users (show)

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

Attachments
sample for "item status changed" is fired twice (11.39 KB, application/vnd.oasis.opendocument.text)
2007-06-19 18:16 UTC, mroe 		no flags Details
suggested patch (533 bytes, patch)
2007-07-02 11:40 UTC, Frank Schönheit 		no flags Details | Diff
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 mroe 2007-06-19 18:15:11 UTC 
I placed a pushbutton at a form with toggle = TRUE, so it can have state = 0 and
state = 1. At the event "item status changed" i registered a sub - but it is
performed twice. :-(

See into the attached sample please.

mroe
Comment 1 mroe 2007-06-19 18:16:15 UTC 
Created attachment 46061 [details]
sample for "item status changed" is fired twice
Comment 2 cno 2007-06-19 23:22:05 UTC 
Hi mroe,
Have you tried the second event in the list (I guess it is "When called/started"
or something like that in English). That launches you code once.

I assume "item status changed" is launched when button is pressed and when the
button is released.
Does this help?
Cor
Comment 3 jsc 2007-07-02 09:16:09 UTC 
jsc -> fs: it looks like a bug for me, can you take a look on it.
Comment 4 Frank Schönheit 2007-07-02 11:40:28 UTC 
Created attachment 46424 [details]
suggested patch
Comment 5 Frank Schönheit 2007-07-02 11:44:20 UTC 
fs->pl: Please review the attached patch. Reasoning for this patch:
The flow is as follows:

PushButton::Tracking( end_tracking ) does (if it is a Toggle button) a Check,
which calls a Toggle. Then, PushButton::Tracking calls a Toggle, again. Thus,
two VCLEVENT_PUSHBUTTON_TOGGLE are fired.
Comment 6 philipp.lohmann 2007-07-02 12:08:50 UTC 
seems to be correct
Comment 7 Frank Schönheit 2007-07-02 20:53:21 UTC 
patch committed to CWS dba23b
Comment 8 Frank Schönheit 2007-07-12 21:49:10 UTC 
fs-> msc: please verify in CWS dba23b
Comment 9 Frank Schönheit 2007-07-12 22:15:36 UTC 
targeting to 2.3, since the fix is part of a CWS aiming for this release
Comment 10 marc.neumann 2007-07-18 10:24:19 UTC 
verified in CWS dba23b

find more information about this CWS, like when it is available in the master
builds, in EIS, the Environment Information System:
http://eis.services.openoffice.org/EIS2/cws.ShowCWS?Path=SRC680%2Fdba23b
Comment 11 marc.neumann 2007-08-07 14:23:07 UTC 
Hi,

this is fixed in the current master. The current master is available at
http://download.openoffice.org/680/index.html

I close this issue now.

Bye Marc