Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#19971 closed Uncategorized (fixed)

Spelling mistake of explicitly (explictly)

Reported by: tobias.carlander@… Owned by: Tobias Carlander
Component: Documentation Version: 1.5
Severity: Normal Keywords: explicitly, explictly
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no


On page /en/1.5/topics/class-based-views/generic-display/#adding-extra-context
In the Note:

"Generally, get_context_data will merge the context data of all parent classes with those of the current class. To preserve this behavior in your own classes where you want to alter the context, you should be sure to call get_context_data on the super class. When no two classes try to define the same key, this will give the expected results. However if any class attempts to override a key after parent classes have set it (after the call to super), any children of that class will also need to explictly set it after super if they want to be sure to override all parents. If you’re having trouble, review the method resolution order of your view."

I have bolded the error, it should read explicitly

Change History (5)

comment:1 by tobias.carlander@…, 11 years ago

Owner: changed from nobody to anonymous
Status: newassigned

I have created Pull Request 877 for this,
My first try to help, hope I got it right.

comment:2 by Tobias Carlander, 11 years ago

Owner: changed from anonymous to Tobias Carlander

comment:3 by Tobias Carlander, 11 years ago

Has patch: set

comment:4 by Tobias Carlander <tobias.carlander@…>, 11 years ago

Resolution: fixed
Status: assignedclosed

In df7668a9e47100bc2ac58a3b0fec4665d4aadb48:

Fix Typo explicitly. Fixes #19971

comment:5 by Aymeric Augustin <aymeric.augustin@…>, 11 years ago

In 2047865964e534646f7fe15165785bfbca892b9d:

Merge pull request #877 from tcarlander/master

Fixed #19971 -- Typo in CBV docs.

Note: See TracTickets for help on using tickets.
Back to Top