[Bug 63158] New: sheet.getLastRowNum() and sheet.getRows() are broken

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

[Bug 63158] New: sheet.getLastRowNum() and sheet.getRows() are broken

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

            Bug ID: 63158
           Summary: sheet.getLastRowNum() and sheet.getRows() are broken
           Product: POI
           Version: 4.0.0-FINAL
          Hardware: PC
            Status: NEW
          Severity: critical
          Priority: P2
         Component: XSSF
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

sheet.getLastRowNum() and sheet.getRows() are broken

sheet.getLastRowNum() returns actual rows number - 1 (empty rows are not
accounted - I's OK)

sheet.getRows() returns all rows except last one

--
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 63158] sheet.getLastRowNum() and sheet.getRows() are broken

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

Dzmitry Kliuchnik <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|                            |All

--- Comment #1 from Dzmitry Kliuchnik <[hidden email]> ---
4.0.1 version

--
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 63158] sheet.getLastRowNum() and sheet.getRows() are 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=63158

--- Comment #2 from PJ Fanning <[hidden email]> ---
*** Bug 63157 has been marked as a duplicate of this bug. ***

--
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 63158] sheet.getLastRowNum() and sheet.getRows() are 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=63158

Yegor Kozlov <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #3 from Yegor Kozlov <[hidden email]> ---
getLastRowNum() is 0-based and it is reflected in the documentation.

There isn't such a method getRows() in the Sheet interface. Do you mean that
iteration over rows skips the last row? I don't think it is so and we have
numerous unit tests that prove it.
If you still think there is a bug, can you please attach a unit test
demonstrating the problem?

Regards,
Yegor

--
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 63158] sheet.getLastRowNum() and sheet.getRows() are 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=63158

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |WORKSFORME
             Status|NEEDINFO                    |RESOLVED

--- Comment #4 from Dominik Stadler <[hidden email]> ---
We could not see anything being wrong in Apache POI here and no more
information provided for a week, thus I am resolving this as WORKSFORME for
now.

If you still think there is a problem, then please reopen with some
code-sample, ideally as self-sufficient unit-test which allows to reproduce
your problem so we can see the actual problem more easily.

--
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]