-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade Methoden umbauen #1391
Upgrade Methoden umbauen #1391
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1391 +/- ##
============================================
+ Coverage 36.90% 37.80% +0.90%
- Complexity 2165 2178 +13
============================================
Files 83 84 +1
Lines 8807 8830 +23
============================================
+ Hits 3250 3338 +88
+ Misses 5557 5492 -65
|
the unit tests worked because $previousVersion = '' is also != to any other version
@hansmorb gefällt mir. Ich habe mir gerade überlegt ob es sinnvoll wäre über wordpress filter oder action hooks die Upgrade-Funktion manipulieren oder in sie einhaken zu können. Also wenn ich ein Power-User bin und meine Instanz den CB-Core z.B. etwas verändert nutzt und migrations erweitern oder überspringen will. Also als Idee für die Zukunft. |
Hmm, stelle ich mir schwierig vor, weil die Ausführung der Funktionen ja an einen CB Version Bump gebunden ist und nicht an den Version Bump eines externen Plugins. Das heißt jede Änderung in dem 3rd Party Plugin würde erst umgesetzt werden, wenn dabei auch CB geupdatet wird. |
Das war ursprünglich Teil von #1306 , wurde aber zum einfacheren Review ausgegliedert.
Warum:
-Die Plugin Klasse wurde mit den ganzen Migrationen langsam etwas unübersichtlich
-Jetzt haben wir eine Möglichkeit Upgrade Tasks nur 1x (nämlich bei dem Upgrade über die Version) laufen zu lassen. So wird zum Beispiel nicht ständig das Farbschema bei jedem Upgrade zurückgesetzt