Bug 27586 - HSSFRequest.addListener doesn't work for >1 listener in 2.5 FINAL
Summary: HSSFRequest.addListener doesn't work for >1 listener in 2.5 FINAL
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: HSSF (show other bugs)
Version: 2.0-FINAL
Hardware: PC Windows XP
: P3 normal with 1 vote (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-03-10 22:25 UTC by Jelai Wang
Modified: 2013-08-04 19:43 UTC (History)
0 users


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jelai Wang 2004-03-10 22:25:05 UTC 
I did a bugzilla query on this bug, but it doesn't appear to be submitted yet.

The bug occurs when using the Event API for HSSF in
org.apache.poi.hssf.eventusermodel.  For each record type, the first listener
that is added to HSSFRequest gets registered, but subsequent listeners (for a
record type with an existing listener) do not get registered.

If you look at org.apache.poi.hssf.eventusermodel.HSSFRequest.addListener() line
74 the cause is obvious.  The fix, of course, is to add the following line:

list.add(lsnr);

Seems too trivial to submit a test case to reproduce this bug, so I'll leave it
out.  Don't want to offend your intelligence.  It's just surprising to me that
this hasn't been submitted yet.

Hope this helps.  Keep up the good work, I'm looking forward to seeing a
memory-tweaked POI 3.0 because I manipulate large spreadsheets.

-jw
Comment 1 Dominik Stadler 2013-08-04 19:43:24 UTC 
This is fixed since ages, at least as long as I could look up SVN history...