Bug: "n products are overdue"-status header on /stockoverview also includes stock entries due today or tomorrow about grocy HOT 6 CLOSED

Please create a private demo instance if you want to provide a reproducible example, the public instances are used by hundreds of people daily and get therefore also reseted multiple times a day - it's impossible to share scenarios there persistently.

from grocy.

now behaves differently than before

Would be cool if you could specify what "before" refers to exactly.

the grey "# products are overdue" header now behaves differently than before. Seemingly it counts expired products (which is logical) plus products due today

Of course this makes no sense and any intended change is, as always, documented on the changelog. Is reproducible, will be fixed for the next release.

from grocy.

Hello,

okay, I created it in a Private Demo: https://test-42cvxut103tkp0ra9zwp9m.demo-prerelease.grocy.info/stockoverview
It behaves the same as well as my 4.0.3 production version on my server. In version 4.0.2 and lower it didn't behave like this, more logical and it was consistent with color coding of products in Product Overview page.

from grocy.

In version 4.0.2 and lower it didn't behave like this

Can't be true. It's caused by using PHP 8.1 or 8.2, which is a requirement and only supported since Grocy v4.0.0 - means this problem first occurred in Grocy v4.0.0, not v4.0.2.

from grocy.

Today I switched from a server with v4.0.2 to a new one with v4.0.3, migrated database and config and when I reregistered the Android app to a new server (I changed a subdomain name) I immediately noticed change in "products are overdue" number.

from grocy.

Then you have simply not noticed it before. Recreate the same example on v4.0.2 and notice that the problem also already exists there. It's about how data types are handled when retrieving data from the database, which has changed since PHP 8.1. There was no change between v4.0.2 and v.4.0.3 in that area.

But whatever, let's simply fix this instead of getting lost in philosophizing where this was introduced exactly.

from grocy.