Fixed a number of places where events were given the document time, but the document had not been given a new time yet, after the eventsave refactoring code changes of [11841]. Fixes issue #2004.
- Legacy-Id: 11901
Note: SVN reference [11841] has been migrated to Git commit d868371aff
This commit is contained in:
parent
c60434950f
commit
f97204ceee
|
@ -1278,7 +1278,7 @@ def adopt_draft(request, name):
|
|||
|
||||
# stream
|
||||
if doc.stream != new_stream:
|
||||
e = DocEvent(type="changed_stream", time=doc.time, by=by, doc=doc)
|
||||
e = DocEvent(type="changed_stream", by=by, doc=doc)
|
||||
e.desc = u"Changed stream to <b>%s</b>" % new_stream.name
|
||||
if doc.stream:
|
||||
e.desc += u" from %s" % doc.stream.name
|
||||
|
@ -1291,7 +1291,7 @@ def adopt_draft(request, name):
|
|||
|
||||
# group
|
||||
if group != doc.group:
|
||||
e = DocEvent(type="changed_group", time=doc.time, by=by, doc=doc)
|
||||
e = DocEvent(type="changed_group", by=by, doc=doc)
|
||||
e.desc = u"Changed group to <b>%s (%s)</b>" % (group.name, group.acronym.upper())
|
||||
if doc.group.type_id != "individ":
|
||||
e.desc += " from %s (%s)" % (doc.group.name, doc.group.acronym.upper())
|
||||
|
@ -1300,7 +1300,7 @@ def adopt_draft(request, name):
|
|||
doc.group = group
|
||||
|
||||
new_notify = get_initial_notify(doc,extra=doc.notify)
|
||||
events.append(make_notify_changed_event(request, doc, by, new_notify, doc.time))
|
||||
events.append(make_notify_changed_event(request, doc, by, new_notify))
|
||||
doc.notify = new_notify
|
||||
|
||||
comment = form.cleaned_data["comment"].strip()
|
||||
|
@ -1309,7 +1309,7 @@ def adopt_draft(request, name):
|
|||
prev_state = doc.get_state("draft-stream-%s" % doc.stream_id)
|
||||
if new_state != prev_state:
|
||||
doc.set_state(new_state)
|
||||
e = add_state_change_event(doc, by, prev_state, new_state, timestamp=doc.time)
|
||||
e = add_state_change_event(doc, by, prev_state, new_state)
|
||||
events.append(e)
|
||||
|
||||
due_date = None
|
||||
|
@ -1322,7 +1322,7 @@ def adopt_draft(request, name):
|
|||
|
||||
# comment
|
||||
if comment:
|
||||
e = DocEvent(type="added_comment", time=doc.time, by=by, doc=doc)
|
||||
e = DocEvent(type="added_comment", by=by, doc=doc)
|
||||
e.desc = comment
|
||||
e.save()
|
||||
events.append(e)
|
||||
|
@ -1427,7 +1427,7 @@ def change_stream_state(request, name, state_type):
|
|||
new_state = form.cleaned_data["new_state"]
|
||||
if new_state != prev_state:
|
||||
doc.set_state(new_state)
|
||||
e = add_state_change_event(doc, by, prev_state, new_state, timestamp=doc.time)
|
||||
e = add_state_change_event(doc, by, prev_state, new_state)
|
||||
events.append(e)
|
||||
|
||||
due_date = None
|
||||
|
@ -1445,7 +1445,7 @@ def change_stream_state(request, name, state_type):
|
|||
if existing_tags != new_tags:
|
||||
doc.tags = new_tags
|
||||
|
||||
e = DocEvent(type="changed_document", time=doc.time, by=by, doc=doc)
|
||||
e = DocEvent(type="changed_document", by=by, doc=doc)
|
||||
added_tags = new_tags - existing_tags
|
||||
removed_tags = existing_tags - new_tags
|
||||
l = []
|
||||
|
@ -1461,7 +1461,7 @@ def change_stream_state(request, name, state_type):
|
|||
|
||||
# comment
|
||||
if comment:
|
||||
e = DocEvent(type="added_comment", time=doc.time, by=by, doc=doc)
|
||||
e = DocEvent(type="added_comment", by=by, doc=doc)
|
||||
e.desc = comment
|
||||
e.save()
|
||||
events.append(e)
|
||||
|
|
|
@ -145,7 +145,6 @@ def edit_material(request, name=None, acronym=None, action=None, doc_type=None):
|
|||
|
||||
if prev_rev != doc.rev:
|
||||
e = NewRevisionDocEvent(type="new_revision", doc=doc, rev=doc.rev)
|
||||
e.time = doc.time
|
||||
e.by = request.user.person
|
||||
e.desc = "New version available: <b>%s-%s</b>" % (doc.name, doc.rev)
|
||||
e.save()
|
||||
|
@ -156,7 +155,6 @@ def edit_material(request, name=None, acronym=None, action=None, doc_type=None):
|
|||
e.desc = u"Changed title to <b>%s</b>" % doc.title
|
||||
if prev_title:
|
||||
e.desc += u" from %s" % prev_title
|
||||
e.time = doc.time
|
||||
e.save()
|
||||
events.append(e)
|
||||
|
||||
|
|
|
@ -1143,7 +1143,7 @@ def upload_session_bluesheets(request, session_id, num):
|
|||
session.sessionpresentation_set.create(document=doc,rev='00')
|
||||
filename = '%s-%s%s'% ( doc.name, doc.rev, ext)
|
||||
doc.external_url = filename
|
||||
e = NewRevisionDocEvent.objects.create(doc=doc,time=doc.time,by=Person.objects.get(name='(System)'),type='new_revision',desc='New revision available: %s'%doc.rev,rev=doc.rev)
|
||||
e = NewRevisionDocEvent.objects.create(doc=doc, by=Person.objects.get(name='(System)'),type='new_revision',desc='New revision available: %s'%doc.rev,rev=doc.rev)
|
||||
doc.save_with_history([e])
|
||||
handle_upload_file(file, filename, session.meeting, 'bluesheets')
|
||||
return redirect('ietf.meeting.views.session_details',num=num,acronym=session.group.acronym)
|
||||
|
|
Loading…
Reference in a new issue