Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Value of a cell is changed to a floating point number if it contains only numerical values #1275

Closed
qburst-salax opened this issue Jul 8, 2022 · 5 comments
Labels
duplicate This issue or pull request already exists

Comments

@qburst-salax
Copy link

qburst-salax commented Jul 8, 2022

Description

Steps to reproduce the issue:

  1. Create a excel file with a value "8_08_78_08_76" inside a cell.
  2. Read the excel file with this package.
  3. Print the cell value read by the package.

Describe the results you received:
"808780876"

Describe the results you expected:
"8_08_78_08_76"

Output of go version:
go 1.17

Excelize version or commit ID:
v2.5.0

Environment details (OS, Microsoft Excel™ version, physical, etc.):
Ubuntu 20.04.4 LTS

@xuri
Copy link
Member

xuri commented Jul 8, 2022

Thanks for your feedback. This issue has been fixed in v2.6.0, please update to the lasted version or using the master branch code.

@xuri xuri closed this as completed Jul 8, 2022
@qburst-salax
Copy link
Author

qburst-salax commented Jul 8, 2022

Hi,

I tried out the new version v2.6.0 but sadly now when my value in the cell is "82_01_022_220131" what I recieve is an empty string. After hours of debugging I found out that a function named "bstrUnmarshal" takes in the value and returns an empty string for some reason. Can you please look in to it.

Thanks for your feedback. This issue has been fixed in v2.6.0, please update to the lasted version or using the master branch code.

@xuri

@xuri
Copy link
Member

xuri commented Jul 8, 2022

I have tested and it works well, could you provide the file attachment without confidential info?

@xuri xuri added the needs more info This issue can't reproduce, need more info label Jul 8, 2022
@xuri
Copy link
Member

xuri commented Jul 8, 2022

Thanks for your feedback, this issue was duplicate with #1219, I have fixed it, please upgrade to the master branch code, and this patch will be released in the next version.

@xuri xuri added duplicate This issue or pull request already exists and removed needs more info This issue can't reproduce, need more info labels Jul 8, 2022
@qburst-salax
Copy link
Author

Ok, will do that. Thank You!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants