Introduce pdo driver for symfony cache

[datengraben]

This enables database based caching (mysql) via Symfony PDOAdapter for hosts where file based caching is not possible.

When a redis connection and pdo is enabled, pdo is used for caching and redis is used for caching the tags.

TODO

• although it works. The pdoAdapter member variable which holds the adapter instance bloats the Plugin/Cache interface (bc it's private). It should be moved in a separated holder type.

[datengraben]

@hansmorb was denkst du zur Verwendung der Oberfläche im Backend?

[codecov]

Codecov Report

Attention: 46 lines in your changes are missing coverage. Please review.

Comparison is base (637cb31) 39.76% compared to head (d22d35d) 42.96%.
Report is 60 commits behind head on master.

Files	Patch %	Lines
src/Service/Cache.php	11.76%	45 Missing ⚠️
src/Plugin.php	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1445      +/-   ##
============================================
+ Coverage     39.76%   42.96%   +3.20%     
- Complexity     2324     2395      +71     
============================================
  Files            91       91              
  Lines          9602     9507      -95     
============================================
+ Hits           3818     4085     +267     
+ Misses         5784     5422     -362     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[hansmorb]

Das ist vielleicht ein bisschen wild aber könnten wir nicht den ganzen Cache Backends ein Interface geben mit einer abstrakten Methode renderConnectionStatus und dann jeweils getConnectionError oder so ähnlich implementieren?
Dann haben wir zwar einen Wrapper für Interfaces in einem Interface gewrappt aber dann würden wir uns nicht so oft wiederholen. Oder wir bauen eine Methode, die den connectionStatus für jedes übergebene Interface rendert.

[datengraben]

Ja fände ich auch nicht verkehrt. Am Liebsten wäre mir, wenn wir einen "normalen" Filebased Cache, einen PDO Cache (mysql) und einen Redis Cache anbieten, für alle nicht technisch versierten User. Und zusätzlich beim Cache instanzieren ein `apply_filter` stehen würde, damit technisch versierte User des Plugins ihr eigenes Caching-System über den Hook konfigurieren können, ohne den Quell-Code des Plugins anfassen zu müssen. Dann hätten wir sicher 80% der Anwendungsfälle abgedeckt und alles weitere sehr einfach möglich gemacht. Am Thu, Dec 07, 2023 at 01:02:47PM -0800 schrieb Hans Morbach:

…

Das ist vielleicht ein bisschen wild aber könnten wir nicht den ganzen Cache Backends ein Interface geben mit einer abstrakten Methode renderConnectionStatus und dann jeweils getConnectionError oder so ähnlich implementieren? Dann haben wir zwar einen Wrapper für Interfaces in einem Interface gewrappt aber dann würden wir uns nicht so oft wiederholen. Oder wir bauen eine Methode, die den connectionStatus für jedes übergebene Interface rendert. -- Reply to this email directly or view it on GitHub: #1445 (comment) You are receiving this because you authored the thread. Message ID: ***@***.***>

[hansmorb]

Wenn ich das aktiviere bekomme ich aktuell noch folgenden Fehler:

Das mit dem apply_filter können wir schon machen, weiss aber nicht ob das jemand ernsthaft nutzen wird. Schon bei dem REDIS Adapter bezweifle ich, dass das jemand nutzt.