contao / official-demo

Contao Official Demo
17 stars 15 forks source link

FAQ bringt Error #109

Closed berecont closed 2 years ago

berecont commented 2 years ago

FAQ > How FAQs work > Fragen Beim Anlegen und/oder Ändern einer (neuen) Frage gibt's einen Internal Server Error: Invalid jump To page: 35

cliffparnitzky commented 2 years ago

@leofeyer This is only a probleme in the system "https://demo.contao.org/" ... The last demo code is fine.

But maybe this is also an error (tl_faq.php line 596) which should be fixed. What is your suggestion what we should do?

fritzmg commented 2 years ago

If this is a bug in Contao and reproducible outside the official demo, then you should open an issue in contao/contao.

cliffparnitzky commented 2 years ago

It is a thrown exception when creating the serp preview and the jumpTo page is null ... so, it seems, someone wanted to pay attention ... maybe it is not a bug, but the behaviour should be improved.

fritzmg commented 2 years ago

If this is reproducible in a vanilla Contao instance or even in the Contao Online Demo on https://demo.contao.org/, please report it as a bug then (@berecont or @cliffparnitzky).

berecont commented 2 years ago

just testet ... in a vanilla contao installation it works. in the online demo it don't work Contao 4.13.6 php81 local installation with laragon

fritzmg commented 2 years ago

just testet ... in a vanilla contao installation it works.

How did you test? Are you sure you did not configure a redirect page in your FAQ archive?

berecont commented 2 years ago

FAQ-Archive with the same datas as Online Demo. CE_Module with 'faq-page' and my FAQ-Archive

fritzmg commented 2 years ago

Yes, the issue is that the demo database references an invalid page ID in tl_faq_category.jumpTo. This needs to be fixed in the demo database, not in Contao.

cliffparnitzky commented 2 years ago

It is reproducable in a vanilla contao installation:

fritzmg commented 2 years ago

@cliffparnitzky you can open an issue on contao/contao with that reproduction.

cliffparnitzky commented 2 years ago

@fritzmg done: https://github.com/contao/contao/issues/4990

leofeyer commented 2 years ago

I have fixed the demo now.