Twill: Settings fail to save when only one language is available

Created on 26 Oct 2018 · 4Comments · Source: area17/twill

Hello, thanks for such an amazing admin package, I absolutely love Twill!

I'm having the following issue: I'm building a CMS for a single language website (english). I enabled the Settings option in the config/twill.php file, I copied the settings migration to my migrations folder as suggested in the documentation but removed the translations table as I will not use it. My "translatable" locales settings array only contains the "en" code so no other languages are available. The problem comes when I try to store some settings as I get the following error:

ErrorException (E_WARNING) Illegal string offset 'en' 66 ErrorException …/vendor/area17/twill/src/Repositories/SettingRepository.php45

I've tried adding the "translated=false" attribute on the form fields but I still get the same error.

I removed the tables and re-executed the migrations but this time leaving the translations table, I enabled the translated=true attribute on the form fields and it worked correctly, however I can't store any images so it does not solve my problem.

Any help would be much appreciated.

Thank you!

working on it bug enhancement

Source

manuelsofia

All 4 comments

Hi @manuelsofia,

thanks for your kind words :) And happy to see that you've been able to sort out many of your issues on your own. Your feedback is super helpful to improve our docs!

2 things about the settings feature:

it does require keeping the translations table, simply because the value is saved there, even if you have a single language. I definitely feel like it shouldn't be in that case, we can think about improving!
for now, only text inputs work in settings forms. This is quite a limitation and could be a very useful change to support more fields there. For now, I'd suggest creating a module, even if it's for a single record.

Cheers!