diff options
author | Bryan Newbold <bnewbold@robocracy.org> | 2019-06-13 12:53:54 -0700 |
---|---|---|
committer | Bryan Newbold <bnewbold@robocracy.org> | 2019-06-13 12:53:54 -0700 |
commit | 95286700bbe06c8c0676e08acc0b6dc4a8f8504d (patch) | |
tree | b18a383685765e7cf4262a25896fe22cc8f5e566 | |
parent | ec7bce74d501df3b2514baab6ec4685baf126818 (diff) | |
download | fatcat-95286700bbe06c8c0676e08acc0b6dc4a8f8504d.tar.gz fatcat-95286700bbe06c8c0676e08acc0b6dc4a8f8504d.zip |
get rid of active_editgroup_id cookie
-rw-r--r-- | python/fatcat_web/editing_routes.py | 18 | ||||
-rw-r--r-- | python/fatcat_web/routes.py | 3 |
2 files changed, 6 insertions, 15 deletions
diff --git a/python/fatcat_web/editing_routes.py b/python/fatcat_web/editing_routes.py index fb45ca7b..c8cebd62 100644 --- a/python/fatcat_web/editing_routes.py +++ b/python/fatcat_web/editing_routes.py @@ -43,7 +43,6 @@ def form_editgroup_get_or_create(api, edit_form): app.log.warning(ae) abort(ae.status) # set this session editgroup_id - session['active_editgroup_id'] = eg.editgroup_id flash('Started new editgroup <a href="/editgroup/{}">{}</a>' \ .format(eg.editgroup_id, eg.editgroup_id)) return eg @@ -168,12 +167,15 @@ def generic_entity_edit(editgroup_id, entity_type, existing_ident, edit_template else: raise NotImplementedError - if not editgroup_id: - form.editgroup_id.data = session.get('active_editgroup_id', None) - editor_editgroups = api.get_editor_editgroups(session['editor']['editor_id'], limit=20) potential_editgroups = [e for e in editor_editgroups if e.changelog_index == None and e.submitted == None] + if not form.is_submitted(): + # default to most recent not submitted, fallback to "create new" + form.editgroup_id.data = "" + if potential_editgroups: + form.editgroup_id.data = potential_editgroups[0].editgroup_id + return render_template(edit_template, form=form, existing_ident=existing_ident, editgroup=editgroup, potential_editgroups=potential_editgroups), status @@ -260,8 +262,6 @@ def file_create(): status = 400 app.log.info("form errors (did not validate): {}".format(form.errors)) else: - editgroup_id = session.get('active_editgroup_id', None) - form.editgroup_id.data = editgroup_id form.urls.append_entry() form.release_ids.append_entry() return render_template('file_create.html', form=form), status @@ -299,8 +299,6 @@ def file_edit(ident): app.log.info("form errors (did not validate): {}".format(form.errors)) else: # not submitted form = FileEntityForm.from_entity(entity) - editgroup_id = session.get('active_editgroup_id', None) - form.editgroup_id.data = editgroup_id return render_template('file_edit.html', form=form, entity=entity), status @app.route('/fileset/<ident>/edit', methods=['GET']) @@ -354,8 +352,6 @@ def release_create(): app.log.info("form errors (did not validate): {}".format(form.errors)) else: # not submitted form.contribs.append_entry() - editgroup_id = session.get('active_editgroup_id', None) - form.editgroup_id.data = editgroup_id return render_template('release_create.html', form=form), status @app.route('/release/<ident>/edit', methods=['GET', 'POST']) @@ -391,8 +387,6 @@ def release_edit(ident): app.log.info("form errors (did not validate): {}".format(form.errors)) else: # not submitted form = ReleaseEntityForm.from_entity(entity) - editgroup_id = session.get('active_editgroup_id', None) - form.editgroup_id.data = editgroup_id return render_template('release_edit.html', form=form, entity=entity), status @app.route('/work/create', methods=['GET']) diff --git a/python/fatcat_web/routes.py b/python/fatcat_web/routes.py index 66f64ca6..6211b90c 100644 --- a/python/fatcat_web/routes.py +++ b/python/fatcat_web/routes.py @@ -403,9 +403,6 @@ def editgroup_accept(ident): except ApiException as ae: app.log.info(ae) abort(ae.status) - # clear active_editgroup_id cookie; this doesn't cover all cases - if eg.editgroup_id == session.get('active_editgroup_id'): - session.pop('active_editgroup_id') return redirect('/editgroup/{}'.format(ident)) @app.route('/editgroup/<ident>/unsubmit', methods=['POST']) |