We are also getting this error since today - however we are still on version 5.6.59 and did not actively update anything.

Kindregards

@DenisChenu can you please look into this? It is caused by this commit https://github.com/LimeSurvey/LimeSurvey/pull/3786. I couldn't reproduce it on my local though, but already a couple users reported this issue.

Rolling back to 5.6.58 (same DB version) restored the normal behaviour.

@proveloict : can you check with https://github.com/LimeSurvey/LimeSurvey/pull/3820

@FabianLucas : try with replace the file models/Template.php with this one https://raw.githubusercontent.com/LimeSurvey/LimeSurvey/a413c130e65fa2ef2534c3cefd9cbdf7e6b78cd7/application/models/Template.php

@DenisChenu : I get the same error after upgrading again and applying the patch (https://github.com/LimeSurvey/LimeSurvey/pull/3820/files) manually.
(I downgraded again to keep the instance online).
Let me know if there is anything else I can do to help.

@proveloict : can you show a SQL result ?

SELECT template_name, lime_templates.folder FROM lime_template_configuration LEFT JOIN lime_templates ON lime_template_configuration.template_name = lime_templates.name WHERE sid IS NULL and gsid is NULL; ?

@DenisChenu

I had to adapt the command because I don't have a prefix on the tables. So i ran
SELECT template_name, templates.folder FROM template_configuration LEFT JOIN templates ON template_configuration.template_name =templates.name WHERE sid IS NULL and gsid is NULL;

And the result was this:

+-------------------------+-------------------------+
| template_name           | folder                  |
+-------------------------+-------------------------+
| bootswatch              | bootswatch              |
| fruity                  | fruity                  |
| vanilla                 | vanilla                 |
| Pro_Velo_extends_fruity | Pro_Velo_extends_fruity |
| material                | material                |
| vanilla_pro_velo        | vanilla_pro_velo        |
| theBP                   | theBP                   |
+-------------------------+-------------------------+
7 rows in set (0.000 sec)

And no material directory in ./theme/ right ?

The issue came from getTemplateListWithPreviews … i didn't update this (broken function)

I create the 2 lines … no way to reproduice …
I create a PR, but unsure if it work

@DenisChenu: indeed, no material direction in ./theme/

PR for 5.X : https://github.com/LimeSurvey/LimeSurvey/pull/3829
Based on https://github.com/LimeSurvey/LimeSurvey/pull/3820 (need both)

Please try with : https://github.com/LimeSurvey/LimeSurvey/blob/1795f45d732d81d5065f0bf59d42f132f2ea618b/application/models/Template.php

I have merged both PRs into 5.x branch.
@DenisChenu is there anything else to be done regarding this ticket?

Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36299

Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36298

Do the last one for 6/master
I do it now or tomorrow morning

master : https://github.com/LimeSurvey/LimeSurvey/pull/3833

For master : https://github.com/LimeSurvey/LimeSurvey/pull/3833

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36311

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36312

Fixed in Release 6.5.5+240429

@DenisChenu, I get the exact same "Invalid material template directory" error with Version 5.6.65+240522 when calling the theme editor at /index.php/themeOptions/index. I already saw that at 2 customer systems, one self hosted, one hosted at limequery.com.
Is that a known issue @tibor.pacalat?

@Mazi they should upgrade to 6.x. Support for 5.x ends: June 1st, 2024

@tibor.pacalat: That can not be the solution. I see quite a lot of people upgrading new. But there are many larger companies or univesrities which usually update pretty late e.g. because of using many custom themes (which have to be re-build).
People still need to be able to access the theme editor any time. We can't force them to update by adding such a bug ;-)

@DenisChenu can you please take a look?

Arg , i found the issue for Survey listing … but not found for Theme listing …

Let me check

This issue may only show up for customers having updated from an older version at which the Material theme was available.

Oh, i got it … it's new surely related to https://github.com/LimeSurvey/LimeSurvey/commit/82501703b34969c6601bffff507150ec2fccaf6c
and some other PR after :)

5.X : https://github.com/LimeSurvey/LimeSurvey/pull/3858
master : https://github.com/LimeSurvey/LimeSurvey/pull/3859

@tibor.pacalat: Can we integrate this important fix at the next 5.x release? We are no seeing this issue at multiple customer systems after having upgraded to 5.6.65.

@Mazi yes it will be a part of the next release.

Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36382

Fix committed to 5.x branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36381

Fixed in Release 6.5.11+240605

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36386