Field header truncated

[alexladerman]

Hi,

Thanks for your tool. Perhaps I am wrong, but I think the field headers should allow up to 10 characters. I try to export with such but I get headers truncated at 8

[tmcw]

Hm, I'm not sure - 10 is, yes, what it's supposed to be, but it's odd that this wouldn't cause other behavior than just shorter fields.

[alexladerman]

The file i am trying to generate.... which is used in an old clipper based
accounting program uses 10 character headers.
On Jul 10, 2014 4:36 PM, "Tom MacWright" notifications@github.com wrote:

Hm, I'm not sure - 10 is, yes, what it's supposed to be, but it's odd that this
wouldn't cause other behavior
https://github.com/mapbox/dbf/blob/master/src/structure.js#L48-L50 than
just shorter fields.

—
Reply to this email directly or view it on GitHub
#11 (comment).

[bakednevada]

@tmcw the reason it doesn't change behavior is twofold: ArrayBuffers are zero-filled by default and the other operations use the correct offsets, so it's leaving null characters in the trailing bytes of the name. Increasing the number of characters should not have any adverse effects.

[coding-spider]

Same issue. Any update?

[northtron]

Hello,
You can override the 8 character limit by changing the f.name.splice(0.8) to (0,10):

field_meta.forEach(function(f, i) { // field name f.name.split('').slice(0, 8).forEach(function(c, x) { view.setInt8(32 + i * 32 + x, c.charCodeAt(0)); }); // field type view.setInt8(32 + i * 32 + 11, f.type.charCodeAt(0)); // field length view.setInt8(32 + i * 32 + 16, f.size); if (f.type == 'N') view.setInt8(32 + i * 32 + 17, 3); });

to

field_meta.forEach(function(f, i) { // field name f.name.split('').slice(0, 10).forEach(function(c, x) { view.setInt8(32 + i * 32 + x, c.charCodeAt(0)); }); // field type view.setInt8(32 + i * 32 + 11, f.type.charCodeAt(0)); // field length view.setInt8(32 + i * 32 + 16, f.size); if (f.type == 'N') view.setInt8(32 + i * 32 + 17, 3); });

[sheindel]

Happy to report this is finally merged in (#33)! Hopefully we can publish an updated npm package soon