Skip to content
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

Merged
merged 10 commits into from
Oct 30, 2023
Merged

Upgrade Methoden umbauen #1391

merged 10 commits into from
Oct 30, 2023

Conversation

hansmorb
Copy link
Contributor

@hansmorb hansmorb commented Oct 26, 2023

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

@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #1391 (04f5310) into master (a43aed3) will increase coverage by 0.90%.
Report is 2 commits behind head on master.
The diff coverage is 59.55%.

@@             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     
Files Coverage Δ
src/Plugin.php 33.61% <0.00%> (+5.07%) ⬆️
src/Service/Upgrade.php 61.62% <61.62%> (ø)

... and 5 files with indirect coverage changes

@hansmorb hansmorb added this to the 2.8.5 milestone Oct 26, 2023
@hansmorb hansmorb added technical Non-functional changes (refactorings or increase test coverage) php Pull requests that update Php code duplicate and removed duplicate labels Oct 26, 2023
@hansmorb hansmorb requested a review from chriwen October 26, 2023 18:42
@datengraben
Copy link
Contributor

@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.

@hansmorb
Copy link
Contributor Author

@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.

@hansmorb hansmorb merged commit f8600e5 into master Oct 30, 2023
10 checks passed
@hansmorb hansmorb deleted the refactoring/upgrade branch November 1, 2023 22:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
php Pull requests that update Php code technical Non-functional changes (refactorings or increase test coverage)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants