chenster / php-excel-reader

Automatically exported from code.google.com/p/php-excel-reader
0 stars 0 forks source link

Undefined errors in version 2.2 #16

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Import any xls file
2.
3.

What is the expected output? What do you see instead?
I expect to only see my data instead I see several undefined errors.

Please provide any additional information below.
The undefined errors have been fixed in the attached version. I hope you
will include them in the next release.

Original issue reported on code.google.com by rdalmul...@gmail.com on 31 Mar 2009 at 6:38

Attachments:

GoogleCodeExporter commented 9 years ago
Do you have an example XLS that shows the problem?
I see your changes, but I'm not sure what problem they really fix.

Original comment by matthew....@gmail.com on 31 Mar 2009 at 2:30

GoogleCodeExporter commented 9 years ago
I should have added the error messages itself:

Notice: Undefined index: formatColor in D:\Program
Files\wamp\www\csvi\administrator\components\com_csvimproved\helpers\excel\excel
_reader2.php
on line 1631

Notice: Undefined variable: fontIndex in D:\Program
Files\wamp\www\csvi\administrator\components\com_csvimproved\helpers\excel\excel
_reader2.php
on line 1641

Fixes:
1. Original return an array with only "string" and missing "formatColor"
2. fontIndex is not defined when being returned in the array

If you still need the file, please let me know and I can e-mail it to you, I 
just
cannot make the file public.

Original comment by rdalmul...@gmail.com on 1 Apr 2009 at 3:19

GoogleCodeExporter commented 9 years ago
This will be fixed in an upcoming patch release

Original comment by matthew....@gmail.com on 2 Apr 2009 at 4:22

GoogleCodeExporter commented 9 years ago
Is there an ETA of when this patch will be released?

Thanks

Original comment by sargea...@gmail.com on 11 Apr 2009 at 10:30

GoogleCodeExporter commented 9 years ago
Fixed in version 2.21

Original comment by matthew....@gmail.com on 14 Apr 2009 at 4:05