diff options
| author | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2016-11-01 00:03:38 -0200 |
|---|---|---|
| committer | Douglas Barbosa Alexandre <dbalexandre@gmail.com> | 2016-11-17 15:10:13 -0200 |
| commit | d45c6eca2750abd5614be41a6fe01033f655ef1e (patch) | |
| tree | f2d52c31f08286a6c14a86b1fefdde09d20b2bd8 | |
| parent | 38f16582725709ee89c7eeadd4415ed3e2fa57dc (diff) | |
| download | gitlab-ce-d45c6eca2750abd5614be41a6fe01033f655ef1e.tar.gz | |
Refactoring label subscription status to accept a project
| -rw-r--r-- | app/helpers/labels_helper.rb | 4 | ||||
| -rw-r--r-- | app/views/shared/_label.html.haml | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/app/helpers/labels_helper.rb b/app/helpers/labels_helper.rb index 221a84b042f..e1db6007c09 100644 --- a/app/helpers/labels_helper.rb +++ b/app/helpers/labels_helper.rb @@ -148,10 +148,10 @@ module LabelsHelper end end - def label_subscription_status(label) + def label_subscription_status(label, project) case label when GroupLabel then 'Subscribing to group labels is currently not supported.' - when ProjectLabel then label.subscribed?(current_user) ? 'subscribed' : 'unsubscribed' + when ProjectLabel then label.subscribed?(current_user, project) ? 'subscribed' : 'unsubscribed' end end diff --git a/app/views/shared/_label.html.haml b/app/views/shared/_label.html.haml index 6ccdef0df46..b985ce4884e 100644 --- a/app/views/shared/_label.html.haml +++ b/app/views/shared/_label.html.haml @@ -20,8 +20,8 @@ = pluralize open_issues_count, 'open issue' - if current_user %li.label-subscription{ data: toggle_subscription_data(label) } - %a.js-subscribe-button.label-subscribe-button.subscription-status{ role: "button", href: "#", data: { toggle: "tooltip", status: label_subscription_status(label) } } %span= label_subscription_toggle_button_text(label) + %a.js-subscribe-button.label-subscribe-button.subscription-status{ role: "button", href: "#", data: { toggle: "tooltip", status: label_subscription_status(label, @project) } } - if can?(current_user, :admin_label, label) %li = link_to 'Edit', edit_label_path(label) @@ -36,8 +36,8 @@ - if current_user .label-subscription.inline{ data: toggle_subscription_data(label) } - %button.js-subscribe-button.label-subscribe-button.btn.btn-transparent.btn-action.subscription-status{ type: "button", title: label_subscription_toggle_button_text(label), data: { toggle: "tooltip", status: label_subscription_status(label) } } %span.sr-only= label_subscription_toggle_button_text(label) + %button.js-subscribe-button.label-subscribe-button.btn.btn-transparent.btn-action.subscription-status{ type: "button", title: label_subscription_toggle_button_text(label, @project), data: { toggle: "tooltip", status: label_subscription_status(label, @project) } } = icon('eye', class: 'label-subscribe-button-icon', disabled: label.is_a?(GroupLabel)) = icon('spinner spin', class: 'label-subscribe-button-loading') |