Website has become extremely slow

[tires]

I have a site with about 5000 subpages and have noticed that it now takes about 10 seconds to save a page.

What could be the reason for this and how can I speed up the site?

[AndZyk]

Hello @tires,

have you tried looking in TracyDebugger if there is an error?
Do you use hooks or special modules?

The amount of subpages should not be an issue in my experience.
Maybe there is something else going on.

Regards, Andreas

[tires]

Thank you very much for your advice!

I have indeed found a hook that was apparently to blame. I can't remember exactly why I added it and what it does ...

$wire->addHookAfter('Pages::saveReady', function($event) {
  $event->modules->get('SearchEngine')->indexPages();
  $page = $event->arguments(0);
  $event->wire('log')->save('Page saved', "Page ID: $page->id / Page Name: $page->name / Page Parents: $page->parents");
});

 

[da²]

  On 12/18/2024 at 11:54 AM, tires said:

$event->modules->get('SearchEngine')->indexPages();

Expand  

You are indexing all the site pages every time a page is saved. You should index only this page since the work for other ones was already done.

[AndZyk]

Happy to hear you found the issue. 😀

You could consider indexing the pages with a Lazy Cron or a cronjob.

[da²]

Solution is here: https://processwire.com/modules/search-engine/

// Alternatively index just a single page (passing in a Page object):
$modules->get('SearchEngine')->indexPage($page);

 

[tires]

  On 12/18/2024 at 3:23 PM, da² said:

Solution is here: https://processwire.com/modules/search-engine/

// Alternatively index just a single page (passing in a Page object):
$modules->get('SearchEngine')->indexPage($page);

 

Expand  

I can't really remember why I used this module (it's been a few years).
Do I really need it to find the content of the pages?

You mean I should simply adapt the ready.php to, right?

$wire->addHookAfter('Pages::saveReady', function($event) {
  $event->modules->get('SearchEngine')->indexPages($page);
  $page = $event->arguments(0);
  $event->wire('log')->save('Page saved', "Page ID: $page->id / Page Name: $page->name / Page Parents: $page->parents");
});

Thank a lot!

[da²]

  On 12/19/2024 at 7:13 PM, tires said:

Do I really need it to find the content of the pages?

Expand  

If you have a search page that allows to search text in a lot of pages containing big texts, you probably need it. Do a performance benchmark with and without.

  On 12/19/2024 at 7:13 PM, tires said:

You mean I should simply adapt the ready.php to, right?

Expand  

Yes but without the mistake you did on the method name. 😁 

Edited December 20, 2024 by da²

[tires]

  On 12/20/2024 at 2:02 AM, da² said:

Yes but without the mistake you did on the method name. 😁 

Expand  

Which mistake do you mean? 🤓

By the way, do I need this hook at all or does the module automatically index the edited page when it is saved?

[da²]

  On 12/21/2024 at 5:31 PM, tires said:

Which mistake do you mean? 🤓

Expand  

I won't help for reading a function name. 😆

  On 12/21/2024 at 5:31 PM, tires said:

By the way, do I need this hook at all or does the module automatically index the edited page when it is saved?

Expand  

I don't know the module, check the documentation.

[tires]

  On 12/21/2024 at 5:34 PM, da² said:

I won't help for reading a function name. 😆

Expand  

Please, come on!
😆

Really, i don't understand what you mean right now.

[zoeck]

  On 1/7/2025 at 10:43 PM, tires said:

Really, i don't understand what you mean right now.

Expand  

In your hook posted above you use “indexPages()”, but the correct one would be “indexPage($page)” without the s at the end.

Your hook creates a completely new search index each time, not only for the changed page but for all pages.

You have to use something like this (not tested)

$wire->addHookAfter('Pages::saveReady', function($event) {
  $page = $event->arguments(0);
  $event->modules->get('SearchEngine')->indexPage($page);
  $event->wire('log')->save('Page saved', "Page ID: $page->id / Page Name: $page->name / Page Parents: $page->parents");
});

 

[teppo]

  On 12/21/2024 at 5:31 PM, tires said:

does the module automatically index the edited page when it is saved?

Expand  

Yes, it does.

As others have pointed out: if you only want to index a single page, indexPage($page) is the correct method — but I also have no idea what the purpose of that would be. My guess is that it was either added due to some kind of misunderstanding, or perhaps you indeed did want to recreate the full index every time a page is saved. If it was the latter reason then that might make some sense, but it is also a very bad idea in terms of performance (as you've noted here) 🙂

[tires]

  On 1/8/2025 at 6:57 AM, zoeck said:

In your hook posted above you use “indexPages()”, but the correct one would be “indexPage($page)” without the s at the end.

Your hook creates a completely new search index each time, not only for the changed page but for all pages.

You have to use something like this (not tested)

$wire->addHookAfter('Pages::saveReady', function($event) {
  $page = $event->arguments(0);
  $event->modules->get('SearchEngine')->indexPage($page);
  $event->wire('log')->save('Page saved', "Page ID: $page->id / Page Name: $page->name / Page Parents: $page->parents");
});

 

Expand  

Thank you very much.
I have now simply removed this hook.