You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by jo...@apache.org on 2012/10/29 21:10:46 UTC
[12/25] git commit: [#5145] Fixed non-production tools showing up for
install on new projects
[#5145] Fixed non-production tools showing up for install on new projects
Signed-off-by: Cory Johns <jo...@geek.net>
Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/d3f88cce
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/d3f88cce
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/d3f88cce
Branch: refs/heads/cj/5076
Commit: d3f88cce738ab93b9257f46c9da042978521e8ae
Parents: 88c64df
Author: Cory Johns <jo...@geek.net>
Authored: Wed Oct 24 16:51:24 2012 +0000
Committer: Cory Johns <jo...@geek.net>
Committed: Fri Oct 26 20:32:10 2012 +0000
----------------------------------------------------------------------
Allura/allura/lib/plugin.py | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/d3f88cce/Allura/allura/lib/plugin.py
----------------------------------------------------------------------
diff --git a/Allura/allura/lib/plugin.py b/Allura/allura/lib/plugin.py
index d6f4ee3..02d652a 100644
--- a/Allura/allura/lib/plugin.py
+++ b/Allura/allura/lib/plugin.py
@@ -314,8 +314,9 @@ class ProjectRegistrationProvider(object):
## Dynamically generating CheckboxSet of installable tools
self.add_project_widget.fields.tools = forms.ew.CheckboxSet(
name="tools", options=[
- forms.ew.Option(label=tool.tool_label, html_value=ep) \
- for ep,tool in g.entry_points["tool"].iteritems() if tool.installable
+ forms.ew.Option(label=tool.tool_label, html_value=ep)
+ for ep,tool in g.entry_points["tool"].iteritems()
+ if tool.installable and tool.status == 'production'
], selected=True
)
# have to update it via index as well because of crazy