Skip to content

Switched use of Session::id to Session::may

Dominik Hayon requested to merge switch-id-to-may into master

What does this MR do?

After talking to @jofranz about getting back into supporting Foodsharing, I got this minor task of switching the previously used check with id() to may(), which is what I've done for this MR.

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

Rather confident.

Links to related issues

None (that I could find).

How to test

Since I'm not that deep into the codebase yet, I'm not quite sure :) Judging from just the pieces I've seen, e.g. uploading a profile photo is affected by this change. However, there are probably no noticeable differences.

Screenshots (if applicable)

Nothing to see here.

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))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Chris Oelmueller

Merge request reports