Fix second 500 error with NULL restricted visibility levels (!30414) · Merge requests · GitLab.org / GitLab

Stan Hu requested to merge sh-clean-up-public-visiblity-level-check into master Apr 24, 2020

!30313 (merged) fixed one 500 error when application_settings.restricted_visibility_levels is NULL, but !22522 (merged) introduced the public visiblity check in two places.

To fix this problem and reduce code duplication, move this check into Gitlab::VisibilityLevel, where NULL values of restricted_visibility_levels was already handled.

Closes #215638 (closed)

Edited Apr 24, 2020 by Stan Hu

Fix second 500 error with NULL restricted visibility levels

Merge request reports