From ed07b3c01fb7359e51436b66e0fa582809d8815a Mon Sep 17 00:00:00 2001
From: EngincanV <enginveske@gmail.com>
Date: Mon, 17 May 2021 12:53:10 +0300
Subject: [PATCH] Remove illegal char check for project creation

---
 .../Abp/Cli/Utils/ProjectNameValidator.cs     | 34 -------------------
 .../Abp/Cli/ProjectNameValidation_Tests.cs    | 26 --------------
 2 files changed, 60 deletions(-)

diff --git a/framework/src/Volo.Abp.Cli.Core/Volo/Abp/Cli/Utils/ProjectNameValidator.cs b/framework/src/Volo.Abp.Cli.Core/Volo/Abp/Cli/Utils/ProjectNameValidator.cs
index e8237c5215..8430cd79a0 100644
--- a/framework/src/Volo.Abp.Cli.Core/Volo/Abp/Cli/Utils/ProjectNameValidator.cs
+++ b/framework/src/Volo.Abp.Cli.Core/Volo/Abp/Cli/Utils/ProjectNameValidator.cs
@@ -17,40 +17,11 @@ namespace Volo.Abp.Cli.Utils
             "LPT2"
         };
 
-        private static readonly char[] IllegalChars = new[]
-        {
-            '/',
-            '?',
-            ':',
-            '&',
-            '\\',
-            '*',
-            '\'',
-            '<',
-            '>',
-            '|',
-            '#',
-            '%',
-        };
-
         private static bool HasParentDirectoryString(string projectName)
         {
             return projectName.Contains("..");
         }
 
-        private static bool HasIllegalChar(string projectName)
-        {
-            foreach (var illegalChar in IllegalChars)
-            {
-                if (projectName.Contains(illegalChar))
-                {
-                    return true;
-                }
-            }
-
-            return false;
-        }
-
         private static bool HasSurrogateOrControlChar(string projectName)
         {
             return projectName.Any(chr => char.IsControl(chr) || char.IsSurrogate(chr));
@@ -76,11 +47,6 @@ namespace Volo.Abp.Cli.Utils
                 throw new CliUsageException("Project name cannot be empty!");
             }
 
-            if (HasIllegalChar(projectName))
-            {
-                return false;
-            }
-
             if (HasSurrogateOrControlChar(projectName))
             {
                 return false;
diff --git a/framework/test/Volo.Abp.Cli.Core.Tests/Volo/Abp/Cli/ProjectNameValidation_Tests.cs b/framework/test/Volo.Abp.Cli.Core.Tests/Volo/Abp/Cli/ProjectNameValidation_Tests.cs
index 2e6d0074bd..cd8581f912 100644
--- a/framework/test/Volo.Abp.Cli.Core.Tests/Volo/Abp/Cli/ProjectNameValidation_Tests.cs
+++ b/framework/test/Volo.Abp.Cli.Core.Tests/Volo/Abp/Cli/ProjectNameValidation_Tests.cs
@@ -38,32 +38,6 @@ namespace Volo.Abp.Cli
             }
         }
 
-        [Fact]
-        public async Task ContainsIllegalChar_Test()
-        {
-            var illegalChars = new[]
-            {
-                '/',
-                '?',
-                ':',
-                '&',
-                '\\',
-                '*',
-                '\'',
-                '<',
-                '>',
-                '|',
-                '#',
-                '%',
-            };
-
-            foreach (var illegalChar in illegalChars)
-            {
-                var args = new CommandLineArgs("new", "Test" + illegalChar);
-                await _newCommand.ExecuteAsync(args).ShouldThrowAsync<CliUsageException>();
-            }
-        }
-
         [Fact]
         public async Task ParentDirectoryContain_Test()
         {