Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Docs: fix docs/extending_cms/extending_examples.rst #961

Merged
merged 3 commits into from Aug 25, 2011

Conversation

Projects
None yet
3 participants
Contributor

applecat commented Aug 19, 2011

No description provided.

@kezabelle kezabelle and 1 other commented on an outdated diff Aug 24, 2011

docs/extending_cms/app_integration.rst
@@ -282,7 +282,13 @@ We would now create a menu out of these categories::
def get_nodes(self, request):
nodes = []
for category in Category.objects.all().order_by("tree_id", "lft"):
- nodes.append(NavigationNode(category.name, category.pk, category.parent_id))
+ nodes = NavigationNode(
@kezabelle

kezabelle Aug 24, 2011

Contributor

There seems to be an error here that you've carried over: the variable is called nodes so in each iteration it'll override the declaration of nodes as a list, and will never append correctly.

@applecat

applecat Aug 25, 2011

Contributor

thanx. I fixed it

ojii added a commit that referenced this pull request Aug 25, 2011

Merge pull request #961 from applecat/develop
Docs: fix docs/extending_cms/extending_examples.rst

@ojii ojii merged commit 12fd36d into divio:develop Aug 25, 2011

Collaborator

ojii commented Aug 25, 2011

thanks a lot

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment