Fix a bunch of mistakes found by PyFlakes in the new materials code

 - Legacy-Id: 7839

Author: OleLaursen

ietf/doc/utils.py

@@ -214,7 +214,7 @@ def get_document_content(key, filename, split=True, markup=True):
     try:
         with open(filename, 'rb') as f:
             raw_content = f.read()
-    except IOError as e:
+    except IOError:
         error = "Error; cannot read ("+key+")"
         return error
 

ietf/doc/views_material.py

@@ -1,12 +1,10 @@
 # views for managing group materials (slides, ...)
-import re
 import os
 import datetime
-import shutil
 
 from django import forms
 from django.shortcuts import render, get_object_or_404, redirect
-from django.http import HttpResponse, HttpResponseForbidden, Http404, HttpResponseRedirect
+from django.http import HttpResponseForbidden, Http404
 from django.utils.html import mark_safe
 from django.utils.text import slugify
 from django.contrib.auth.decorators import login_required
@@ -19,7 +17,6 @@
 from ietf.doc.utils import add_state_change_event
 from ietf.group.models import Group
 from ietf.group.utils import can_manage_materials
-from ietf.ietfauth.utils import has_role
 
 @login_required
 def choose_material_type(request, acronym):

ietf/group/edit.py

@@ -9,18 +9,15 @@
 from django.shortcuts import render, get_object_or_404, redirect
 from django.http import HttpResponse, HttpResponseForbidden, Http404, HttpResponseRedirect
 from django.utils.html import mark_safe
-from django.utils.text import slugify
 from django.contrib.auth.decorators import login_required
-from django.core.urlresolvers import reverse as urlreverse
 
 import debug                            # pyflakes:ignore
 
-from ietf.doc.models import Document, DocAlias, DocTagName, DocTypeName, DocEvent, State
-from ietf.doc.models import NewRevisionDocEvent, save_document_in_history
-from ietf.doc.utils import get_tags_for_stream_id, add_state_change_event
+from ietf.doc.models import Document, DocAlias, DocTagName, State, save_document_in_history
+from ietf.doc.utils import get_tags_for_stream_id
 from ietf.group.models import ( Group, Role, GroupEvent, GroupHistory, GroupStateName,
     GroupStateTransitions, GroupTypeName, GroupURL, ChangeStateGroupEvent )
-from ietf.group.utils import save_group_in_history, can_manage_group_type, can_manage_materials
+from ietf.group.utils import save_group_in_history, can_manage_group_type
 from ietf.group.utils import get_group_or_404
 from ietf.ietfauth.utils import has_role
 from ietf.person.forms import EmailsField

ietf/group/feeds.py

@@ -22,7 +22,7 @@ def title(self, obj):
     def link(self, obj):
 	if not obj:
 	    raise FeedDoesNotExist
-        return group.about_url()
+        return obj.about_url()
 
     def description(self, obj):
 	return self.title(obj)
@@ -40,7 +40,7 @@ def item_link(self, obj):
         if isinstance(obj, DocEvent):
             return urlreverse("doc_view", kwargs={'name': obj.doc_id })
         elif isinstance(obj, GroupEvent):
-            return group.about_url()
+            return obj.group.about_url()
 
     def item_pubdate(self, obj):
 	return obj.time

ietf/group/info.py

@@ -35,10 +35,9 @@
 import os
 import itertools
 from tempfile import mkstemp
-import glob
 from collections import OrderedDict
 
-from django.shortcuts import get_object_or_404, render
+from django.shortcuts import render
 from django.template.loader import render_to_string
 from django.http import HttpResponse, Http404, HttpResponseRedirect
 from django.conf import settings

ietf/group/migrations/0007_auto__add_field_group_description__add_field_grouphistory_description.py

@@ -1,8 +1,6 @@
 # -*- coding: utf-8 -*-
-from south.utils import datetime_utils as datetime
 from south.db import db
 from south.v2 import SchemaMigration
-from django.db import models
 
 
 class Migration(SchemaMigration):
@@ -314,4 +312,4 @@ def backwards(self, orm):
         }
     }
 
-    complete_apps = ['group']
+    complete_apps = ['group']

ietf/group/milestones.py

@@ -5,13 +5,13 @@
 import json
 
 from django import forms
-from django.http import HttpResponse, HttpResponseForbidden, HttpResponseBadRequest, HttpResponseRedirect
-from django.shortcuts import render, get_object_or_404, redirect
+from django.http import HttpResponse, HttpResponseForbidden, HttpResponseBadRequest, HttpResponseRedirect, Http404
+from django.shortcuts import render, redirect
 from django.contrib.auth.decorators import login_required
 
 from ietf.doc.models import Document, DocEvent
 from ietf.doc.utils import get_chartering_type
-from ietf.group.models import Group, GroupMilestone, MilestoneGroupEvent
+from ietf.group.models import GroupMilestone, MilestoneGroupEvent
 from ietf.group.utils import (save_milestone_in_history, can_manage_group_type, milestone_reviewer_for_group_type,
                               get_group_or_404)
 from ietf.name.models import GroupMilestoneStateName

ietf/name/migrations/0020_sort_role_names.py

@@ -1,8 +1,5 @@
 # -*- coding: utf-8 -*-
-from south.utils import datetime_utils as datetime
-from south.db import db
 from south.v2 import DataMigration
-from django.db import models
 
 class Migration(DataMigration):