not a valid zip file

[justdoitgo]

wps

[xuri]

What's the version of the Go and Excelize you are using? Please provides environment details (OS, WPS version) and file attachments if you can and provide detailed information according to the issue template when creating an issue.

[vinceyoumans]

I am also getting an error when I try to load this xlsx file.
err=====================
zip: not a valid zip file

`//package PBCONVxlsx2csv03
package main

import (
"fmt"

"github.com/360EntSecGroup-Skylar/excelize"

)

func main() {
//excelFileName := "EPlan_AP_LastSixMonths.xlsx"
//excelFileName := "./EPlan_AP_LastSixMonths.xlsx"
//excelFileName := "EPlan_SubmitRequest_LastSixMonths.xlsx"
excelFileName := "./EPlan_SubmitRequest_LastSixMonths.xlsx"

//xlsx, err := excelize.OpenFile("./Book1.xlsx")
xlsx, err := excelize.OpenFile(excelFileName)
if err != nil {
	fmt.Println("err=====================")
	fmt.Println(err)
	return
}
// Get value from cell by given worksheet name and axis.
cell := xlsx.GetCellValue("Sheet1", "B2")
fmt.Println(cell)
// Get all the rows in the Sheet1.
rows := xlsx.GetRows("Sheet1")
for _, row := range rows {
	for _, colCell := range row {
		fmt.Print(colCell, "\t")
	}
	fmt.Println()
}

}
`

/Users/vyoumans/Documents/PBeachChallenge/EPlan_AP_LastSixMonths.xlsx
EPlan_AP_LastSixMonths.xlsx

EPlan_AP_LastSixMonths.xlsx
EPlan_AP_LastSixMonths.xlsx

[navono]

I am also have the same problem.

OS: win10 1803
Go: 1.10.2
Excelize: master

OpenFile xls file get zip: not a valid zip file error. I copy the file content to a new xlsx file, the OpenFile successes.

[xuri]

@vinceyoumans Thanks for your issue. Please make sure your attachment EPlan_AP_LastSixMonths.xlsx can be open by Microsoft Excel™ application.

[navono]

对比了下xls和xlsx文件，发现它们的文件起始的格式不一样。zip包的findSignatureInBlock会去判断前面的字符来确认文件格式。所以导致加载xls失败。
1.xlsx

上面1.xlsx是Office 2013生成的。在上传xls的过程中，居然也出现了下面的格式不正确的错误。

[calvinchankf]

@navono thanks for your info, so what is your way to workaround it?

[navono]

@calvinchankf for now, copy the content to a new .xls or xlsx file by hand....

[calvinchankf]

Lol thanks for sharing 😂 I will request my clients to export a xlsx

…

On Wed, Jun 20, 2018 at 21:23 Darkness ***@***.***> wrote: @calvinchankf <https://github.com/calvinchankf> for now, copy the content to a new .xls file by hand.... — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#207 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFJ3J4GiMEm_1doWxA7FgF0BLYtbuDw1ks5t-kzLgaJpZM4TL5Sp> .

[navono]

@calvinchankf You're welcome. If you find a better way, please let me know.

[markx1916]

I also meet this problem , I just copy the content to a new .xls or xlsx file by hand .

need help!

[navono]

If you have a lot file .xls files, may be can write a script to transfer all files extension to .xlsx.
I haven't test the solution, just a thought.

[mahesh-sanikommu]

any fix for this issue?

[vasiliyb]

@mahesh-sanikommu convert .xls to .xlsx ... i think

[xuri]

This lib doesn't support the older binary xls format currently. I will close this issue. The issue #44 still discussing the xls format.

[iSenninha]

@mahesh-sanikommu convert .xls to .xlsx ... i think

so sad....

[BlackCodes]

so sad...

[winter-dev]

so sad...

[MKquant]

so sad...

[cahyaramadhan]

so sad...

[XieWeiXie]

so sad...

[shikingram]

No Solution?

[xuri]

Hi @shikingram, this issue has been closed and merged into issue #44. Currently, there is no plan to add support to the legacy xls format spreadsheet.

[blusewang]

so sad...

[yasir16]

I just had this problem, 4 days ago it going well, but yesterday this issue appeared