[Bug 56781] XSSFName.validateName(String) Only Checks for the First Character's Validity and Presence of Spaces

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56781

Javen O'Neal  changed:

   What|Removed |Added

 Status|NEEDINFO|RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Javen O'Neal  ---
Fixed in r1764854.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56781] XSSFName.validateName(String) Only Checks for the First Character's Validity and Presence of Spaces

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56781

Javen O'Neal  changed:

   What|Removed |Added

 Blocks||60246

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56781] XSSFName.validateName(String) Only Checks for the First Character's Validity and Presence of Spaces

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56781

--- Comment #4 from Javen O'Neal  ---
Reverted back to using Character.isLetter in r1749305 because Regular
Expressions \p{IsAlphabetic} metaclass is not supported in Java 6.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56781] XSSFName.validateName(String) Only Checks for the First Character's Validity and Presence of Spaces

[bugzilla]

https://bz.apache.org/bugzilla/show_bug.cgi?id=56781

Javen O'Neal  changed:

   What|Removed |Added

 Status|NEW |NEEDINFO

--- Comment #3 from Javen O'Neal  ---
That Excel help forum may not be reputable (for example, unicode characters are
allowed, allowing backslash as the first character but not subsequent
characters raises flags).

Better would be documentation from Microsoft on allowed names in Excel 97-2003
and 2007+. This would need to be verified using Excel (preferably these files
could be used for POI unit tests).

I made up a regular expression that validates the entire string in r1749293.
This regular expression will likely need tweaking to match Excel's rules.

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56781] XSSFName.validateName(String) Only Checks for the First Character's Validity and Presence of Spaces

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=56781

--- Comment #1 from Ryan O'Meara rome...@live.com ---
Side note: If I manage to get the POI environment setup, I'd be willing to take
a crack at a patch for this

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

[Bug 56781] XSSFName.validateName(String) Only Checks for the First Character's Validity and Presence of Spaces

[bugzilla]

https://issues.apache.org/bugzilla/show_bug.cgi?id=56781

--- Comment #2 from Nick Burch apa...@gagravarr.org ---
http://poi.apache.org/howtobuild.html and http://poi.apache.org/guidelines.html
should hopefully cover you on getting started to contribute

FormulaParser currently has logic for deciding if a sequence of characters is a
valid (anything) for a formula, my first hunch is it might be possible to
perhaps re-use the name validating parts of that?

-- 
You are receiving this mail because:
You are the assignee for the bug.

-
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org