Sorry.  Should have moved this back to the ticket...

-----Original Message-----
From: Allison, Timothy B. [mailto:talli...@mitre.org] 
Sent: Wednesday, November 2, 2016 7:06 AM
To: POI Developers List <dev@poi.apache.org>
Subject: RE: [Bug 60329] New: Avoid NPE when styleid is null

Sounds good.  Do we want to auto-generate a styleid and risk collision with an 
actual styleid (chances would be very, very low) or handle the null?  Thank you.

-----Original Message-----
From: Mark Murphy [mailto:jmarkmur...@gmail.com]
Sent: Tuesday, November 1, 2016 5:25 PM
To: POI Developers List <dev@poi.apache.org>
Subject: Re: [Bug 60329] New: Avoid NPE when styleid is null

Seems to me that style ID is not optional because it is used to refer to the 
style from elsewhere in the document, and the style ID must also be unique. 
However, if the style ID is missing from the document, one should be assigned 
in any manner we choose, and it won't be able to be referenced.
That is ok as the document is mal-formed.

On Tue, Nov 1, 2016 at 2:55 PM, <bugzi...@apache.org> wrote:

> https://bz.apache.org/bugzilla/show_bug.cgi?id=60329
>
>             Bug ID: 60329
>            Summary: Avoid NPE when styleid is null
>            Product: POI
>            Version: 3.16-dev
>           Hardware: PC
>             Status: NEW
>           Severity: normal
>           Priority: P2
>          Component: XWPF
>           Assignee: dev@poi.apache.org
>           Reporter: talli...@mitre.org
>   Target Milestone: ---
>
> On TIKA-2152, Seva Alekseyev submitted a docx file that triggers an 
> NPE when trying to get the styleid.  If I read OOXML Ecma 1 [1] 
> correctly, styleid is optional.  We should try to avoid an NPE if the 
> styleid is null.
>
> [1]
> http://www.ecma-international.org/publications/files/ECMA-
> ST/ECMA-376,%20Fourth%20Edition,%20Part%201%20-%
> 20Fundamentals%20And%20Markup%20Language%20Reference.zip
> : pdf page 3861/printed page 3851
>
> --
> You are receiving this mail because:
> You are the assignee for the bug.
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional 
> commands, e-mail: dev-h...@poi.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org For additional commands, 
e-mail: dev-h...@poi.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Reply via email to