-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix(ZMS-3253 ZMS-3466 ZMS-3415 ZMS-1891): Replicate frontend validation in the backend for the availability opening hours to improve data integrity and frontend validation messaging #825
Open
ThomasAFink
wants to merge
199
commits into
next
Choose a base branch
from
bugfix-zms-3253-3466-3415-1891-validation-opening-hours-of-the-same-appointment-type-must-not-overlap
base: next
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 194 commits
Commits
Show all changes
199 commits
Select commit
Hold shift + click to select a range
310f536
fix(ZMS-3253): validate and block adding conflicting openingtimes
bbd32b1
fix(ZMS-3253): add backend validation for opening hours and improve f…
5543f36
fix(ZMS-3253): fix zmsentities unit test
03bc86e
fix(ZMS-3253): fix zmsdb unit test
ffcdb0a
fix(ZMS-3253): add logging to zmsapi tests
47565c8
fix(ZMS-3253): fix one zmsapi unit test
3f8030e
fix(ZMS-3253): try fix one zmsapi unit test
f5e08ed
fix(ZMS-3253): try fix one zmsapi unit test
e4b36ea
fix(ZMS-3253): try fix some unit tests
e5f123c
fix(ZMS-3253): try fix some unit tests
9528271
fix(ZMS-3253): try fix some unit tests
770e847
fix(ZMS-3253): try fix some unit tests
4a79605
fix(ZMS-3253): try fix some unit tests
cdd0d8c
fix(ZMS-3253): try fix some unit tests
8a1aa79
fix(ZMS-3253): try fix some unit tests
09c10fb
fix(ZMS-3253): try fix some unit tests
afd6d5d
fix(ZMS-3253): try fix some unit tests
ce5d795
fix(ZMS-3253): try fix some unit tests
07f74c8
Revert "fix(ZMS-3253): try fix some unit tests"
da65ff3
fix(ZMS-3253): try fix some unit tests
93b2820
fix(ZMS-3253): try fix some unit tests
e0d41c4
fix(ZMS-3253): try fix some unit tests
bde21f3
fix(ZMS-3253): try fix some unit tests
0cd0ad1
fix(ZMS-3253): try fix some unit tests
29b7cff
fix(ZMS-3253): try fix some unit tests
9e11157
fix(ZMS-3253): try fix some unit tests
c2df444
fix(ZMS-3253): try fix some unit tests
5faea71
fix(ZMS-3253): try fix some unit tests
4cd3618
fix(ZMS-3253): try fix some unit tests
4840338
fix(ZMS-3253): try fix some unit tests
c8262a0
fix(ZMS-3253): backend validation work in AvailabilityAdd
8281b0a
fix(ZMS-3253): correct backend validation for timeoverlaps
390aba7
fix(ZMS-3253): fix a validation
78013ea
fix(ZMS-3253): try fixing a test
8a07621
fix(ZMS-3253): try fixing a test
7892fe0
fix(ZMS-3253): try fixing a test
9522585
fix(ZMS-3253): try fixing a test
d65ccb8
fix(ZMS-3253): try fixing availability not found test
3fa84a1
fix(ZMS-3253): try fixing availability mockdata for testRendering
317d127
fix(ZMS-3253): try fixing availability mockdata for testRendering
c02e4e3
fix(ZMS-3253): try fixing availability mockdata for testRendering
c75e3c4
fix(ZMS-3253): try fixing availability mockdata for testRendering
2b39546
fix(ZMS-3253): try fixing availability mockdata for testRendering
acb62ac
fix(ZMS-3253): try fix missing scope testRendering
a16b2e4
fix(ZMS-3253): fix zmsadmin availabilities conflict test
1bd40cb
fix(ZMS-3253): show availability opening hour conflicts in the future…
4025319
fix(ZMS-3253): remove error_logs
95dcef0
fix(ZMS-3253): refactor function logic getDateTimeRangeFromList to on…
39ef3a6
Merge remote-tracking branch 'origin/next' into bugfix-zms-3253-valid…
3eb8912
fix(ZMS-3253): renable twig cache
46c0af9
fix(ZMS-3253): renable twig cache
fdd0c72
fix(ZMS-3253): fix unit test
ThomasAFink f319de3
fix(ZMS-3253): fix unit test
ThomasAFink b8fd6b6
fix(ZMS-3253): fix unit test
ThomasAFink a2691c1
fix(ZMS-3253): fix unit test
ThomasAFink fc26e13
fix(ZMS-3253): fix unit test
ThomasAFink bffc9fa
fix(ZMS-3253): fix unit test
ThomasAFink 5784eb4
fix(ZMS-3253): try fix unit test
ThomasAFink 51e25ca
fix(ZMS-3253): add unit tests for testing overlapping availability op…
ThomasAFink 4f2a125
fix(ZMS-3253): add unit tests for testing overlapping availability op…
ThomasAFink e8cae55
fix(ZMS-3253): remove space
ThomasAFink e0f4fa6
fix(ZMS-3253): remove space
ThomasAFink d17ffa2
fix(ZMS-3253): add unit tests for testing overlapping availability op…
ThomasAFink 8a41469
fix(ZMS-3253): add unit tests for testing validation availability ope…
ThomasAFink e230b26
fix(ZMS-3253): add unit tests for testing overlapping availability op…
ThomasAFink 5aadb7b
fix(ZMS-3253): add unit tests for testing validation availability ope…
ThomasAFink 246b4d1
fix(ZMS-3253): add unit tests for testing overlapping availability op…
ThomasAFink 17dcb07
fix(ZMS-3253): add unit tests for testing overlapping availability op…
ThomasAFink e984774
fix(ZMS-3253): add unit tests for testing overlapping availability op…
ThomasAFink d5135e2
fix(ZMS-3253): add unit tests for testing overlapping availability op…
ThomasAFink ca99ef2
fix(ZMS-3253): add unit tests for testing validation availability ope…
ThomasAFink d5c6051
fix(ZMS-3253): add unit tests for testing validation availability ope…
ThomasAFink 5527c58
fix(ZMS-3253): refactor object creation
ThomasAFink ad89835
fix(ZMS-3253): refactor exception messages
ThomasAFink 2a7d978
fix(ZMS-3253): clean up commented code
ThomasAFink ea55b27
fix(ZMS-3253): fix grammar
ThomasAFink 932fe5f
fix(ZMS-3253): comment error_log
ThomasAFink 5aa9d6b
fix(ZMS-3253): Improve frontend validation for opening hours availabi…
ThomasAFink fb7efb7
cleanup(ZMS-3415): remove invalid text instruction for graph view ope…
ThomasAFink 1fd8f65
fix(ZMS-1891): Add missing frontend validation for timepicker
f8f536f
Merge branch 'next' into bugfix-zms-3253-validation-opening-hours-of-…
e4981ff
fix(ZMS-1891): Improve frontend validation for time formats
7916a70
fix(ZMS-3253): fix frontend exclusion availability validation
ThomasAFink 462be60
fix(ZMS-3253): fix backend exclusion availability validation
ThomasAFink 46db273
fix(ZMS-3253): allow exclusion availability on current date
ThomasAFink 490ddce
fix(ZMS-3253): improve validation and cleanup code
ThomasAFink 2ce2363
fix(ZMS-3253): move js spinner
ThomasAFink 735c579
Merge remote-tracking branch 'origin/next' into bugfix-zms-3253-valid…
ThomasAFink 97cee9c
clean(ZMS-3253): remove console logs
ThomasAFink a11f7b8
Merge branch 'next' into bugfix-zms-3253-validation-opening-hours-of-…
b620a54
fix(ZMS-3466): fix bookable day range validation
e3375c1
fix(ZMS-3466): fix error accumulation
e646a4c
fix(ZMS-3466): improve exception handling
8b7a1f6
fix(ZMS-3466): improve validation prevent divide by zero
2a3cbb7
fix(ZMS-3466): improve validation prevent divide by zero
4cbe423
fix(ZMS-3466): improve validation prevent divide by zero
0053e60
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
ThomasAFink cab1b1d
fix(ZMS-3466): Fix to from default values in opening hours availability
ThomasAFink ab51699
fix(ZMS): error message
227e741
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
694194b
fix(ZMS-3466): fix open from and open to placeholders in form
d53b28e
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
088356f
fix(ZMS-3466): improve default hours
484cbb4
fix(ZMS-3466): improve hours selection
e92684e
fix(ZMS-3466): closing new opening hours
faa90c0
fix(ZMS-3253): add backend validation check for new availabilities ov…
8d863ce
fix(ZMS-3253): unit test
ff16fc0
fix(ZMS-3253): unit tests
4043a66
fix(ZMS-3253): unit tests
860abc6
fix(ZMS-3253): add frontend validation check for new availabilities o…
3a3491f
fix(ZMS-3253): new availability conflict error display
64c884b
fix(ZMS-3253): when footer buttons are disabled
36724e4
fix(ZMS-3466): disable editing availabilities in the past
ea9b087
fix(ZMS-3466): improve infobox message format for non errors
687af94
fix(ZMS-3466): fix white space scroll error
d86c998
fix(ZMS-3466): disable maintanence daily cronjob hours from opening h…
39759a4
fix(ZMS-3466): add validation tests
65d6e79
fix(ZMS-3466): refactor overlapping new availabilities into one function
551438b
fix(ZMS-3466): add overlap tests in zmsadmin
2f92832
fix(ZMS-3466): add overlap test in zmsentities
36e81f3
fix(ZMS-3466): add more tests
c595cad
fix(ZMS-3466): fix detect conflict in the future with one time openin…
b7d7ac8
fix(ZMS-3466): Add tests
ce1dc34
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
1612ee4
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
b504141
fix(ZMS-3466): Overlapping new availabilities
0a49139
fix(ZMS-3466): fix cancel update availability clear errors
7402576
fix(ZMS-3466): fix allow 0 in form for appointment days in future
8e0cdc8
fix(ZMS-3466): fix allow 0 in form for appointment days in future end…
b89844d
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
75dd8ed
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
26b63cf
fix(ZMS-3253): cleaned up some code from review and added success mes…
b9c938d
fix(ZMS-3253): fix unit test
29049a6
fix(ZMS-3253): improve time selection
97e70f1
clean(ZMS-3253): remove console log
4deb527
fix(ZMS-3253): check boxes disabled for past opening hours
93bde12
fix(ZMS-3253): allow overlapping weekdays
4f31871
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
a52d441
fix(ZMS-3235): fix revert form error messages and fix disabled timesl…
5cf8728
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
5acfab2
fix(ZMS-3253): filter availability conflicts by weekday to prevent bl…
a53f7c7
fix(ZMS-3253): Add two unit tests
1e5cd4d
fix(ZMS-3253): Add two unit tests
8824b41
fix(ZMS-3253): fix two unit tests
308de88
fix(ZMS-3253): fix two unit tests
20d6cbf
fix(ZMS-3253): fix two unit tests
9dbb73a
fix(ZMS-3253): fix two unit tests
d5e4f5b
fix(ZMS-3253): fix two unit tests
e88a979
fix(ZMS-3253): fix two unit tests
cc2a6c0
fix(ZMS-3253): fix two unit tests
a45cbc2
fix(ZMS-3253): fix two unit tests
33b5a73
fix(ZMS-3253): fix two unit tests
ca2526a
fix(ZMS-3253): fix two unit tests
0731d6f
fix(ZMS-3253): fix two unit tests
11f7496
fix(ZMS-3253): fix two unit tests
8fb3354
fix(ZMS-3253): fix two unit tests
72646c3
fix(ZMS-3253): fix two unit tests
896b327
fix(ZMS-3253): fix weekday to date validation
3eaaa1b
fix(ZMS-3253): conflicts for exclusions
52fc4cd
clean(ZMS-3253): remove error logs
3b65923
fix(ZMS-3253): unit tests
35dfce4
fix(ZMS-3253): unit test
fb01fac
fix(ZMS-3253): display weekdays in conflicts
ee65b9b
fix(ZMS-3253): unit test
5745df3
fix(ZMS-3253): unit tests
c89eb94
feat(ZMS-3253): add color schema to opening hours accordion with legend
b1a3d3c
fix(ZMS-3253): when creating exclusion is disabled
5c82cb3
fix(ZMS-3253): when creating exclusion is disabled
c6410ad
fix(ZMS-3253): syntax error
ThomasAFink a943be8
feat(ZMS-3253): include color coding outside of just editing
2b0f8eb
fix(ZMS-3253): disable buttons for dates in the past
c14d277
fix(ZMS-3253): update button when disabled
6419403
fix(ZMS-3253): default description new opening hours and improve time…
0dba8c3
fix(ZMS-3253): syntax error
d316efb
feat(ZMS-3253): Add tab to view and edit all opening hours for a scope
c6e2dc0
clean(ZMS-3253): Add missing grammar
7f6f94f
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
cde1a14
feat(ZMS-3253): add sort function for the table
68d68fb
Update zmsadmin/src/Zmsadmin/AvailabilityListByScope.php
ThomasAFink 61f3823
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
0151302
clean(ZMS-3253): format code
244cef3
clean(ZMS-3253): reduce zmsadmin AvailabilityConflicts complexity
9a315fe
clean(ZMS-3253): zmsentities remove unused vars
f6a641a
clean(ZMS-3253): code formatting
6f98308
clean(ZMS-3253): code formatting
8e1c425
clean(ZMS-3253): reduce complexity in AvailabilityUpdate and Availabi…
4cb3ded
clean(ZMS-3253): remove unused parameter
1f28b01
clean(ZMS-3253): formatting
fcf3a7d
clean(ZMS-3253): package.lock
85be2b5
fix(ZMS-3253): weekday validation logic
4ebc9cc
clean(ZMS-3253): formatting
393edf5
clean(ZMS-3253): formatting
a6e2d58
clean(ZMS-3253): remove color and all availability view
27065a1
clean(ZMS-3253): pass corrected resolveReferences
c4164cb
clean(ZMS-3253): remove unused style in js
48e1fba
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
c4115e5
Merge branch 'next' into bugfix-zms-3253-3466-3415-1891-validation-op…
341674e
fix(ZMS-3253): save amendment field overwrite in availability
35d0ffb
fix(ZMS-3253): .keep cache folder in zmsadmin
ThomasAFink File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -7,7 +7,6 @@ | |
*/ | ||
class Condition | ||
{ | ||
|
||
protected $collection = []; | ||
|
||
public function __construct(Valid ...$validList) | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
**/ | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
<?php | ||
|
||
/** | ||
* @package Mellon | ||
* @copyright BerlinOnline Stadtportal GmbH & Co. KG | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Security Risk: Unsafe HTML rendering.
Using
dangerouslySetInnerHTML
with unsanitized content fromitem.message
exposes the application to XSS attacks.Consider these safer alternatives:
📝 Committable suggestion
🧰 Tools
🪛 Biome (1.9.4)
[error] 33-33: Avoid passing content using the dangerouslySetInnerHTML prop.
Setting content using code can expose users to cross-site scripting (XSS) attacks
(lint/security/noDangerouslySetInnerHtml)