partizand / finquote

Scripts for russian stock for perl finance-quotes
13 stars 3 forks source link

После обновления GnuCash до 5.0 модуль перестал работать. #11

Closed Mark-Seem closed 1 year ago

Mark-Seem commented 1 year ago

Обновил GnuCash до 5.0 и заодно Finance:Quote после этого модуль перестал работать. До этого все работало. Проверка выдает следующую ошибку image

Подскажите, пожалуйста, с чем это может быть связано?

partizand commented 1 year ago

В 5 версии поменяли работу с котировками. Не разбирался еще. Сижу на debian, а там древние версии GnuCash. Дойдут руки посмотрю.

The Online Quotes facility has been completely rewritten and the old gnc-fq-check, gnc-fq-dump, and gnc-fq-helper programs have been replaced with finance-quote-wrapper. The functions performed by those programs may now be accomplished by passing commands to gnucash-cli -Q, see gnucash-cli --help for specifics. The perl module requirements have changed with the rewrite: The new version doesn't need Date::Manip but needs JSON::Parse instead. gnc-fq-update has been, er, updated to reflect that.

Mark-Seem commented 1 year ago

Спасибо, на 4.13 все продолжает работать. На 5 версию обновляться, значит, пока рано.

ddarwish commented 1 year ago

Just vote up to fix this issue

fertkir commented 1 year ago

Добавлю контекста:

❯ gnucash-cli --quotes get ~/my.gnucash
Price retrieval failed: Failed to initialize Finance::Quote: missing_modules JSON::Parse

Починил: cpan JSON::Parse

fertkir commented 1 year ago

Появилась другая ошибка:

❯ gnucash-cli --quotes dump moex_stock FXRU
Price retrieval failed: Finance::Quote check returned error Subroutine methods redefined at /usr/local/share/perl5/5.36/Finance/Quote/Moex.pm line 56.
Subroutine labels redefined at /usr/local/share/perl5/5.36/Finance/Quote/Moex.pm line 82.

Выглядит так, что Finance::Quote какую-то проверку своих модулей добавили.

fertkir commented 1 year ago

Вот так заработало: https://github.com/partizand/finquote/pull/12/files Разумеется, в таком виде вливать не предлагаю.

partizand commented 1 year ago

Вот так заработало: https://github.com/partizand/finquote/pull/12/files Разумеется, в таком виде вливать не предлагаю.

Спасибо, посмотрю, проверю и добавлю Хотя думаю в реквесте достаточно удалить закомментированное и его можно было бы принять, после проверки работы

fertkir commented 1 year ago

@partizand, обновил pr

partizand commented 1 year ago

Исправлено, просьба проверить В cpan чуть позже выложу

partizand commented 1 year ago

Исправлено в релизе https://github.com/partizand/finquote/releases/tag/v0.6

PingWIn87 commented 1 year ago

а когда ждать сборку в cpan?

partizand commented 1 year ago

а когда ждать сборку в cpan?

cpan обновил