Skip to content

Fixed bug in Database.php class where count() was returning bool (0/1) instead...

Johannes requested to merge fix-count-return-type into master

What does this MR do?

Fixed bug in Database.php class where count(...) is returning bool (0/1) instead of the actual amount in int.

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

Sould work. Checked all uses.

Links to related issues

Should have done this earlier in a separate MR. Please merge this before merging !788 (merged)

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 Chris Oelmueller

Merge request reports