DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=42474>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=42474

           Summary: Patches for a couple of bugs in HSLF
           Product: POI
           Version: unspecified
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P2
         Component: HSLF
        AssignedTo: poi-dev@jakarta.apache.org
        ReportedBy: [EMAIL PROTECTED]


1. Incorrect matching of notes to slides

This is a long standing problem apparently, I noticed a TODO comment about 
it... 
anyway, after a bit of trial and error the problem seems to be that for some 
PPT 
files the SlidePersistAtom from the note's SlideAtomSet has a wrong 
slideIdentifier sometimes, so matching it to the associated slide is 
impossible. 
But if you look at the NotesAtom from the actual Notes record (retrieved via 
getCoreRecordForSAS()) it has the correct slideId, so that's what I use in my 
patch.

I don't know yet why SlideAtomSet.getSlidePersistAtom().getSlideIdentifier() is 
different from Notes.getNotesAtom().getSlideId(), maybe that's the real bug and 
my patch is merely working around it? Someone more knowledgeable could shed 
some 
light on whether these two values are supposed to be always equal, in the 
meantime this patch works perfectly for all my test cases which previously 
exhibited incorrect note->slide association.


2. NPE in RichTextRun.isBold() when the RichTextRun comes from a Notes model 
object

The Notes object wasn't doing setSheet(this) on the text runs it returned, 
causing a NPE inside RichTextRun.isCharFlagsTextPropVal() when any style 
accessor is called, because it tries to get the style info from the master 
sheet 
if not present. However, doing setSheet() in Notes was not enough, it merely 
moved the NPE a couple of lines down because NotesMaster objects are not yet 
implemented in HSLF, so I had to add a few null checks in RichTextRun too.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
Mailing List:    http://jakarta.apache.org/site/mail2.html#poi
The Apache Jakarta POI Project: http://jakarta.apache.org/poi/

Reply via email to