Skip to content
This repository has been archived by the owner on Apr 8, 2023. It is now read-only.

refactor FormValidException #40

Closed
Nef10 opened this issue Nov 17, 2014 · 0 comments
Closed

refactor FormValidException #40

Nef10 opened this issue Nov 17, 2014 · 0 comments
Labels
[P] minor

Comments

@Nef10
Copy link
Contributor

Nef10 commented Nov 17, 2014

If everything is ok, this should not throw an exception (in information_pages/view.py).

Ideas:

  • handle_edit could return None on success
  • handle_edit could throw exception if the form does not validate, but therefore you have to put the context in the exception
l--f pushed a commit to l--f/1327 that referenced this issue Nov 17, 2014
fixed fsr-de#40, also corrects the redirect when the title of a docum…
030d273 
…ent is changed
l--f pushed a commit to l--f/1327 that referenced this issue Nov 17, 2014
fixed fsr-de#40, also corrects the redirect when the title of a docum…
56c55a6 
…ent is changed
@invliD invliD closed this as completed in 3e7901a Nov 17, 2014
invliD added a commit that referenced this issue Nov 17, 2014
@invliD
Merge pull request #45 from l--f/fix#40
942b0b8 
fixed #40, also corrects the redirect for changing title of a document
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
[P] minor
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Nef10