Skip to content
Snippets Groups Projects
Commit 974cb382 authored by John Carlson's avatar John Carlson
Browse files

Fix for tabs being off for a project with disabled sections

parent aa1babc0
No related branches found
No related tags found
No related merge requests found
Pipeline #
Loading
Loading
@@ -35,21 +35,21 @@ class ProjectSectionsPagerAdapter(context: ProjectActivity, fm: FragmentManager)
 
init {
 
val project = context.project
if (!project!!.isBuildEnabled) {
val project = context.project!!
if (isDisabled(project.isBuildEnabled)) {
Timber.d("Builds are disabled")
disabledSections.add(BUILDS_POS)
disabledSections.add(PIPELINES_POS)
}
if (!project.isIssuesEnabled) {
if (isDisabled(project.isIssuesEnabled)) {
Timber.d("Issues are disabled")
disabledSections.add(ISSUES_POS)
}
if (!project.isMergeRequestsEnabled) {
if (isDisabled(project.isMergeRequestsEnabled)) {
Timber.d("Merge requests are disabled")
disabledSections.add(MERGE_REQUESTS_POS)
}
if (!project.isIssuesEnabled && !project.isMergeRequestsEnabled) {
if (isDisabled(project.isIssuesEnabled) && isDisabled(project.isMergeRequestsEnabled)) {
Timber.d("Milestones are disabled")
disabledSections.add(MILESTONES_POS)
}
Loading
Loading
@@ -65,10 +65,7 @@ class ProjectSectionsPagerAdapter(context: ProjectActivity, fm: FragmentManager)
}
 
override fun getPageTitle(position: Int): CharSequence {
var position = position
position = getCorrectPosition(position)
return titles[position]
return titles[getCorrectPosition(position)]
}
 
override fun getItem(position: Int): Fragment {
Loading
Loading
@@ -91,6 +88,13 @@ class ProjectSectionsPagerAdapter(context: ProjectActivity, fm: FragmentManager)
throw IllegalStateException("Position exceeded on view pager")
}
 
private fun isDisabled(enabledState: Boolean?) : Boolean{
if (enabledState != null && !enabledState) {
return true
}
return false
}
private fun getCorrectPosition(position: Int): Int {
var correctPosition = position
for (i in 0..position) {
Loading
Loading
@@ -99,6 +103,6 @@ class ProjectSectionsPagerAdapter(context: ProjectActivity, fm: FragmentManager)
}
}
 
return position
return correctPosition
}
}
Loading
Loading
@@ -45,15 +45,15 @@ public class Project {
@JsonField(name = "path_with_namespace")
String pathWithNamespace;
@JsonField(name = "issues_enabled")
boolean issuesEnabled;
Boolean issuesEnabled;
@JsonField(name = "merge_requests_enabled")
boolean mergeRequestsEnabled;
Boolean mergeRequestsEnabled;
@JsonField(name = "wiki_enabled")
boolean wikiEnabled;
Boolean wikiEnabled;
@JsonField(name = "builds_enabled")
boolean buildEnabled;
Boolean buildEnabled;
@JsonField(name = "snippets_enabled")
boolean snippetsEnabled;
Boolean snippetsEnabled;
@JsonField(name = "created_at")
Date createdAt;
@JsonField(name = "last_activity_at")
Loading
Loading
@@ -135,23 +135,23 @@ public class Project {
return pathWithNamespace;
}
 
public boolean isIssuesEnabled() {
public Boolean isIssuesEnabled() {
return issuesEnabled;
}
 
public boolean isMergeRequestsEnabled() {
public Boolean isMergeRequestsEnabled() {
return mergeRequestsEnabled;
}
 
public boolean isWikiEnabled() {
public Boolean isWikiEnabled() {
return wikiEnabled;
}
 
public boolean isBuildEnabled() {
public Boolean isBuildEnabled() {
return buildEnabled;
}
 
public boolean isSnippetsEnabled() {
public Boolean isSnippetsEnabled() {
return snippetsEnabled;
}
 
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment