Default number of results per page: gives out of memory

The pager query in the theme function theme_autocomplete_node_finder_results_page sucks and works like this

say you have these 6 result nids: 35, 45, 55, 65, 75, 85

and you have a pager with 3 per page

it actually does this

pager_query("SELECT * from {node} WHERE nid IN (35, 45, 55, 65, 75, 85)", 3, $delta, $count_sql);

so if you have a lot of possible results it will just keep stringing them on into that query

I am stupid - there is no reason to even do a query here, since we know what the results are, we could just pull the right range of results out of the array.

I will get back to you on this.

This has been fixed in the newest versions!

I tried the latest version for D5 -> 5.x-3.1 but it does not work, maybe only fixed for the 6.0+ versions?

err... do you mean 3.3 ??

Feel free to reopen this if you are still experiencing this problem with the latest releases. Just to clarify the releases 5.x-3.1 and 5.x-3.3 are compatible versions. The 5.x means it will work on Drupal 5, and the -3 means it is the same branch of code. The .1 and .3 are the different revisions, .1 was the buggy version, .3 has the fix - so it's 5.x-3.3 that you'll need.

Hello,

I'm opening this again, because i'm experiencing a similar problem, just the message is different:

Fatal error: Allowed memory size of 67108864 bytes exhausted (tried to allocate 4194304 bytes) in C:\xampp\htdocs\drupal5\sites\all\modules\autocomplete_node_finder\autocomplete_node_finder.module on line 1186

As the OP said, this happens when we change "Default number of results per page" to other than zero (the default).

Version 5.3.4 (latest so far).

OK I will do some testing of the results pages soon.

It seems a recent patch was done against the wrong revision causing these problems to come back. Please try latest version.

Daniel, you're on a roll!

I can confirm that the error message doesnt come up anymore on the new v5.3.7 .