From a21fc59ea5b6e38068d90e64c7a2a2c097d588a3 Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Mon, 09 Apr 2012 08:43:03 -0400 Subject: [PATCH] Fixed: could not clear any RepositoryModel setting list (issue 81) --- src/com/gitblit/GitBlit.java | 32 +++++++++++++++++++++++++++----- 1 files changed, 27 insertions(+), 5 deletions(-) diff --git a/src/com/gitblit/GitBlit.java b/src/com/gitblit/GitBlit.java index 7b557d7..e166795 100644 --- a/src/com/gitblit/GitBlit.java +++ b/src/com/gitblit/GitBlit.java @@ -1050,21 +1050,43 @@ config.setBoolean("gitblit", null, "showReadme", repository.showReadme); config.setBoolean("gitblit", null, "skipSizeCalculation", repository.skipSizeCalculation); config.setBoolean("gitblit", null, "skipSummaryMetrics", repository.skipSummaryMetrics); - config.setStringList("gitblit", null, "federationSets", repository.federationSets); config.setString("gitblit", null, "federationStrategy", repository.federationStrategy.name()); config.setBoolean("gitblit", null, "isFederated", repository.isFederated); - if (!ArrayUtils.isEmpty(repository.preReceiveScripts)) { + + // federation sets + if (ArrayUtils.isEmpty(repository.federationSets)) { + config.unset("gitblit", null, "federationSets"); + } else { + config.setStringList("gitblit", null, "federationSets", repository.federationSets); + } + + // pre receive scripts + if (ArrayUtils.isEmpty(repository.preReceiveScripts)) { + config.unset("gitblit", null, "preReceiveScript"); + } else { config.setStringList("gitblit", null, "preReceiveScript", repository.preReceiveScripts); } - if (!ArrayUtils.isEmpty(repository.postReceiveScripts)) { + + // post receive scripts + if (ArrayUtils.isEmpty(repository.postReceiveScripts)) { + config.unset("gitblit", null, "postReceiveScript"); + } else { config.setStringList("gitblit", null, "postReceiveScript", repository.postReceiveScripts); } - if (!ArrayUtils.isEmpty(repository.mailingLists)) { + + // mailing lists + if (ArrayUtils.isEmpty(repository.mailingLists)) { + config.unset("gitblit", null, "mailingList"); + } else { config.setStringList("gitblit", null, "mailingList", repository.mailingLists); } - if (!ArrayUtils.isEmpty(repository.indexedBranches)) { + + // indexed branches + if (ArrayUtils.isEmpty(repository.indexedBranches)) { + config.unset("gitblit", null, "indexBranch"); + } else { config.setStringList("gitblit", null, "indexBranch", repository.indexedBranches); } try { -- Gitblit v1.9.1