search

csrdelft / csrdelft.nl

De webstek van Civitas Studiosorum Reformatorum, wordt onderhouden door de PubCie.
https://csrdelft.nl
17 stars 11 forks source link

Voorpagina werkt niet als fotoalbum map niet bestaat #1102

Closed pmvz closed 1 year ago

pmvz commented 1 year ago

De voorpagina wil niet laden omdat csrdelft_voorpagina_fotoalbum niet gevonden wordt. Geen flauw idee wat er precies mis gaat, want PhpStorm heeft wel een volgbaar linkje. Als regel 16 van voorpagina.html.twig wordt weggelaten laadt de voorpagina wel, met een foutmelding dat voorpagina_posters ook niet wordt gevonden.

qurben commented 1 year ago

csrdelft_voorpagina_fotoalbum is een route in VoorpaginaController en het lijkt er inderdaad op dat dit faalt als de voorpagina_posters cms pagina mist (oftewel in de Fixtures).

pmvz commented 1 year ago

Hoe kan ik dit fixen?

qurben commented 1 year ago

Oh ik keek verkeerd, hij faalt omdat de map data/foto/fotoalbum niet bestaat.

qurben commented 1 year ago

Wordt al afgevangen in de tests. Het zou beter zijn als dit niet nodig is tho. https://github.com/csrdelft/csrdelft.nl/blob/6bc1f1829320b86dfa98671aacb9e1d71f5b5852/tests/lib/BrowserTestCase.php#L85-L88