Skip to content
Snippets Groups Projects

New issue button was not follows to external tracker if it is selected. ...

Merged gitlab-qa-bot requested to merge github/fork/Andrew8xx8/issue-button-fix into master

Created by: Andrew8xx8

...fixed #3386 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
49 49 project_issues_filter_path(@project)
50 50 else
51 url = Settings[:issues_tracker][@project.issues_tracker]["project_url"]
51 url = Gitlab.config.issues_tracker[@project.issues_tracker]["project_url"]
52 52 url.gsub(':project_id', @project.id.to_s)
53 53 .gsub(':issues_tracker_id', @project.issues_tracker_id.to_s)
54 54 end
55 55 end
56 56
57 def url_for_new_issue
58 return "" if @project.nil?
59
60 if @project.used_default_issues_tracker?
61 url = new_project_issue_path project_id: @project
62 else
63 url = Gitlab.config.issues_tracker[@project.issues_tracker]["new_issue_url"]
  • Created by: dzaporozhets

    Lets use Gitlab.config everywhere. Using both Settings and Gitlab.config makes confusion

    By Administrator on 2013-03-26T17:04:16 (imported from GitLab project)

    By Administrator on 2013-03-26T17:04:16 (imported from GitLab)

  • Created by: coveralls

    Coverage decreased (-0.16%) when pulling 5e6f45b0 on Andrew8xx8:issue-button-fix into 90db28d6 on gitlabhq:master.

    View Details

    By Administrator on 2013-03-26T17:37:57 (imported from GitLab project)

    By Administrator on 2013-03-26T17:37:57 (imported from GitLab)

  • Created by: dzaporozhets

    thank you

    By Administrator on 2013-03-30T22:10:12 (imported from GitLab project)

    By Administrator on 2013-03-30T22:10:12 (imported from GitLab)

  • Please register or sign in to reply
    Loading