Stats sum is wrong, if "This Year" is selected

[heikok88]

Please complete the following information

• Installation Type: Docker
• Have you tried using the latest docker image / code base yes

Describe the bug
I have only one Vehicle in Hammond with only 5 fillups in 2022.
The stats are wrong if I select "This Year".
"Past 3 Months" and "All Time" are correct. See the screenshots

Expected behavior
If "This Year" is selected, the first fillup (Jan 25, 2022, 78,93 €) is missing in the sum and should be included.

Screenshots

[akhilrex]

Will look into it

[heikok88]

Thx! If you need my db, logs or something like that let me know.

[heikok88]

Might it possible, that the first fillup to a vehicle is not included in the sum?

If you need help let me know, I'm a (embedded) software engineer ;-)

[heikok88]

I've looked into the code and found the error:
In your statsWidget.vue, you have to pass 0 as month for january. The days start by 1, but the month start by 0.

[heikok88]

I've found another error. If in the stats "This week" is selected, you need to call toDate.setHours(0, 0, 0, 0) to set the time to midnight.
Because otherwise the the fillup of mondays before the time of the requests are not included.

[heikok88]

Should I create a pullrequest?

[akhilrex]

Please do

…

On Tue, 12 Jul 2022 at 00:24, heikok88 ***@***.***> wrote: Should I create a pullrequest? — Reply to this email directly, view it on GitHub <#87 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEMVPSIULZ5UE76LGGZQCDVTRUVRANCNFSM5UGFB5WQ> . You are receiving this because you commented.Message ID: ***@***.***>

-- Thanks and Regards, Akhil Gupta

[heikok88]

Just created the PR
Greets