Posted by ramsalt on October 4, 2009 at 7:26pm
Jump to:
| Project: | Endless Page |
| Version: | 6.x-1.3 |
| Component: | Views Integration |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Issue Summary
In the function endless_page_create on line 169 you call node_load for each node loaded. This is extremely expensive considering another node_load is called on line 200 for the same nid. This slows down the performance a great deal. Just wanted to point this out.
To be able to use this I had to set the if check on line 169 to true and handle this kind of security with views.
Comments
#1
Multiple calls to node_load are cached so I'm not sure that having multiple calls is a problem, per se.
Calling node_load at all seems like a necessary part of the module, though, in order to render and display the nodes, so I'm tempted to mark this "by design".