GetRows() does not work on 1.8.1

[pztrn]

% go version
go version go1.8.1 linux/amd64

Code:

package main

import (
        "fmt"
        "log"
        "github.com/Luxurioust/excelize"
)

func main() {
        xlsx, err1 := excelize.OpenFile("/tmp/1.xlsx")
        if err1 != nil {
                log.Fatal("Failed to open XLSX file: " + err1.Error())
        }

        // We will always use first sheet.
        sheet_name := xlsx.GetSheetName(1)
        log.Println("Will use sheet: '" + sheet_name + "'")

        // Read file data.
        //file_data := make(map[string]string)
        rows := xlsx.GetRows(sheet_name)
        for _, row := range rows {
                fmt.Println(row)
        }
}

[pztrn]

Same on go 1.8.

Tested with files created in MS Office 2013 and Libreoffice 5.3.2.

[xuri]

@pztrn Thanks for your issue. Function GetRows() doesn't support get rows by sheet name currently, it need to name "Sheet" + index. Relate issue #25.

[pztrn]

Why documentation says opposing?

https://godoc.org/github.com/Luxurioust/excelize#File.GetRows

[xuri]

Sorry. I'll fix the ambiguity of the doc.

[pztrn]

I think for now it should be something like:

For now you should use sheet_name like "sheet3" where "sheet" is a constant part and "3" is a sheet number. For example, if sheet named as "SomeUniqueData" and it is second if spreadsheet program interface - you should use "sheet2" here.