search

d4l-data4life / kirby3-static-site-generator

Static site generator plugin for Kirby 3+. With this plugin you can create a directory with assets, media and static html files generated from your pages. The result is an even faster site with less potential vulnerabilities.
MIT License
139 stars 7 forks source link

Allowed memory size of 134217728 bytes exhausted #77

Closed Anjade closed 1 year ago

Anjade commented 1 year ago

Looks like the memory is not enough. Or maybe the programming is not perfect and the building must be done in parts?

The allowed memory size of 134217728 bytes was exhausted (tried to allocate 20480 bytes)

Anjade commented 1 year ago

php_value memory_limit 256M

helps, but the question is still, if the plugin walks through all pages in one go?

Anjade commented 1 year ago

Meanwhile, I have 500 pages in two languages; the memory needs now 512MB; I got again an error with 256MB.

Anjade commented 1 year ago

I try now: php_value memory_limit 1024M php_value max_execution_time 180

Maybe it will help.

Maybe we can divide the build somehow, please? Maybe per top-level-pages; like: build all, build 1= x, y, z and 2 = a, b, c ... to have more than one builds for more than one parts of the system?

jonathan-reisdorf commented 1 year ago

This was never a problem up until latest version of Kirby. With latest version of Kirby I now get this error, too. Will debug, my suspicion is that there is a new infinite recursion issue

jonathan-reisdorf commented 1 year ago

The issues have been introduced with the latest changes regarding supporting kirby paginations. With version 1.10.0 I have now reverted that code and deprecated support for Kirby paginations. Memory issues no longer appear. For your pagination use case you would have to create your own pagination url method instead of using Kirby's pagination method.