Merge branch '25106-hide-issue-mr-button-for-not-loggedin' into 'master'

hide new issue or MR button for not signed users for a group ## What does this MR do? This MR prevents user creating issues or MR without signing in ## Are there points in the code the reviewer needs to double check? NR ## Why was this MR needed? This issue is needed to ensure an user to be logged in before creating an issue or MR ## Screenshots (if relevant) ## Does this MR meet the acceptance criteria? - [x] [Changelog entry](https://docs.gitlab.com/ce/development/changelog.html) added - Tests - [x] All builds are passing - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [ ] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Closes #25106 See merge request !7902

Merge branch '25106-hide-issue-mr-button-for-not-loggedin' into 'master'
b544ec86 · Fatih Acet · 01b767bd · c877b152 · b544ec86 · b544ec86
Commit b544ec86 authored Dec 12, 2016 by Fatih Acet
3 changed files
--- a/app/views/groups/issues.html.haml
+++ b/app/views/groups/issues.html.haml
@@ -6,13 +6,13 @@
 - if group_issues(@group).exists?
  .top-area
    = render 'shared/issuable/nav', type: :issues
-    .nav-controls
+    - if current_user
-      - if current_user
+      .nav-controls
        = link_to url_for(params.merge(format: :atom, private_token: current_user.private_token)), class: 'btn' do
          = icon('rss')
          %span.icon-label
            Subscribe
-      = render 'shared/new_project_item_select', path: 'issues/new', label: "New Issue"
+        = render 'shared/new_project_item_select', path: 'issues/new', label: "New Issue"
  = render 'shared/issuable/filter', type: :issues

--- a/app/views/groups/merge_requests.html.haml
+++ b/app/views/groups/merge_requests.html.haml
@@ -2,8 +2,9 @@
 .top-area
  = render 'shared/issuable/nav', type: :merge_requests
-  .nav-controls
+  - if current_user
-    = render 'shared/new_project_item_select', path: 'merge_requests/new', label: "New Merge Request"
+    .nav-controls
+      = render 'shared/new_project_item_select', path: 'merge_requests/new', label: "New Merge Request"
 = render 'shared/issuable/filter', type: :merge_requests

--- a/changelogs/unreleased/25106-hide-issue-mr-button-for-not-loggedin.yml
+++ b/changelogs/unreleased/25106-hide-issue-mr-button-for-not-loggedin.yml
+---
+title: Prevent user creating issue or MR without signing in for a group
+merge_request: 7902
+author: