Skip to content

Fix initialization of AsyncMail

Nick Sellen requested to merge pass-mem-to-async-mail into master

What does this MR do?

In my unified DI changes (!544 (merged)) I failed to change some AsyncMail instantiations to have an instance of Mem passed in.

This caused mail sending to fail (for some cases at least). I found 3 occurrences where this was missed.

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

Very, intellij showed me error, and is happy afterwards :)

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible... meh, this is quickfix for now, we have poor test coverage of these extra kind of services
  • 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 Nick Sellen

Merge request reports