Skip to content
This repository was archived by the owner on Mar 9, 2020. It is now read-only.

Named cells validation #529

Open
righthandpaw wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Named cells validation #529

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
2d7c122
Added New Validation for Named Cells
righthandpaw Jul 7, 2019
64aaf4f
Updated the named range validity checking rules
Aug 26, 2019
5d6652d
Added Temporary fix to force a valid named range
Oct 24, 2019
1ede1d0
Resolved Bug in Named Range Validation
righthandpaw Oct 24, 2019
a3f5cd2
Changed Named Range Validation to use .NET3.5 method of clearing strings
righthandpaw Oct 24, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion EPPlus/ExcelNamedRangeCollection.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,9 +66,12 @@ internal ExcelNamedRangeCollection(ExcelWorkbook wb, ExcelWorksheet ws)
public ExcelNamedRange Add(string Name, ExcelRangeBase Range)
{
ExcelNamedRange item;
if(!ExcelAddressUtil.IsValidName(Name))

if (!ExcelAddressUtil.IsValidName(Name))
{
throw (new ArgumentException($"Name {Name} contains invalid characters")); //Issue 458
//Name = ExcelAddressUtil.GetValidName(Name); //temporary fix

}
if (Range.IsName)
{
Expand Down
Loading