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

AR Invoice Entry Error on Udate with no due-by date #3223

Closed
sbts opened this issue Oct 6, 2017 · 0 comments
Closed

AR Invoice Entry Error on Udate with no due-by date #3223

sbts opened this issue Oct 6, 2017 · 0 comments
Milestone

Comments

@sbts
Copy link
Member

sbts commented Oct 6, 2017

Adding a new invoice, with the "first" customer selected in the dropdown and no Due Date
Hitting Update throws the below error

Expected behavior is to populate the due date

'The 'month' parameter ("13") to DateTime::new did not pass the 'an integer between 1 and 12' callback at /usr/lib/x86_64-linux-gnu/perl5/5.20/DateTime.pm line 197. DateTime::new(undef, "month", 13, "day", 10, "year", 2017) called at /srv/ledgersmb/tools/../lib/LedgerSMB/PGDate.pm line 199 LedgerSMB::PGDate::new("LedgerSMB::PGDate", "month", 13, "day", 10, "year", 2017) called at /srv/ledgersmb/tools/../lib/LedgerSMB/PGDate.pm line 272 LedgerSMB::PGDate::from_input("LedgerSMB::PGDate", "13/10/2017") called at /srv/ledgersmb/tools/../bin/is.pl line 1125 lsmb_legacy::update called at /srv/ledgersmb/tools/../bin/old-handler.pl line 167 lsmb_legacy::try {...} () called at /usr/share/perl5/Try/Tiny.pm line 81 eval {...} called at /usr/share/perl5/Try/Tiny.pm line 72 Try::Tiny::try(CODE(0xe983f40), Try::Tiny::Catch=REF(0xecb2490)) called at /srv/ledgersmb/tools/../bin/old-handler.pl line 183 require bin/old-handler.pl called at /srv/ledgersmb/tools/../lib/LedgerSMB/PSGI.pm line 110 LedgerSMB::PSGI::try {...} () called at /usr/share/perl5/Try/Tiny.pm line 81 eval {...} called at /usr/share/perl5/Try/Tiny.pm line 72 Try::Tiny::try(CODE(0x9b10fc0)) called at /srv/ledgersmb/tools/../lib/LedgerSMB/PSGI.pm line 116 LedgerSMB::PSGI::_run_old() called at /srv/ledgersmb/tools/../lib/LedgerSMB/PSGI.pm line 74 LedgerSMB::PSGI::ANON() called at /usr/share/perl5/CGI/Emulate/PSGI.pm line 29 CGI::Emulate::PSGI::ANON(HASH(0xeb49820)) called at /usr/share/perl5/Plack/App/URLMap.pm line 71 Plack::App::URLMap::call(Plack::App::URLMap=HASH(0x9ca84c0), HASH(0xeb49820)) called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::ANON(HASH(0xeb49820)) called at /srv/ledgersmb/tools/starman.psgi line 72 LedgerSMB::FCGI::ANON(HASH(0xeb49820)) called at /usr/share/perl5/Plack/Middleware/Conditional.pm line 16 Plack::Middleware::Conditional::call(Plack::Middleware::Conditional=HASH(0x9cb7028), HASH(0xeb49820)) called at /usr/share/perl5/Plack/Component.pm line 50 Plack::Component::ANON(HASH(0xeb49820)) called at /usr/share/perl5/Plack/Util.pm line 142 eval {...} called at /usr/share/perl5/Plack/Util.pm line 142 Plack::Util::run_app(CODE(0x9cb6f80), HASH(0xeb49820)) called at /usr/share/perl5/Starman/Server.pm line 281 Starman::Server::process_request(Starman::Server=HASH(0x20c4ed0), Net::Server::Proto::TCP=GLOB(0xe970260)) called at /usr/share/perl5/Net/Server.pm line 74 Net::Server::run_client_connection(Starman::Server=HASH(0x20c4ed0)) called at /usr/share/perl5/Net/Server/PreFork.pm line 229 eval {...} called at /usr/share/perl5/Net/Server/PreFork.pm line 229 Net::Server::PreFork::run_child(Starman::Server=HASH(0x20c4ed0)) called at /usr/share/perl5/Net/Server/PreFork.pm line 187 Net::Server::PreFork::run_n_children(Starman::Server=HASH(0x20c4ed0), 5) called at /usr/share/perl5/Net/Server/PreFork.pm line 111 Net::Server::PreFork::loop(Starman::Server=HASH(0x20c4ed0)) called at /usr/share/perl5/Net/Server.pm line 60 Net::Server::run(Starman::Server=HASH(0x20c4ed0), "port", ARRAY(0xa0e38d0), "host", "*", "proto", "tcp", "serialize", "none", ...) called at /usr/share/perl5/Starman/Server.pm line 89 Starman::Server::run(Starman::Server=HASH(0x20c4ed0), CODE(0x9cb6f80), HASH(0x9c9c1d8)) called at /usr/share/perl5/Plack/Handler/Starman.pm line 18 Plack::Handler::Starman::run(Plack::Handler::Starman=HASH(0x20e1bb0), CODE(0x9cb6f80)) called at /usr/share/perl5/Plack/Loader.pm line 84 Plack::Loader::run(Plack::Loader=HASH(0x200f430), Plack::Handler::Starman=HASH(0x20e1bb0)) called at /usr/share/perl5/Plack/Runner.pm line 277 Plack::Runner::run(Plack::Runner=HASH(0x1d6a830)) called at /usr/bin/starman line 38 '

@sbts sbts added this to the 1.5 milestone Oct 6, 2017
@sbts sbts added the bug:other label Oct 6, 2017
ehuelsmann added a commit to ehuelsmann/LedgerSMB that referenced this issue Oct 29, 2017
Note that it's a miracle how much functionality *did* work,
 given that the day was passed as the month number, which
 causes DateTime to die for any day > 12.
ehuelsmann added a commit that referenced this issue Oct 29, 2017
* Fix #3223: incorrect parsing of date format 'dd/mm/yyyy'
ehuelsmann added a commit that referenced this issue Oct 29, 2017
* Fix #3223: incorrect parsing of date format 'dd/mm/yyyy'
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

1 participant