Skip to content

Commit

Permalink
Merge pull request #249 from kevinlondon/patch-1
Browse files Browse the repository at this point in the history
Fixed performance doc spelling / grammar
  • Loading branch information
lukaszb committed Oct 14, 2014
2 parents 1d2a524 + 83bc40d commit cb15548
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions docs/userguide/performance.rst
Expand Up @@ -4,7 +4,7 @@ Performance tuning
===================

It is important to remember that by default ``django-guardian`` uses generic
foreign keys to retain relation with any Django model. For most cases it's
foreign keys to retain relation with any Django model. For most cases, it's
probably good enough, however if we have a lot of queries being spanned and
our database seems to be choking it might be a good choice to use *direct*
foreign keys. Let's start with quick overview of how generic solution work and
Expand Down Expand Up @@ -60,8 +60,8 @@ Something similar to:
As there are no real foreing keys pointing at the target model this solution
might not be enough for all cases. In example if we try to build an issues
As there are no real foreign keys pointing at the target model, this solution
might not be enough for all cases. For example, if we try to build an issues
tracking service and we'd like to be able to support thousands of users and
their project/tickets, object level permission checks can be slow with this
generic solution.
Expand Down Expand Up @@ -99,13 +99,13 @@ models:
``content_object`` as underlying queries depends on it.


from now on ``guardian`` will figure out that ``Project`` model has direct
From now on, ``guardian`` will figure out that ``Project`` model has direct
relation for user/group object permissions and will use those models. It is
also possible to use only user or only group based direct relation, however it
also possible to use only user or only group-based direct relation, however it
is discouraged (it's not consistent and might be a quick road to hell from the
mainteinence point of view, especially).
maintainence point of view, especially).

.. note::
By defining direct relation models we can also tweak that object permission
model, i.e. by adding some fields
model, i.e. by adding some fields.

0 comments on commit cb15548

Please sign in to comment.