Skip to content

Improve/correct user badge count for foodsaver

Michael Aydinbas requested to merge pmayd-small-bugfix-foodsaver-post-stats into master

What does this MR do?

Improve/correct user badge count: to fetch only valid posts, milestone = 0 was added to SQL query that fetches the user posts from table fs_betrieb_notiz

How confident are you it won't break things if deployed?

Added one criterion (milestone) to the SQL query for the stat_post var from fs_betrieb_notiz, so there should be no sideeffects or break ups. Changes were proposed by @jofranz.

Proper working can be seen/tested via calling ./scripts/docker-compose run app php -f run.php Stats foodsaver. Updates the stats correctly by ignoring all posts with milestone != 0.

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
Edited by Michael Aydinbas

Merge request reports