diff options
| author | NaturalGao <43291304+NaturalGao@users.noreply.github.com> | 2022-08-19 23:24:13 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-08-19 23:24:13 +0800 |
| commit | 76f336809f5419343702de5b3284d46feb9ed266 (patch) | |
| tree | 207546e0abb538b3fb6d646b046b4c919e4c5711 /excelize.go | |
| parent | d1e76fc432ac5c9bde99591ec5e88e46b62d9c3d (diff) | |
This closes #849, add new function `DeleteComment` for delete comment (#1317)
- Update unit tests for the delete comment
- Add 3 errors function for error messages
Diffstat (limited to 'excelize.go')
| -rw-r--r-- | excelize.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/excelize.go b/excelize.go index ef438dd..f3b4381 100644 --- a/excelize.go +++ b/excelize.go @@ -231,7 +231,7 @@ func (f *File) workSheetReader(sheet string) (ws *xlsxWorksheet, err error) { ok bool ) if name, ok = f.getSheetXMLPath(sheet); !ok { - err = fmt.Errorf("sheet %s is not exist", sheet) + err = newNoExistSheetError(sheet) return } if worksheet, ok := f.Sheet.Load(name); ok && worksheet != nil { @@ -240,7 +240,7 @@ func (f *File) workSheetReader(sheet string) (ws *xlsxWorksheet, err error) { } for _, sheetType := range []string{"xl/chartsheets", "xl/dialogsheet", "xl/macrosheet"} { if strings.HasPrefix(name, sheetType) { - err = fmt.Errorf("sheet %s is not a worksheet", sheet) + err = newNotWorksheetError(sheet) return } } @@ -251,7 +251,7 @@ func (f *File) workSheetReader(sheet string) (ws *xlsxWorksheet, err error) { } if err = f.xmlNewDecoder(bytes.NewReader(namespaceStrictToTransitional(f.readBytes(name)))). Decode(ws); err != nil && err != io.EOF { - err = fmt.Errorf("xml decode error: %s", err) + err = newDecodeXMLError(err) return } err = nil @@ -424,7 +424,7 @@ func (f *File) UpdateLinkedValue() error { for _, name := range f.GetSheetList() { ws, err := f.workSheetReader(name) if err != nil { - if err.Error() == fmt.Sprintf("sheet %s is not a worksheet", trimSheetName(name)) { + if err.Error() == newNotWorksheetError(name).Error() { continue } return err |