p30928647
/
excelize
1
0
Fork 1
Code Issues Pull Requests Projects Releases 23 Wiki Activity

- Fix `DeleteSheet()` make broken file, relate issue #165; 

- godoc updated
This commit is contained in:
Ri Xu 2017-12-04 23:54:32 +08:00
parent 6d634ca320
commit 749d12eece
No known key found for this signature in database
GPG Key ID: BA5E5BB1C948EDF7
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
sheet.go
View File

@ -195,8 +195,10 @@ func replaceRelationshipsNameSpace(workbookMarshal string) string {
return strings.Replace(workbookMarshal, oldXmlns, newXmlns, -1) return strings.Replace(workbookMarshal, oldXmlns, newXmlns, -1)
} }
// SetActiveSheet provides function to set default active sheet of XLSX by given // SetActiveSheet provides function to set default active worksheet of XLSX by
// index. // given index. Note that active index is different with the index that got by
// function GetSheetMap, and it should be greater than 0 and less than total
// worksheet numbers.
func (f *File) SetActiveSheet(index int) { func (f *File) SetActiveSheet(index int) {
if index < 1 { if index < 1 {
index = 1 index = 1
@ -227,7 +229,6 @@ func (f *File) SetActiveSheet(index int) {
} }
} }
} }
return
} }
// GetActiveSheetIndex provides function to get active sheet of XLSX. If not // GetActiveSheetIndex provides function to get active sheet of XLSX. If not
@ -385,6 +386,7 @@ func (f *File) DeleteSheet(name string) {
f.SheetCount-- f.SheetCount--
} }
} }
f.SetActiveSheet(len(f.GetSheetMap()))
} }
// deleteSheetFromWorkbookRels provides function to remove worksheet // deleteSheetFromWorkbookRels provides function to remove worksheet