Always use declared_params(include_missing: false)
so that we don't give nils for optional arguments

If internal auth is disabled and user is not an LDAP user, present
the user with an alert to create a personal access token if he does
not have one already.

Closes #34052

This was broken by
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/12322.
Signed-off-by: NRémy Coutable <remy@rymai.me>

In certain combination of tests, @env_already_stubbed could be set
to `true` even though it never ran for that specific test.

Closes #33758

Being the good boyscouts, but mainly because of [the comment in the
review](https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/12378#note_33302115)
the words used for classes and variables are changed to not use builds
anymore.

In CE only the admin has access to all private groups & projects. In EE also an
auditor can have full private access.

To overcome merge conflicts, or accidental incorrect access rights, abstract
this out in `User#full_private_access?`.

`User#admin?` now only should be used for admin-only features. For private
access-related features `User#full_private_access?` should be used.

Backported from gitlab-org/gitlab-ee!2199

Fix https://gitlab.com/gitlab-org/gitlab-ce/issues/33223