madeyourday / contao-rocksolid-slider

RockSolid Slider Contao Extension
http://rocksolidthemes.com/de/contao/plugins/responsive-slider
Other
9 stars 7 forks source link

Contao 5.3.11 + Rocksolid Slider 2.2.5 > Module error #92

Closed biowebfr closed 3 months ago

biowebfr commented 4 months ago

Hi, I tried the slider today on the lastest contao and I get this error when creating/editing a module :

image

Thank you for your time.

ausi commented 3 months ago

I was not able to reproduce that error. Where exactly does this happen?

ausi commented 3 months ago

What language are you using in the backend?

ausi commented 3 months ago

Can you please post the whole stacktrace (or a screenshot of it)?

biowebfr commented 3 months ago

Sorry for the lack of details.

I did some test, so :

It happens when I create or edit a slider (module or content) in APP_ENV=dev.

image image

It's French in BE.

Thank you.

ausi commented 3 months ago

I was still not able to reproduce it.

Are there any other extensions installed?

Does the error also happen if you switch the language in the Backend to English?

Do you have any custom translations files?
Something like contao/languages/fr/tl_module.php for example?

biowebfr commented 3 months ago

Yes there are some extensions installed

It happens in English too.

I don't have custom translations.

Do you think it could help if I give you a BE access to my test environnement? Can we talk in private?

biowebfr commented 3 months ago

Ok I made some investigations and I found that it happens when this extension is installed. https://github.com/codefog/contao-news_categories/

I don't understand what relation is there between them.

ausi commented 3 months ago

The issue was caused by that line: https://github.com/codefog/contao-news_categories/blob/f27222b3215079ca637a97735f4785dc3532d4a6/contao/languages/en/tl_module.php#L30

However the bug was still that the slider extension used the wrong label reference for the rsts_thumbs_imgSize field.

This is fixed now in version 2.2.6. So after an update the issue should disappear.

biowebfr commented 3 months ago

Thank you.