Skip to content

Commit

Permalink
Fixes #195: Make GetRows return value avoid empty cell
Browse files Browse the repository at this point in the history
* #195: proposed resolution to the issue

* Make GetRows return value avoid empty cell

* Update test file to fix broken testing.
  • Loading branch information
rafaelmiti authored and xuri committed Jan 1, 2019
1 parent fabd9d0 commit dea7ba0
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 10 deletions.
19 changes: 9 additions & 10 deletions rows.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,10 +31,9 @@ import (
//
func (f *File) GetRows(sheet string) [][]string {
xlsx := f.workSheetReader(sheet)
rows := [][]string{}
name, ok := f.sheetMap[trimSheetName(sheet)]
if !ok {
return rows
return [][]string{}
}
if xlsx != nil {
output, _ := xml.Marshal(f.Sheet[name])
Expand All @@ -44,15 +43,12 @@ func (f *File) GetRows(sheet string) [][]string {
d := f.sharedStringsReader()
var inElement string
var r xlsxRow
var row []string
tr, tc := f.getTotalRowsCols(name)
for i := 0; i < tr; i++ {
row = []string{}
for j := 0; j <= tc; j++ {
row = append(row, "")
}
rows = append(rows, row)
rows := make([][]string, tr)
for i := range rows {
rows[i] = make([]string, tc+1)
}
var row int
decoder := xml.NewDecoder(bytes.NewReader(f.readXML(name)))
for {
token, _ := decoder.Token()
Expand All @@ -70,12 +66,15 @@ func (f *File) GetRows(sheet string) [][]string {
c := TitleToNumber(strings.Map(letterOnlyMapF, colCell.R))
val, _ := colCell.getValueFrom(f, d)
rows[cr][c] = val
if val != "" {
row = r.R
}
}
}
default:
}
}
return rows
return rows[:row]
}

// Rows defines an iterator to a sheet
Expand Down
Binary file modified test/Book1.xlsx
Binary file not shown.

0 comments on commit dea7ba0

Please sign in to comment.