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

Checking Balance with money money is broken (again) #3

Open
iSchorsch opened this issue Nov 5, 2020 · 10 comments
Open

Checking Balance with money money is broken (again) #3

iSchorsch opened this issue Nov 5, 2020 · 10 comments

Comments

@iSchorsch
Copy link

That is the error message i get:

"bergfuerst.lua:103: attempt to perform arithmetic on local 'totalInterestAmount' (a nil value)"

It would be awesome, if this could be fixed!?

THANK YOU FOR YOUR EFFORT!

@fkerber
Copy link

fkerber commented Nov 10, 2020

Changing line 64 to
local totalInterestAmount = tonumber((html:xpath("//*[@id='top']/div[2]/div/div[1]/div[2]/div/div[3]/p[2]/strong"):get(1):text():sub(1, -5):gsub(",", ".")))
(all in one line)
did fix the issue on my side.

@iSchorsch
Copy link
Author

replaced line 64 with your line of code. Unfortunately same error message:

bergfuerst.lua:103: attempt to perform arithmetic on local 'totalInterestAmount' (a nil value)

@nicolindemann
Copy link
Owner

Please check here (https://github.com/nicolindemann/bergfuerst-moneymoney/blob/develop/src/bergfuerst.lua) my fix. On my side everything seems to work now again. It would be nice if a second person could double check this.

@iSchorsch
Copy link
Author

Thanks for the fix. Unfortunately I still get an error message and no data is pulled. Also tried to re-initiate the plug in. Still error:

Web Banking account: Bergfürst (Main)
Lade Wertpapierbestand für Bergfürst ...
Sending: GET https://de.bergfuerst.com/meine-investments

      bergfuerst.lua:103: attempt to perform arithmetic on local 'totalInterestAmount' (a nil value)

@nicolindemann
Copy link
Owner

Does MoneyMoney mention the correct version number of the plugin?

1> Web Banking Engine: Using user-supplied extension Bergfuerst.lua version 1.20.

@iSchorsch
Copy link
Author

Thank you for getting back to me. I just doublechecked and reinstalled your plug in 1.2. I does show version 1.2. now.
It would not let me updated the berffuerst balance in MoneyMoney so I stared from scratch with my Bergfürst account in money money, aka acted as if I would add that account for the first time.
It went through the authorisation flawlessly but didn't pull the write amounts of my investments (zero) just one worked:
Please see screenshot. If this would not be sensitive data, I would just give you my login credentials to troubleshoot. What surprises me is the fact, that it works on your end, but not on mine!?
Bildschirmfoto 2021-02-28 um 19 02 38
Bildschirmfoto 2021-02-28 um 19 02 19

@nicolindemann
Copy link
Owner

I will try to come up with some kind of debug output in the console for troubleshooting. I only ask for a little more patience.

@anju1337
Copy link

Hi Nico,

will there be a signed version again?

Thanks 4 your effort!

@iSchorsch
Copy link
Author

Hi Nico, is this PlugIn-Project discontinued? I would still love to check my Bergfürst Account with Money Money. When it worked, it was beautiful

@nicolindemann
Copy link
Owner

Not really discontinued, but very neglected. I'll try to look over it again in the next few days.

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

4 participants