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

TypeScript errors - make input optional #320

Open
mchepel opened this issue Aug 11, 2023 · 6 comments
Open

TypeScript errors - make input optional #320

mchepel opened this issue Aug 11, 2023 · 6 comments

Comments

@mchepel
Copy link

mchepel commented Aug 11, 2023

capture
Should getSourceDocumentParser require "input" parameter?

getSourceDocumentParser( input: FilePath | ReadStream, options?: PDFReaderOptions ): PDFReader;

It seems it may not be required

PDFParser* PDFDocumentHandler::GetSourceDocumentParser() { return mParser; }

@mchepel mchepel closed this as not planned Won't fix, can't repro, duplicate, stale Aug 18, 2023
@julianhille
Copy link
Owner

why did you close it? from first perspective it looks like a but to be fixed.

@mchepel
Copy link
Author

mchepel commented Aug 29, 2023

I closed it because there was no feedback for a week, I figured it is very low priority. After not getting this library working for what I needed it for, I just moved on. I will re-open the issue, maybe others will find it useful.

@mchepel mchepel reopened this Aug 29, 2023
@julianhille
Copy link
Owner

May I ask what you needed what the lib could not do?

@julianhille
Copy link
Owner

julianhille commented Aug 31, 2023

The args are thrown against

Local<Value> newInstance = copyingContext->holder->GetNewPDFReader(args);

which instanciates:

v8::Local<v8::Value> ConstructorsHolder::GetNewPDFReader(const ARGS_TYPE& args)
need to dig deeper

@mchepel
Copy link
Author

mchepel commented Sep 1, 2023

May I ask what you needed what the lib could not do?

I needed to fill out a pdf file (find / replace text) but I just could not get it to work. Tried everything I could find online. No errors but no result either. I can share the code if you are interested.

@julianhille
Copy link
Owner

Please share the code.

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