Only show a link to the tools page for a group if the tools page is likely to exist, i.e., the group is a WG and it is or has been active.

 - Legacy-Id: 7264

Author: levkowetz

ietf/group/models.py

@@ -104,6 +104,9 @@ def json_dict(self, host_scheme):
         group1['comments']     = self.comments
         return group1
 
+    def has_tools_page(self):
+        return self.type_id in ['wg', ] and self.state_id in ['active', 'dormant', 'replaced', 'conclude']
+
 class GroupHistory(GroupInfo):
     group = models.ForeignKey(Group, related_name='history_set')
     acronym = models.CharField(max_length=40)

ietf/templates/wginfo/group_base.html

@@ -1,5 +1,5 @@
 {% extends "base.html" %}
-{% comment %}
+{% comment %}<!--
 Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
 All rights reserved. Contact: Pasi Eronen <pasi.eronen@nokia.com> 
 
@@ -31,6 +31,7 @@
 THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+-->
 {% endcomment %}
 {% load ietf_filters %}
 {% block title %}{{ group.name }} ({{ group.acronym }}) - {% block group_subtitle %}{% endblock %}{% endblock %}
@@ -67,11 +68,13 @@ <h1>{{ group.name}} ({{ group.acronym }})
   <div>
   <a {% if selected == "documents" %}class="selected"{% else %}href="{% url "ietf.wginfo.views.group_documents" acronym=group.acronym %}"{% endif %}>Documents</a> |
   <a {% if selected == "charter" %}class="selected"{% else %}href="{% url "ietf.wginfo.views.group_charter" acronym=group.acronym %}"{% endif %}>Charter</a> |
-  <a {% if selected == "history" %}class="selected"{% else %}href="{% url "ietf.wginfo.views.history" acronym=group.acronym %}"{% endif %}>History</a> |
+  <a {% if selected == "history" %}class="selected"{% else %}href="{% url "ietf.wginfo.views.history" acronym=group.acronym %}"{% endif %}>History</a>
   {% if group.list_archive|startswith:"http:" or group.list_archive|startswith:"https:" or group.list_archive|startswith:"ftp:" %}
-  <a href="{{ group.list_archive }}">List Archive &raquo;</a> |
+  | <a href="{{ group.list_archive }}">List Archive &raquo;</a>
+  {% endif %}
+  {% if group.has_tools_page %}
+  | <a href="http://tools.ietf.org/wg/{{ group.acronym }}/">Tools WG Page &raquo;</a>
   {% endif %}
-  <a href="http://tools.ietf.org/wg/{{ group.acronym }}/">Tools WG Page &raquo;</a>
   </div>
 
   {% if menu_actions %}