@@ -57,6 +57,18 @@ class Comment(search.SearchableModel): def username(self): return self.user.nickname().split("@")[0] + def radar_exists(self): + try: + return self.radar != None + except db.Error: + return False + + def radarnumber(self): + return self.radar_exists() and self.radar.number or "Deleted" + + def radartitle(self): + return self.radar_exists() and self.radar.title or "Deleted" + def replies(self): return Comment.gql("WHERE is_reply_to = :1", self) models.py @@ -9,7 +9,7 @@ <div class="commentWithReplies"> <div class="comment"> <a class="keyref" name="{{comment.key}}"></a> - <h4><a href="/{{ comment.radar.number }}#{{ comment.key }}">Open Radar {{ comment.radar.number }}: {{ comment.radar.title }}</a></h4> + <h4><a href="/{{ comment.radarnumber }}#{{ comment.key }}">Open Radar {{ comment.radarnumber }}: {{ comment.radartitle }}</a></h4> {% if comment.subject %} <h3>{{ comment.subject|escape }}</h3> {% endif %} @@ -17,7 +17,7 @@ {{ comment.html_body }} </div> <div class="meta"> - By <strong>{{ comment.username }}</strong> at <a href="/{{ comment.radar.number }}#{{ comment.key }}">{{ comment.posted_at }}</a> + By <strong>{{ comment.username }}</strong> at <a href="/{{ comment.radarnumber }}#{{ comment.key }}">{{ comment.posted_at }}</a> </div> </div> </div> templates/comments-recent.html efd1bbcde9958db33a31320d301301631e68f012 Tim Burks tim@neontology.com http://github.com/timburks/openradar/commit/ad8ad465eb602549298796254d4279804b87d3fc ad8ad465eb602549298796254d4279804b87d3fc 2009-01-15T20:38:39-08:00 2009-01-15T20:38:39-08:00 Made comments feed tolerant of comments whose radars have been deleted. In the future, we should delete comments associated with a radar when that radar is deleted. 06246efb51dd21e1f8da2b31d94dc5cb0d5fa781 Tim Burks tim@neontology.com