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

solve structure problem #42

Open
HAPpleee opened this issue Feb 28, 2023 · 1 comment
Open

solve structure problem #42

HAPpleee opened this issue Feb 28, 2023 · 1 comment

Comments

@HAPpleee
Copy link

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch dbf@0.1.4 for the project I'm working on.

Here is the diff that solved my problem:

diff --git a/node_modules/dbf/src/structure.js b/node_modules/dbf/src/structure.js
index 4b02783..f2b4c0b 100644
--- a/node_modules/dbf/src/structure.js
+++ b/node_modules/dbf/src/structure.js
@@ -55,7 +55,7 @@ module.exports = function structure(data, meta) {
         if (f.type == 'N') view.setInt8(32 + i * 32 + 17, 3);
     });
 
-    offset = fieldDescLength + 32;
+    var offset = fieldDescLength + 32;
 
     data.forEach(function(row, num) {
         // delete flag: this is not deleted

This issue body was partially generated by patch-package.

@sheindel
Copy link
Contributor

Hello! Thanks for noting this! There was actually fixed in a version released last June, so I believe this issue should be resolved with dbf@0.2.0. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants