ID:               19724
 Updated by:       [EMAIL PROTECTED]
 Reported By:      [EMAIL PROTECTED]
 Status:           Assigned
 Bug Type:         dBase related
 Operating System: All
 PHP Version:      4.2.3
 Assigned To:      vlad
 New Comment:

didn't have time to look at PHP bugs for a while...

The status:
- No soul seems to know what 'padded field headers' 
- The only case when I can imagine it breaking is in dbase5 with
[255...1024] fields ([EMAIL PROTECTED] did not prove or disprove
that)
- I do not want to remove that padding algorythm, because obviously it
worked for somebody.

The easy solution would be to just bump up the maximum number of
fields. Memory consumption is not *that* much larger, it won't break
anything, and, since dbase is dead, I am sure no-one will implement a
version that has more than 1024 fields :)

I will do that once a check out a fresh tree. I'll close the bug when I
do that.


Previous Comments:
------------------------------------------------------------------------

[2002-10-27 19:10:11] [EMAIL PROTECTED]

Setting to 'Open' to prevent automatic closing due the no-feedback..

Vlad, what is the status on this?



------------------------------------------------------------------------

[2002-10-08 12:28:50] [EMAIL PROTECTED]

Actually, it seems to return only up to 254 fields, which is a bug on
its own. I believe, there was a limit of 255 fields for dbase4 format.
I do not believe this is the case with dbase5. 

This patch broke it:
http://cvs.php.net/diff.php/php4/ext/dbase/dbf_head.c?login=2&r1=1.9&r2=1.10&ty=u

The simpliest way to fix that would be just to increase the constant
from 254 to 1024. The better way would be figure out what the author of
the patch meant by 'padded field headers'.

I'll look into that.


[EMAIL PROTECTED], do you use dbase5? or something else?

------------------------------------------------------------------------

[2002-10-02 15:52:27] [EMAIL PROTECTED]

When using dbase_get_record or dbase_get_record_with_names to read from
a DBF file, it only returns a maximum of 255 fields now. It did not do
this in previous versions.

------------------------------------------------------------------------


-- 
Edit this bug report at http://bugs.php.net/?id=19724&edit=1

Reply via email to