Skip to content

Commit

Permalink
crazy
Browse files Browse the repository at this point in the history
  • Loading branch information
pydanny committed Sep 22, 2013
1 parent 9d63af5 commit d5b61ed
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions DjangoConEurope2013/CBVs.rst
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ Problems with function based generic views
* No control over logic flow
* No re-use between views

.. note:: Another thing against function-based generic views was that people would implement their own broken CBV system.
.. warning:: Another thing against function-based generic views was that people can and do implement their own broken CBV system. Leaky states is a serious issue.

Django 1.3
-----------
Expand Down Expand Up @@ -100,4 +100,14 @@ Implementation Choices
* Doesn't have HTTP Verb support
* Suffers from serious state issues

.. warning:: Don't put ``self`` on Django admin objects or you will cause state issues.
.. warning:: Don't put ``self`` on Django admin objects or you will cause state issues.

Other concept design:

* Change the ``urls.py`` contract

* Current: a callable
* Change to: A callable or a class
* The problem is that this would have forced them to change a lot of code and make things under the hood much more complex.

* Decision: keep the ``urls.py`` contract clear

0 comments on commit d5b61ed

Please sign in to comment.