[Bug 61727] New: [PATCH] Merge cell ranges broken

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 61727] New: [PATCH] Merge cell ranges broken

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61727

            Bug ID: 61727
           Summary: [PATCH] Merge cell ranges broken
           Product: POI
           Version: 3.16-FINAL
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: SS Common
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

Created attachment 35496
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35496&action=edit
patch created using "ant -f patch.xml"

The merge functionality of cell ranges in
org.apache.poi.ss.util.CellRangeUtil.mergeCellRanges(CellRangeAddress[]) is
broken: some of the provided input cell ranges get lost during merge under
circumstances.

Example: Merge of cell ranges (A1), (B1) and (A2) results in cell ranges
(A1:A2). (B1) gets lost.

The Patch provides test cases revealing the broken merge functionality and
testing its fix in CellRangeUtil. The patch was created using "ant -f
patch.xml".

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61727] [PATCH] CellRangeUtil Merge cell ranges broken

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61727

Javen O'Neal <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[PATCH] Merge cell ranges   |[PATCH] CellRangeUtil Merge
                   |broken                      |cell ranges broken
                 OS|                            |All

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 61727] [PATCH] CellRangeUtil Merge cell ranges broken

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=61727

--- Comment #1 from Javen O'Neal <[hidden email]> ---
Applied in r1814432. Thanks for the patch.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]