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

'NoRep' Einstellung entfernen #1378

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from
Draft

'NoRep' Einstellung entfernen #1378

wants to merge 10 commits into from

Conversation

hansmorb
Copy link
Contributor

Das entfernt die Funktion "keine Wiederholung" für Zeitrahmen einzustellen und passt vorhandene Zeitrahmen entsprechend an. Dazu wird bei allen Zeitrahmen die 'norep' Einstellung zu 'd' (daily) geändert.
Da Zeitrahmen ohne Wiederholung ohne Enddatum nur für den Tag des Startdatums gültig waren, wird bei diesen Zeitrahmen das Enddatum auf das Startdatum gesetzt.

closes #1005

We achieve this by giving single day timeframes an end-date and making the isInTimeframe check for start- and enddate
@hansmorb hansmorb added this to the Holiday milestone Oct 11, 2023
@hansmorb hansmorb added enhancement New feature or request php Pull requests that update Php code labels Oct 11, 2023
@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

Merging #1378 (6bcd939) into master (a4de922) will decrease coverage by 2.35%.
Report is 2 commits behind head on master.
The diff coverage is 82.35%.

❗ Current head 6bcd939 differs from pull request most recent head 9052bdc. Consider uploading reports for the commit 9052bdc to get more accurate results

@@             Coverage Diff              @@
##             master    #1378      +/-   ##
============================================
- Coverage     38.22%   35.88%   -2.35%     
+ Complexity     2193     2119      -74     
============================================
  Files            84       83       -1     
  Lines          8902     8600     -302     
============================================
- Hits           3403     3086     -317     
- Misses         5499     5514      +15     
Files Coverage Δ
src/Wordpress/CustomPostType/Timeframe.php 11.88% <ø> (-1.81%) ⬇️
src/Migration/Migration.php 0.00% <0.00%> (ø)
src/Model/Day.php 74.33% <75.00%> (-7.02%) ⬇️
src/Plugin.php 30.36% <91.66%> (-2.98%) ⬇️

... and 21 files with indirect coverage changes

# Conflicts:
#	src/Model/Day.php
#	src/Service/Upgrade.php
#	src/Wordpress/CustomPostType/Timeframe.php
#	tests/php/Model/DayTest.php
#	tests/php/PluginTest.php
#	tests/php/Service/UpgradeTest.php
#	tests/php/View/ViewTest.php
@hansmorb hansmorb marked this pull request as draft November 27, 2023 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request php Pull requests that update Php code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Zeitrahmen: Keine Wiederholung entfernen
1 participant