Link to personal profile pages from group photo pages also when no photo is available -- the profile pages now contains more than just the photo.

 - Legacy-Id: 15409

Author: levkowetz

ietf/templates/group/all_photos.html

@@ -28,18 +28,18 @@ <h1>{{ group_type | upper }} {{ role }} Photos</h1>
         {% for person_with_groups in person_groups %}
          <li>
          <div class="well photo-thumbnail">
-	   {% if person_with_groups.grouper.photo %}<a href="{% url 'ietf.person.views.profile' email_or_name=person_with_groups.grouper.plain_name %}">{% endif %}
-	   <div>
-           {% if person_with_groups.grouper.photo_thumb %}
-	     <img width=100 src="{{person_with_groups.grouper.photo_thumb.url}}" alt="Photo of {{person_with_groups.grouper.name}}"/>
-           {% else %}
-	     <img width=100 src="{{ MEDIA_URL }}photo/nopictureavailable.jpg" alt="No picture available"/>
-           {% endif %}
-	   </div>
-	   <div class="photo-name">
-	     <strong>{{person_with_groups.grouper.plain_name}}</strong>
-	   </div>
-	   {% if person_with_groups.grouper.photo %}</a>{% endif %}
+	   <a href="{% url 'ietf.person.views.profile' email_or_name=person_with_groups.grouper.plain_name %}">
+	     <div>
+	     {% if person_with_groups.grouper.photo_thumb %}
+	       <img width=100 src="{{person_with_groups.grouper.photo_thumb.url}}" alt="Photo of {{person_with_groups.grouper.name}}"/>
+	     {% else %}
+	       <img width=100 src="{{ MEDIA_URL }}photo/nopictureavailable.jpg" alt="No picture available"/>
+	     {% endif %}
+	     </div>
+	     <div class="photo-name">
+	       <strong>{{person_with_groups.grouper.plain_name}}</strong>
+	     </div>
+	   </a>
            <div class="photo-role-list">
            {% for role in person_with_groups.list %}   
 	     <a href="{% url 'ietf.group.views.group_home' acronym=role.group.acronym %}">{{role.group.acronym}}</a>

ietf/templates/group/group_photos.html

@@ -26,22 +26,22 @@ <h1>{{ group }} ({{group.acronym}}) Photos</h1>
         <ul class="list-inline">
         {% regroup role_name.list by person as person_groups %}
         {% for person_with_groups in person_groups %}
-         <li>
-	 {% if person_with_groups.grouper.photo %}<a href="{% url 'ietf.person.views.profile' email_or_name=person_with_groups.grouper.plain_name %}">{% endif %}
-         <div class="well photo-thumbnail">
-	   <div>
-           {% if person_with_groups.grouper.photo_thumb %}
-		 <img width=100 src="{{person_with_groups.grouper.photo_thumb.url}}" alt="Photo of {{person_with_groups.grouper.name}}" />
-           {% else %}
-		 <img width=100 src="{{ MEDIA_URL }}photo/nopictureavailable.jpg" alt="No picture available"/>
-           {% endif %}
-	   </div>
-	   <div >
-	     <strong>{{person_with_groups.grouper.plain_name}}</strong>
-	   </div>
-         </div>
-	 {% if person_with_groups.grouper.photo %}</a>{% endif %}
-         </li>
+	  <li>
+	    <a href="{% url 'ietf.person.views.profile' email_or_name=person_with_groups.grouper.plain_name %}">
+	     <div class="well photo-thumbnail">
+	       <div>
+	       {% if person_with_groups.grouper.photo_thumb %}
+		     <img width=100 src="{{person_with_groups.grouper.photo_thumb.url}}" alt="Photo of {{person_with_groups.grouper.name}}" />
+	       {% else %}
+		     <img width=100 src="{{ MEDIA_URL }}photo/nopictureavailable.jpg" alt="No picture available"/>
+	       {% endif %}
+	       </div>
+	       <div >
+		 <strong>{{person_with_groups.grouper.plain_name}}</strong>
+	       </div>
+	     </div>
+	    </a>
+	  </li>
         {% endfor %}
         </ul>
       </div>