You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by bu...@apache.org on 2014/09/04 14:25:15 UTC

[Bug 56914] New: XSSFRowShifter.updateConditionalFormatting throws IOOBE when there are more than 1 CTConditionalFormatting

https://issues.apache.org/bugzilla/show_bug.cgi?id=56914

            Bug ID: 56914
           Summary: XSSFRowShifter.updateConditionalFormatting throws
                    IOOBE when there are more than 1
                    CTConditionalFormatting
           Product: POI
           Version: 3.11-dev
          Hardware: All
                OS: All
            Status: NEW
          Severity: normal
          Priority: P2
         Component: XSSF
          Assignee: dev@poi.apache.org
          Reporter: yaniv@kundas.net

The iteration done over the sheet's CTConditionalFormatting objects possibly
uses ctWorksheet.removeConditionalFormatting(j), but does not update the
iteration index j.

For example, if there are 2 CTConditionalFormatting objects, and the first
iteration removes it (at index 0), the second one fails to get it, since index
1 is already out of bounds.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

[Bug 56914] XSSFRowShifter.updateConditionalFormatting throws IOOBE when there are more than 1 CTConditionalFormatting

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=56914

Andreas Beeker <an...@gmx.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Andreas Beeker <an...@gmx.de> ---
Thank you for the patch.
Applied with r1622759

-- 
You are receiving this mail because:
You are the assignee for the bug.

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

[Bug 56914] XSSFRowShifter.updateConditionalFormatting throws IOOBE when there are more than 1 CTConditionalFormatting

Posted by bu...@apache.org.
https://issues.apache.org/bugzilla/show_bug.cgi?id=56914

--- Comment #1 from Yaniv Kunda <ya...@kundas.net> ---
Created attachment 31964
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=31964&action=edit
56914

Fixes the iteration problem by using a backwards iteration, and using more
efficient one-time fetch of the XMLBeans array.
Modified BaseTestConditionalFormatting.testShiftRows to use two conditional
formatting objects.

-- 
You are receiving this mail because:
You are the assignee for the bug.

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