Skip to content
This repository has been archived by the owner on Oct 6, 2019. It is now read-only.

Conversation

SergeyDryomin
Copy link
Contributor

Fixed "Blank name is accepted as Group name"
Added validation for white spaces string, when new group added.
*No UT added - to add it have to add UT new project

Summary

dnnsoftware/Dnn.Platform#2927
DNN-18333
Note: same issues possible to fix into 'DNN Platform\Library\Common\Requires.cs' (with adding new method or just to change NotNullOrEmpty to IsNullOrWhiteSpace validation)

@dnfclas
Copy link

dnfclas commented Aug 7, 2019

CLA assistant check
All CLA requirements met.

@valadas valadas added the bug Something isn't working label Aug 8, 2019
Copy link
Contributor

@valadas valadas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, if you have time to do the other PR for the Platform, it would be even nicer :)

@valadas valadas added this to the 9.4.1 milestone Aug 8, 2019
@valadas valadas modified the milestones: 9.4.1, 10.0.0 Sep 17, 2019
@valadas
Copy link
Contributor

valadas commented Sep 17, 2019

@SergeyDryomin this is currently targetting development which means it would be included in Dnn 10, would you like to re-target the release/3.0.x branch for inclusion in Dnn 9.4.1 ?

@valadas valadas modified the milestones: 10.0.0, 9.4.2 Sep 29, 2019
@valadas valadas merged commit d1eb942 into dnnsoftware:development Sep 29, 2019
@SergeyDryomin SergeyDryomin deleted the bugfix/DNN-18333_Blank_Name_is_accepted_as_Group_name branch July 15, 2020 12:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants