Skip to content

fix POI Bug 60197 - setSheetOrder doesn't update names scope#1014

Merged
tonyqus merged 1 commit into
nissl-lab:masterfrom
9rnsr:Workbook_SetSheetOrder
Feb 24, 2023
Merged

fix POI Bug 60197 - setSheetOrder doesn't update names scope#1014
tonyqus merged 1 commit into
nissl-lab:masterfrom
9rnsr:Workbook_SetSheetOrder

Conversation

@9rnsr
Copy link
Copy Markdown
Contributor

@9rnsr 9rnsr commented Feb 4, 2023

This PR fixes issue #931. In practice, I've encountered it too when I tried to change sheet order after setting PrintArea in each sheets.

Referenced POI source code version: POI 5.2.3


POI Bug Tracker issue:

https://bz.apache.org/bugzilla/show_bug.cgi?id=60197

Tracked POI changes:

(comment: Actual logic fix was only in first revision, rests were test code adjustments.)

Referenced POI source code version: POI 5.2.3

POI Bug Tracker issue:

  https://bz.apache.org/bugzilla/show_bug.cgi?id=60197

Tracked POI changes:

- bug 60197: Workbook#setSheetOrder should update named range sheet indices

  https://svn.apache.org/repos/asf/poi/trunk@1763939

- sonar fixes - use assert in junit tests

  https://svn.apache.org/repos/asf/poi/trunk@1872397

- #65026 - Migrate tests to Junit 5

  https://svn.apache.org/repos/asf/poi/trunk@1884783
@9rnsr 9rnsr force-pushed the Workbook_SetSheetOrder branch from 6063eed to d03dff9 Compare February 4, 2023 15:01
@tonyqus tonyqus added the bug label Feb 18, 2023
@tonyqus tonyqus added this to the NPOI 2.6.1 milestone Feb 18, 2023
@tonyqus
Copy link
Copy Markdown
Member

tonyqus commented Feb 24, 2023

LGTM

@tonyqus tonyqus merged commit 4449ef8 into nissl-lab:master Feb 24, 2023
@9rnsr
Copy link
Copy Markdown
Contributor Author

9rnsr commented Feb 25, 2023

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants