search

zicht / messages-bundle

Library - ZichtMessagesBundle - Edit your translations in the database
MIT License
1 stars 0 forks source link

Rewire custom database loader #27

Closed erik-zicht closed 3 years ago

erik-zicht commented 4 years ago

Fix #25

7ochem commented 4 years ago

Ik heb een commit Catch DB Exceptions toegevoegd. D.m.v. een catch (\Doctrine\DBAL\Exception\ConnectionException $e) kun je een falende database connectie afvangen en gaat de cache warmup gewoon door.

Graag zou ik tijdens developen gewoon een cache:clear willen kunnen doen en dan weer een "verse start" hebben, incl. die vertalingen. Ik zou dan niet nog een command uit willen voeren...

We zouden bij een falende DB connectie kunnen kijken of we op een of andere manier een warning kunnen weergeven, zodat je weet dat die DB entries er nog niet bij zitten.

erik-zicht commented 4 years ago

Graag zou ik tijdens developen gewoon een cache:clear willen kunnen doen en dan weer een "verse start" hebben, incl. die vertalingen. Ik zou dan niet nog een command uit willen voeren...

We zouden in de CacheHook in warmUp kunnen bepalen of de Loader wel of niet uit moet o.b.v. de debug-state van de applicatie...?

erik-zicht commented 3 years ago

Ik ga dit PR voorlopig even sluiten, mocht er nog verder gewerkt worden aan deze specifieke oplossing, de branch blijft gewoon bestaan.

erik-zicht commented 3 years ago

Graag zou ik tijdens developen gewoon een cache:clear willen kunnen doen en dan weer een "verse start" hebben, incl. die vertalingen. Ik zou dan niet nog een command uit willen voeren...

Ik denk niet dat dat gaat. Zolang de Zicht\Bundle\MessagesBundle\Translation\Loader op enabled staat gaat ie in de database kijken. Ik heb een zicht:messages:cache command gemaakt, die regelt nu het wegschrijven van de cache. Je kunt deze in de composer-install scripts zetten en mogelijk lokaal in de z-yaml achter de z content:db:pull ... commando's plakken?