Skip to content
Snippets Groups Projects

Fix display of promote to group label

Merged Jan Provaznik requested to merge jp-label-fix into master

What does this MR do?

Fix display of promote to group label

Since label presenter is used in label index view, label class check doesn't work as expected because the class is now LabelPresenter.

Also label.subject doesn't work as expected now because Label's model subject method is shadowed by Gitlab's presenter's method which uses subject for referencing the original object.

Instead we use a presenter's method for both checks now.

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/62200

Does this MR meet the acceptance criteria?

Conformity

Performance and testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Jan Provaznik

Merge request reports

Pipeline #63459737 failed

Pipeline failed for 58dc21e7 on jp-label-fix

Test coverage 56.94% (-19.19%) from 1 job
Approved by

Merged by Lin Jen-ShinLin Jen-Shin 5 years ago (May 28, 2019 10:05am UTC)

Pipeline #63480460 failed

Pipeline failed for 9c2d0d87 on master

Test coverage 56.72% (-19.19%) from 1 job

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jan Provaznik
  • mentioned in issue #62217 (closed)

  • Jan Provaznik added 1 commit

    added 1 commit

    • f9a55f93 - Fix display of promote to group label

    Compare with previous version

  • assigned to @godfat

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading