-
-
Save VincentLoy/e693aa1f149a59f465a5a71b6d937b9a to your computer and use it in GitHub Desktop.
Thanks for the help! :)
Glad to be helpful!
Possibly I made a mistake in my PR... I'll have to revisit.
Maybe, it is my confusion. Just to make it clear, I do not used/tested routed pages, I just implemented your method to send context in a Page. Let's say there is a Blog Index Page, and I show breadcrumb Homepage>Blog Index.
@ismayil-ismayilov To be clear, you're not going from your browser to the page? IE the page.route
method does not get invoked?
Sorry about confusion, it actually hits one time no matter the depth in my case. I am not sure what may cause such difference. It has to get > "parent" data from somewhere, either it has to be prefetched already and cached or it has to be requested.
How it would else be possible not to hit DB using get_ancestors()?
Get ancestors will for now always perform a query - it is not cached.
The parent objects however are cached upon routing to that page.
IE.
Homepage
> BlogPage
> > BlogIndexPage
You should be able to travel from BlogIndexPage
to Homepage
using get_parent
without it performing any queries (inside the serve method of BlogIndexPage
, after page.route
has been invoked by the page serve view)
Can you tell me if this is the case for you?
Unfortunately, I do not think that's a case for me. I strongly believe I caused a confusion since I may not have the same application method as you refer, sorry about it.
I used your Wagtail repo. It still hits DB.
This is my "simplified" Page class. I basically, applied your method to create a breadcrumb. Nothing more or less.
Hmm - very odd. No - it really shouldn't hit the DB, no matter the depth...
Possibly I made a mistake in my PR... I'll have to revisit.
I'll submit a fix - my bad. In the future you should then have zero query breadcrumbs - I think I made a mistake not setting the cache on the specific instance of the page.
Thanks for the help! :)
[EDIT]
@ismayil-ismayilov would you mind trying this repo to see if it still queries?
https://github.com/Nigel2392/wagtail/tree/update_page_routing
can be done with
pip install git+https://github.com/Nigel2392/wagtail.git@update_page_routing