Skip to content

Commit 498777a

Browse files
committed
Made ietf/secr/{drafts,roles,ipradmin} pyflakes-clean.
- Legacy-Id: 7482
1 parent 5bbbe24 commit 498777a

13 files changed

Lines changed: 55 additions & 74 deletions

File tree

ietf/secr/drafts/email.py

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,15 @@
1+
import datetime
2+
import glob
3+
import os
4+
import time
5+
16
from django.conf import settings
2-
from django.core.exceptions import ObjectDoesNotExist
37
from django.template.loader import render_to_string
48

59
from ietf.message.models import Message, SendQueue
610
from ietf.message.utils import send_scheduled_message_from_send_queue
711
from ietf.doc.models import DocumentAuthor
812
from ietf.person.models import Person
9-
from ietf.secr.utils.document import get_start_date
10-
11-
import datetime
12-
import glob
13-
import os
14-
import time
1513

1614
def announcement_from_form(data, **kwargs):
1715
'''

ietf/secr/drafts/forms.py

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,15 @@
1+
import datetime
2+
import re
3+
import os
4+
15
from django import forms
2-
from django.forms.formsets import BaseFormSet
36

4-
from ietf.doc.models import *
5-
from ietf.name.models import IntendedStdLevelName
7+
from ietf.doc.models import Document, DocAlias, State
8+
from ietf.name.models import IntendedStdLevelName, DocRelationshipName
69
from ietf.group.models import Group
10+
from ietf.person.models import Person, Email
11+
from ietf.secr.groups.forms import get_person
712

8-
from ietf.secr.utils.ams_utils import get_base, get_revision
9-
from ietf.secr.groups.forms import RoleForm, get_person
10-
11-
import datetime
12-
import re
13-
from os.path import splitext
1413

1514
# ---------------------------------------------
1615
# Select Choices
@@ -365,15 +364,15 @@ def clean(self):
365364
names = []
366365
for file in (txt,xml,pdf,ps):
367366
if file:
368-
base = splitext(file.name)[0]
367+
base = os.path.splitext(file.name)[0]
369368
if base not in names:
370369
names.append(base)
371370

372371
if len(names) > 1:
373372
raise forms.ValidationError, "All files must have the same base name"
374373

375374
# ensure that the basename is unique
376-
base = splitext(txt.name)[0]
375+
base = os.path.splitext(txt.name)[0]
377376
if Document.objects.filter(name=base[:-3]):
378377
raise forms.ValidationError, "This document filename already exists: %s" % base[:-3]
379378

ietf/secr/drafts/models.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
from django.db import models
1+

ietf/secr/drafts/tests.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,24 +1,23 @@
11
from django.core.urlresolvers import reverse
2-
from ietf.utils import TestCase
32

3+
from ietf.utils import TestCase
44
from ietf.doc.models import Document
55
from ietf.utils.test_data import make_test_data
66

7-
from pyquery import PyQuery
87

98
SECR_USER='secretary'
109

1110
class MainTestCase(TestCase):
1211
def test_main(self):
1312
"Main Test"
14-
draft = make_test_data()
13+
make_test_data()
1514
url = reverse('drafts')
1615
response = self.client.get(url, REMOTE_USER=SECR_USER)
1716
self.assertEqual(response.status_code, 200)
1817

1918
def test_view(self):
2019
"View Test"
21-
draft = make_test_data()
20+
make_test_data()
2221
drafts = Document.objects.filter(type='draft')
2322
url = reverse('drafts_view', kwargs={'id':drafts[0].name})
2423
response = self.client.get(url, REMOTE_USER=SECR_USER)

ietf/secr/drafts/urls.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
from django.conf.urls import *
1+
from django.conf.urls import patterns, url
22

33
urlpatterns = patterns('ietf.secr.drafts.views',
44
url(r'^$', 'search', name='drafts'),

ietf/secr/drafts/views.py

Lines changed: 23 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -1,34 +1,33 @@
1+
import datetime
2+
import glob
3+
import os
4+
import shutil
5+
16
from django.conf import settings
2-
from django.contrib.auth.decorators import login_required
37
from django.contrib import messages
4-
from django.core.exceptions import ObjectDoesNotExist, MultipleObjectsReturned
5-
from django.core.urlresolvers import reverse
6-
from django.db.models import get_model, Max, Q
8+
from django.db.models import Max
79
from django.forms.formsets import formset_factory
8-
from django.forms.models import inlineformset_factory, modelformset_factory
9-
from django.http import HttpResponseRedirect, HttpResponse
1010
from django.shortcuts import render_to_response, get_object_or_404, redirect
1111
from django.template import RequestContext
12+
from django.template.loader import render_to_string
1213

13-
from email import *
14-
from forms import *
14+
#from email import *
15+
from ietf.doc.models import Document, DocumentAuthor, DocAlias, DocRelationshipName, RelatedDocument, State
16+
from ietf.doc.models import DocEvent, NewRevisionDocEvent
17+
from ietf.doc.models import save_document_in_history
1518
from ietf.meeting.models import Meeting
1619
from ietf.name.models import StreamName
17-
from ietf.doc.models import Document, DocumentAuthor
18-
from ietf.doc.utils import augment_with_start_time
19-
from ietf.submit.models import Submission, Preapproval, DraftSubmissionStateName, SubmissionEvent
20-
from ietf.utils.draft import Draft
20+
from ietf.person.models import Person
21+
from ietf.secr.drafts.email import announcement_from_form, get_email_initial
22+
from ietf.secr.drafts.forms import ( AddModelForm, AuthorForm, BaseRevisionModelForm, EditModelForm,
23+
EmailForm, ExtendForm, ReplaceForm, RevisionModelForm, RfcModelForm,
24+
RfcObsoletesForm, SearchForm, UploadForm, WithdrawForm )
2125
from ietf.secr.proceedings.proc_utils import get_progress_stats
2226
from ietf.secr.sreq.views import get_meeting
23-
from ietf.secr.utils.ams_utils import get_base, get_email
27+
from ietf.secr.utils.ams_utils import get_base
2428
from ietf.secr.utils.document import get_rfc_num, get_start_date
25-
26-
import datetime
27-
import glob
28-
import os
29-
import shutil
30-
import textwrap
31-
import time
29+
from ietf.submit.models import Submission, Preapproval, DraftSubmissionStateName, SubmissionEvent
30+
from ietf.utils.draft import Draft
3231

3332

3433
# -------------------------------------------------
@@ -64,14 +63,6 @@ def get_action_details(draft, session):
6463

6564
return result
6665

67-
def get_doc_url(doc):
68-
name = doc.name
69-
if doc.get_state_slug() == "rfc":
70-
aliases = self.docalias_set.filter(name__startswith="rfc")
71-
if aliases:
72-
name = aliases[0].name
73-
return urlreverse('drafts_view', kwargs={ 'id': name })
74-
7566
def handle_uploaded_file(f):
7667
'''
7768
Save uploaded draft files to temporary directory
@@ -395,9 +386,7 @@ def do_withdraw(draft,request):
395386
# Reporting View Functions
396387
# -------------------------------------------------
397388
def report_id_activity(start,end):
398-
399-
from django.db.models import Min
400-
389+
401390
# get previous meeting
402391
meeting = Meeting.objects.filter(date__lt=datetime.datetime.now(),type='ietf').order_by('-date')[0]
403392
syear,smonth,sday = start.split('-')
@@ -437,9 +426,9 @@ def report_id_activity(start,end):
437426
monday = Meeting.get_ietf_monday()
438427
cutoff = monday + datetime.timedelta(days=3)
439428
ff1_date = cutoff - datetime.timedelta(days=28)
440-
ff2_date = cutoff - datetime.timedelta(days=21)
441-
ff3_date = cutoff - datetime.timedelta(days=14)
442-
ff4_date = cutoff - datetime.timedelta(days=7)
429+
#ff2_date = cutoff - datetime.timedelta(days=21)
430+
#ff3_date = cutoff - datetime.timedelta(days=14)
431+
#ff4_date = cutoff - datetime.timedelta(days=7)
443432

444433
ff_docs = Document.objects.filter(type='draft').filter(docevent__type='new_revision',
445434
docevent__newrevisiondocevent__rev='00',

ietf/secr/ipradmin/views.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
from django.utils.safestring import mark_safe
1111
from django.shortcuts import redirect
1212

13-
from ietf.secr.lib import template, jsonapi
13+
from ietf.secr.lib.template import template, jsonapi
1414
from ietf.secr.ipradmin.managers import IprDetailManager
1515
from ietf.secr.ipradmin.forms import IprDetailForm, IPRContactFormset
1616
from ietf.secr.utils.document import get_rfc_num, is_draft

ietf/secr/lib/__init__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
from template import template, jsonapi
1+

ietf/secr/roles/forms.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
1-
from django import forms
2-
from ietf.secr.areas.models import *
31
import re
42

3+
from django import forms
4+
5+
56
class LiaisonForm(forms.Form):
67
liaison_name = forms.CharField(max_length=100,label='Name',help_text="To see a list of people type the first name, or last name, or both.")
78
affiliation = forms.CharField(max_length=50)

ietf/secr/roles/models.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
from django.db import models
1+

0 commit comments

Comments
 (0)