This partially reverts be25bbc4.
Signed-off-by: NRémy Coutable <remy@rymai.me>

The /unsubscribe slash command means that we check if the current user is
subscribed to the issuable without having an explicit subscription. That means
that we use the UserParser to find references to them in the notes.

The UserParser (and all parsers inheriting from BaseParser) use RequestStore to
cache ActiveRecord objects, so that we don't need to load the User object each
time, if we're parsing references a bunch of times in the same request.

However, it was always returning _all_ of the previously cached items, not just
the ones matching the IDs passed. This would mean that we did two runs through
with UserParser if you were mentioned in a comment, and then mentioned someone
else in your comment while using /unsubscribe:

1. Because /unsubscribe was used, we see if you were mentioned in any comments.
2. Because you mentioned someone, we find them - but we would also get back your
   user, even if you didn't mention yourself. This would have the effect of
   creating a mention or directly addressed todo for yourself incorrectly.

The fix is simple: only return values from the cache matching the IDs passed.

'New issue'/'New merge request' dropdowns should show only projects with issues/merge requests feature enabled

This is needed because these variables depend (directly or indirectly)
on the 'app_root' variable which can be changed in the default file.
If app_root has a non-standard value, the value of e.g. gitaly_dir
generated in the init script becomes invalid.

There was a race condition issue when the application was generating an
email and was using a language that was previously being used in other
request.

This reverts commit b0498c17