Skip to content

Commit 3220bf3

Browse files
chore: replace last few datetime.date.today() calls with date_today()
1 parent 5b5e2dd commit 3220bf3

27 files changed

Lines changed: 65 additions & 47 deletions

bin/add-old-drafts-from-archive.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
from django.core.validators import validate_email, ValidationError
1717
from ietf.utils.draft import PlaintextDraft
1818
from ietf.submit.utils import update_authors
19+
from ietf.utils.timezone import date_today
1920

2021
import debug # pyflakes:ignore
2122

@@ -140,7 +141,7 @@ def turn_into_unicode(s):
140141
doc = doc,
141142
rev = rev,
142143
by = system,
143-
desc = "Revision added from id-archive on %s by %s"%(datetime.date.today(),sys.argv[0]),
144+
desc = "Revision added from id-archive on %s by %s"%(date_today(),sys.argv[0]),
144145
time=time,
145146
)
146147
events.append(e)

ietf/bin/rfc-editor-index-updates

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ from django.core.mail import mail_admins
2929
from ietf.doc.utils import rebuild_reference_relations
3030
from ietf.utils.log import log
3131
from ietf.utils.pipe import pipe
32+
from ietf.utils.timezone import date_today
3233

3334
import ietf.sync.rfceditor
3435

@@ -39,7 +40,7 @@ parser.add_option("-d", dest="skip_date",
3940

4041
options, args = parser.parse_args()
4142

42-
skip_date = datetime.date.today() - datetime.timedelta(days=365)
43+
skip_date = date_today() - datetime.timedelta(days=365)
4344
if options.skip_date:
4445
skip_date = datetime.datetime.strptime(options.skip_date, "%Y-%m-%d").date()
4546

ietf/doc/factories.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
from ietf.person.factories import PersonFactory
2020
from ietf.group.factories import RoleFactory
2121
from ietf.utils.text import xslugify
22+
from ietf.utils.timezone import date_today
2223

2324

2425
def draft_name_generator(type_id,group,n):
@@ -321,7 +322,7 @@ def states(obj, create, extracted, **kwargs):
321322
# This is very skeletal. It is enough for the tests that use it now, but when it's needed, it will need to be improved with, at least, a group generator that backs the object with a review team.
322323
class ReviewFactory(BaseDocumentFactory):
323324
type_id = 'review'
324-
name = factory.LazyAttribute(lambda o: 'review-doesnotexist-00-%s-%s'%(o.group.acronym,datetime.date.today().isoformat()))
325+
name = factory.LazyAttribute(lambda o: 'review-doesnotexist-00-%s-%s'%(o.group.acronym,date_today().isoformat()))
325326
group = factory.SubFactory('ietf.group.factories.GroupFactory',type_id='review')
326327

327328
class DocAliasFactory(factory.django.DjangoModelFactory):

ietf/doc/models.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -922,8 +922,9 @@ def future_presentations(self):
922922
def last_presented(self):
923923
""" returns related SessionPresentation objects for the most recent meeting in the past"""
924924
# Assumes no two meetings have the same start date - if the assumption is violated, one will be chosen arbitrariy
925-
candidate_presentations = self.sessionpresentation_set.filter(session__meeting__date__lte=datetime.date.today())
926-
candidate_meetings = set([p.session.meeting for p in candidate_presentations if p.session.meeting.end_date()<datetime.date.today()])
925+
today = date_today()
926+
candidate_presentations = self.sessionpresentation_set.filter(session__meeting__date__lte=today)
927+
candidate_meetings = set([p.session.meeting for p in candidate_presentations if p.session.meeting.end_date()<today])
927928
if candidate_meetings:
928929
mtg = sorted(list(candidate_meetings),key=lambda x:x.date,reverse=True)[0]
929930
return self.sessionpresentation_set.filter(session__meeting=mtg)

ietf/doc/tests_ballot.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
from ietf.utils.test_utils import TestCase, login_testing_unauthorized
3232
from ietf.utils.mail import outbox, empty_outbox, get_payload_text
3333
from ietf.utils.text import unwrap
34+
from ietf.utils.timezone import date_today
3435

3536

3637
class EditPositionTests(TestCase):
@@ -899,7 +900,7 @@ def test_make_last_call_yang_document(self):
899900

900901
mailbox_before = len(outbox)
901902

902-
last_call_sent_date = datetime.date.today()
903+
last_call_sent_date = date_today()
903904
expire_date = last_call_sent_date+datetime.timedelta(days=14)
904905

905906
r = self.client.post(url,

ietf/doc/tests_review.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -899,7 +899,7 @@ def test_double_submit_review(self):
899899
assignment.review_request.team.acronym,
900900
assignment.review_request.type.slug,
901901
xslugify(assignment.reviewer.person.ascii_parts()[3]),
902-
datetime.date.today().isoformat(),
902+
date_today().isoformat(),
903903
]
904904
review_name = "-".join(c for c in name_components if c).lower()
905905
Document.objects.create(name=review_name,type_id='review',group=assignment.review_request.team)

ietf/doc/tests_utils.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
from ietf.doc.utils import (update_action_holders, add_state_change_event, update_documentauthors,
1717
fuzzy_find_documents, rebuild_reference_relations)
1818
from ietf.utils.draft import Draft, PlaintextDraft
19+
from ietf.utils.timezone import date_today
1920
from ietf.utils.xmldraft import XMLDraft
2021

2122

@@ -146,9 +147,10 @@ def test_update_action_holders_resets_age(self):
146147
dah.time_added = datetime.datetime(2020, 1, 1, tzinfo=datetime.timezone.utc) # arbitrary date in the past
147148
dah.save()
148149

149-
self.assertNotEqual(doc.documentactionholder_set.get(person=self.ad).time_added.date(), datetime.date.today())
150+
today = date_today()
151+
self.assertNotEqual(doc.documentactionholder_set.get(person=self.ad).time_added.date(), today)
150152
self.update_doc_state(doc, State.objects.get(slug='ad-eval'))
151-
self.assertEqual(doc.documentactionholder_set.get(person=self.ad).time_added.date(), datetime.date.today())
153+
self.assertEqual(doc.documentactionholder_set.get(person=self.ad).time_added.date(), today)
152154

153155
def test_update_action_holders_add_tag_need_rev(self):
154156
"""Adding need-rev tag adds authors as action holders"""

ietf/doc/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -732,7 +732,7 @@ def update_telechat(request, doc, by, new_telechat_date, new_returning_item=None
732732

733733
e.save()
734734

735-
has_short_fuse = doc.type_id=='draft' and new_telechat_date and (( new_telechat_date - datetime.date.today() ) < datetime.timedelta(days=13))
735+
has_short_fuse = doc.type_id=='draft' and new_telechat_date and (( new_telechat_date - date_today() ) < datetime.timedelta(days=13))
736736

737737
from ietf.doc.mails import email_update_telechat
738738

ietf/doc/utils_search.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@
1414
from ietf.doc.utils import augment_docs_and_user_with_user_info
1515
from ietf.meeting.models import SessionPresentation, Meeting, Session
1616
from ietf.review.utils import review_assignments_to_list_for_docs
17+
from ietf.utils.timezone import date_today
18+
1719

1820
def wrap_value(v):
1921
return lambda: v
@@ -34,8 +36,9 @@ def fill_in_telechat_date(docs, doc_dict=None, doc_ids=None):
3436
seen.add(e.doc_id)
3537

3638
def fill_in_document_sessions(docs, doc_dict, doc_ids):
37-
beg_date = datetime.date.today()-datetime.timedelta(days=7)
38-
end_date = datetime.date.today()+datetime.timedelta(days=30)
39+
today = date_today()
40+
beg_date = today-datetime.timedelta(days=7)
41+
end_date = today+datetime.timedelta(days=30)
3942
meetings = Meeting.objects.filter(date__gte=beg_date, date__lte=end_date).prefetch_related('session_set')
4043
# get sessions
4144
sessions = Session.objects.filter(meeting_id__in=[ m.id for m in meetings ])

ietf/doc/views_review.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -535,7 +535,7 @@ class CompleteReviewForm(forms.Form):
535535
review_url = forms.URLField(label="Link to message", required=False)
536536
review_file = forms.FileField(label="Text file to upload", required=False)
537537
review_content = forms.CharField(widget=forms.Textarea, required=False, strip=False)
538-
completion_date = DatepickerDateField(date_format="yyyy-mm-dd", picker_settings={ "autoclose": "1" }, initial=datetime.date.today, help_text="Date of announcement of the results of this review")
538+
completion_date = DatepickerDateField(date_format="yyyy-mm-dd", picker_settings={ "autoclose": "1" }, initial=date_today, help_text="Date of announcement of the results of this review")
539539
completion_time = forms.TimeField(widget=forms.HiddenInput, initial=datetime.time.min)
540540
cc = MultiEmailField(required=False, help_text="Email addresses to send to in addition to the review team list")
541541
email_ad = forms.BooleanField(label="Send extra email to the responsible AD suggesting early attention", required=False)
@@ -708,7 +708,7 @@ def complete_review(request, name, assignment_id=None, acronym=None):
708708
team.acronym,
709709
request_type.slug,
710710
xslugify(reviewer.person.ascii_parts()[3]),
711-
datetime.date.today().isoformat(),
711+
date_today().isoformat(),
712712
]
713713
review_name = "-".join(c for c in name_components if c).lower()
714714
if not Document.objects.filter(name=review_name).exists():

0 commit comments

Comments
 (0)