Project

General

Profile

Actions
Copy link

Bug #251

closed

Widget [storePasswordCheck] not found

Added by rhabacker about 4 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Category:
Gwenhywfar
Start date:
12/30/2021
Due date:
Betriebssystem:
Windows
AqBanking-Version:
aqbanking 6.4.0 gwenhywfar 5.7.3.0
Anwendung:
KMyMoney
Version der Anwendung:

Description

Trying to update a paypal account from kmymoney returns in the ui

19:56:26 AqBanking v6.4.0.0
19:56:26 Sende Aufträge an die Bank(en)
19:56:26 Aufträge nach Konto sortieren
19:56:26 Konten-Aufträge nach Providern sortieren
19:56:26 Aufträge an Provider verteilen
19:56:26 Aufträge an Provider "AQPAYPAL" übergeben
19:56:26 Zugriff auf Benutzer "14"
19:56:30 Vorgang abgeschlossen, Sie können das Fenster nun schließen.

In the log there is shown:

3:2021/12/29 19-53-02:(null)(8515):cppgui.cpp: 141: CppGuiLinker::OpenDialog
3:2021/12/29 19-53-02:(null)(8515):provider_sendcmd.c: 298: Handling user "xxx"
3:2021/12/29 19-53-06:gwen(8515):dialog.c: 695: Widget [storePasswordCheck] not found
3:2021/12/29 19-53-06:gwen(8515):padd.c: 210: First byte is not a 0x60

Files

0001-Fix-warning-dialog.c-695-Widget-storePasswordCheck-n.patch (1.99 KB) 0001-Fix-warning-dialog.c-695-Widget-storePasswordCheck-n.patch rhabacker, 02/16/2022 09:00 AM
Actions
Copy link
 #1

Updated by rhabacker about 4 years ago

  • File 0001-Fix-warning-dialog.c-695-Widget-storePasswordCheck-n.patch added
Actions
Copy link
 #2

Updated by mlenk about 4 years ago

I just briefly looked into th patch you provided. When looking at this line:

if (xdlg->flags & 2)

Using literal flag values is ugly, because it hides the intention of checked condition of the if clause.

Did you mean to use the GWEN_GUI_INPUT_FLAGS_SHOW flag as defined in src/gui/gui.h?

Regards,
Micha

Actions
Copy link
 #3

Updated by rhabacker about 4 years ago

If this value is used at the https://github.com/aqbanking/gwenhywfar/blob/0f041ffe9520bba4a2fc0ace2b300041a8f0c91e/src/gui/dlg_input.c#L64 location, then yes. I simply used the the same value.

Actions
Copy link
 #4

Updated by rhabacker about 4 years ago

  • File 0001-Fix-warning-dialog.c-695-Widget-storePasswordCheck-n.patch added

Updated patch to match conditions used in GWEN_DlgInput_new()

Actions
Copy link
 #5

Updated by rhabacker about 4 years ago

  • File deleted (0001-Fix-warning-dialog.c-695-Widget-storePasswordCheck-n.patch)
Actions
Copy link
 #6

Updated by rhabacker almost 4 years ago

Patch updated

To avoid duplicated code to set `n` a new input dialog attribute `storePasswordCheckEnabled` has been added and used in GWEN_DlgInput_Fini().

Actions
Copy link
 #7

Updated by martin almost 4 years ago

  • Status changed from New to Resolved

Thanks!!, applied to git.

Actions
Copy link
 #8

Updated by martin almost 4 years ago

  • Status changed from Resolved to Closed
Actions
Copy link
 #9

Updated by rhabacker over 3 years ago

I saw this fix in 'master' branch, not in 'maint' - is this intended ?

Actions
Copy link

Also available in: Atom PDF