#3790 - Critical error when viewing cms:cms-comcode-pages

This is a spacer post for a website comment topic. The content this topic relates to: #3790 - Critical error when viewing cms:cms-comcode-pages
Since the successful upgrade to v10.0.24, the error is still the same, only now it's on line 1,103.

Fatal error: Call to a member function is_empty() on null in /home/bosslife/public_html/sources/comcode_renderer.php on line 1103
I think you have a loop with your Comcode [include] tags on a page.

Try this change...

diff --git a/sources/site.php b/sources/site.php
index ef7dd34a0..6872415ed 100644
--- a/sources/site.php
+++ b/sources/site.php
@@ -1175,7 +1175,7 @@ function request_page($codename, $required, $zone = null, $page_type = null, $be
if ($REQUEST_PAGE_NEST_LEVEL > 20) {
$REQUEST_PAGE_NEST_LEVEL = 0;
attach_message(do_lang_tempcode('STOPPED_RECURSIVE_RESOURCE_INCLUDE', escape_html($codename), escape_html(do_lang('PAGE'))), 'warn');
- return null;
+ return new Tempcode();
}

// Run hooks, if any exist
I have no idea.

I made the change and it seemed to have worked. But I was curious and reverted your change to the original code, and it's still working. Only thing I did since was add a new page (I had to type in the correct parameters manually into the URL as the cms-comcode-pages wasn't accessible).

I can tell you, I'm 99% sure it doesn't involve an include...I usually don't include pages within pages and haven't used that tag in quite a bit. The only thing I do is include templates within templates.

I'll keep you posted if I have more issues but as for now it's working.
The Comcode page cache had been cleared I think, meaning it had to rebuild it from all the pages. Any of those pages could have had an include. After the fix the cache got rebuilt, so the fix was no longer needed.
Main issue fixed.
Dry-run getting stuck on once selected fixed.

I couldn't reproduce or trace an issue related to the upgrade option being missing. There's a 3 hour cache on the upgrade block, and a 5 minute cache on the upgrade info call, but neither should be getting in the way of a version older than 3 hours. If it can't connect to the compo.sr web service it should say so, but I can foresee that not working right in some situations, and our server has been having performance issues (bot attack) recently, so perhaps that is it.
0 guests and 0 members have recently viewed this.