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

Report in Module Bank #24756

Closed
ptagesen opened this issue May 12, 2023 · 1 comment
Closed

Report in Module Bank #24756

ptagesen opened this issue May 12, 2023 · 1 comment
Labels
Bug This is a bug (something does not work as expected) Event: DevCamp 2023 Bordeaux

Comments

@ptagesen
Copy link

Bug

In version 17.0.1 (using php 8.0) the sums in monthly report in the module "bank" are incorrect. I found a problem in the file compta/bank/annuel.php. In line 199 and 200 the annual sums for "sorties" and "entrees" are set to zero every month in the loop over years. So as a final result I only see the sum of December.
Instead of

for ($annee = $year_start; $annee <= $year_end; $annee++) {
$totsorties[$annee] = 0;
$totentrees[$annee] = 0;

it should be like this:

$totsorties[$annee] = 0;|
$totentrees[$annee] = 0;|
for ($annee = $year_start; $annee <= $year_end; $annee++) {

I'm not a developer, so maybe someone can fix this for future releases.

thx, Peter

Environment Version

17.0.1

Environment OS

synology disk station - linux

Environment Web server

nginx

Environment PHP

8.0

Environment Database

MariaDB 10

Environment URL(s)

No response

Expected and actual behavior

explained in Bug-Description.

Steps to reproduce the behavior

In the menu Bank open the monthly report. The sum for the year is incorrect.

Attached files

No response

@ptagesen ptagesen added the Bug This is a bug (something does not work as expected) label May 12, 2023
@aspangaro
Copy link
Member

#24996

eldy added a commit that referenced this issue Jun 10, 2023
FIX #24756 Total sum on annual report bank
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug (something does not work as expected) Event: DevCamp 2023 Bordeaux
Projects
None yet
Development

No branches or pull requests

3 participants