search

a-sit / PDF-Over

https://technology.a-sit.at/en/pdf-over/
European Union Public License 1.2
17 stars 4 forks source link

Automatic placeholder search broken using linux command line #78

Closed rarzberger4 closed 1 year ago

rarzberger4 commented 1 year ago

Steps to Reproduce

  1. pdf-over 4.4.4 installed on ubuntu 22.04.2 LTS
  2. open pdf-over
  3. change the settings inside the gui to automatic positioning of the signature and activate placeholder-qr-code search
  4. reopen pdf-over using: "sh pdf-over_linux.sh -i signatur_Test.pdf -a" using -a or not is not relevant for the crash to occur
  5. a menu pops up and select anything --> crash

Expected Behavior

Expected to put the signature on the placeholder qr-code automatically so only thing to do is authorize the signature

Current Behavior

The program reports a found qr-placeholder

2023-03-01 11_44_16-192 168 2 196 (rarzberger)

If you click any option the program crashes 2023-03-01 11_44_35-PDF-Over @ubuntutest

Error in the command line after crashing: 2023-03-01 11_45_07-192 168 2 196 (rarzberger)

But if you open pdf-over with default config and set the placeholder search like in step 3. and then use the file browser to select a document the placeholder works as expected.

iaik-jheher commented 1 year ago

possibly dup #77?

rarzberger4 commented 1 year ago

probably the same problem, but it would be interesting if the error occured also for #77 if onyl the placeholder qr-code is used without a second one

rarzberger4 commented 1 year ago

So it seems that not the automatic placeholder is the problem. If i run pdf-over without a invalid argument: "sh pdf-over_linux.sh -i" then this happens: image

If I set a argument for a valid path but not a pdf-file: "sh pdf-over_linux.sh -i /home/rarzberger/PDF-Over/toSign/" this happens: image

The same Error is written in the cli: image

The problem occurs when a GUI menu appears automatically before I do anything in the program itself. The menu has no valid actions. It doesn't matter what menu-item I click, the program crashes.

rarzberger4 commented 1 year ago

Error does not appear on windows.

iaik-jheher commented 1 year ago

Is "check for updates" enabled (in the "about" tab of the configuration)?

rarzberger4 commented 1 year ago

Yes is activated

iaik-jheher commented 1 year ago

Managed to reproduce.

iaik-jheher commented 1 year ago

PR #80 referenced above should address this. If you could quickly check that it also works for you, that'd be appreciated.

Lightweight CI builds for the PR are available in the PR comments.

rarzberger4 commented 1 year ago

Tested the builds an the problem is gone - thank you