From ff046c72004f0c00827d615a60f37153f54a6347 Mon Sep 17 00:00:00 2001 From: Marin Jankovski Date: Thu, 17 Oct 2013 10:42:39 +0200 Subject: Fallback to internal issue tracker if config is invalid. --- app/helpers/issues_helper.rb | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'app') diff --git a/app/helpers/issues_helper.rb b/app/helpers/issues_helper.rb index 5977c9cbae2..5e89aec4f3a 100644 --- a/app/helpers/issues_helper.rb +++ b/app/helpers/issues_helper.rb @@ -16,7 +16,7 @@ module IssuesHelper def url_for_project_issues return "" if @project.nil? - if @project.used_default_issues_tracker? + if @project.used_default_issues_tracker? || config_disabled? project_issues_path(@project) else url = Gitlab.config.issues_tracker[@project.issues_tracker]["project_url"] @@ -28,7 +28,7 @@ module IssuesHelper def url_for_new_issue return "" if @project.nil? - if @project.used_default_issues_tracker? + if @project.used_default_issues_tracker? || config_disabled? url = new_project_issue_path project_id: @project else url = Gitlab.config.issues_tracker[@project.issues_tracker]["new_issue_url"] @@ -40,7 +40,7 @@ module IssuesHelper def url_for_issue(issue_iid) return "" if @project.nil? - if @project.used_default_issues_tracker? + if @project.used_default_issues_tracker? || config_disabled? url = project_issue_url project_id: @project, id: issue_iid else url = Gitlab.config.issues_tracker[@project.issues_tracker]["issues_url"] @@ -59,4 +59,9 @@ module IssuesHelper "" end end + + def config_disabled? + return false if Gitlab.config.issues_tracker && Gitlab.config.issues_tracker.values.any? + true + end end -- cgit v1.2.1 From 78e591c737f65f7c5998d554d4c059b9a2d0674f Mon Sep 17 00:00:00 2001 From: Marin Jankovski Date: Thu, 17 Oct 2013 10:57:58 +0200 Subject: Use more descriptive name for config check. --- app/helpers/issues_helper.rb | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'app') diff --git a/app/helpers/issues_helper.rb b/app/helpers/issues_helper.rb index 5e89aec4f3a..f6e8ad911b7 100644 --- a/app/helpers/issues_helper.rb +++ b/app/helpers/issues_helper.rb @@ -16,7 +16,7 @@ module IssuesHelper def url_for_project_issues return "" if @project.nil? - if @project.used_default_issues_tracker? || config_disabled? + if @project.used_default_issues_tracker? || config_issues_tracker? project_issues_path(@project) else url = Gitlab.config.issues_tracker[@project.issues_tracker]["project_url"] @@ -28,7 +28,7 @@ module IssuesHelper def url_for_new_issue return "" if @project.nil? - if @project.used_default_issues_tracker? || config_disabled? + if @project.used_default_issues_tracker? || config_issues_tracker? url = new_project_issue_path project_id: @project else url = Gitlab.config.issues_tracker[@project.issues_tracker]["new_issue_url"] @@ -40,7 +40,7 @@ module IssuesHelper def url_for_issue(issue_iid) return "" if @project.nil? - if @project.used_default_issues_tracker? || config_disabled? + if @project.used_default_issues_tracker? || config_issues_tracker? url = project_issue_url project_id: @project, id: issue_iid else url = Gitlab.config.issues_tracker[@project.issues_tracker]["issues_url"] @@ -60,7 +60,7 @@ module IssuesHelper end end - def config_disabled? + def config_issues_tracker? return false if Gitlab.config.issues_tracker && Gitlab.config.issues_tracker.values.any? true end -- cgit v1.2.1 From 52739518bc8c01f0c9821b01ce5bc3bcd92f2159 Mon Sep 17 00:00:00 2001 From: Marin Jankovski Date: Thu, 17 Oct 2013 11:36:23 +0200 Subject: Make clearer use of the check. --- app/helpers/issues_helper.rb | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) (limited to 'app') diff --git a/app/helpers/issues_helper.rb b/app/helpers/issues_helper.rb index f6e8ad911b7..2221583912d 100644 --- a/app/helpers/issues_helper.rb +++ b/app/helpers/issues_helper.rb @@ -16,7 +16,7 @@ module IssuesHelper def url_for_project_issues return "" if @project.nil? - if @project.used_default_issues_tracker? || config_issues_tracker? + if @project.used_default_issues_tracker? || !external_issues_tracker_enabled? project_issues_path(@project) else url = Gitlab.config.issues_tracker[@project.issues_tracker]["project_url"] @@ -28,7 +28,7 @@ module IssuesHelper def url_for_new_issue return "" if @project.nil? - if @project.used_default_issues_tracker? || config_issues_tracker? + if @project.used_default_issues_tracker? || !external_issues_tracker_enabled? url = new_project_issue_path project_id: @project else url = Gitlab.config.issues_tracker[@project.issues_tracker]["new_issue_url"] @@ -40,7 +40,7 @@ module IssuesHelper def url_for_issue(issue_iid) return "" if @project.nil? - if @project.used_default_issues_tracker? || config_issues_tracker? + if @project.used_default_issues_tracker? || !external_issues_tracker_enabled? url = project_issue_url project_id: @project, id: issue_iid else url = Gitlab.config.issues_tracker[@project.issues_tracker]["issues_url"] @@ -60,8 +60,12 @@ module IssuesHelper end end - def config_issues_tracker? - return false if Gitlab.config.issues_tracker && Gitlab.config.issues_tracker.values.any? - true + # Checks if issues_tracker setting exists in gitlab.yml + def external_issues_tracker_enabled? + if Gitlab.config.issues_tracker && Gitlab.config.issues_tracker.values.any? + true + else + false + end end end -- cgit v1.2.1