To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=84979
------- Additional comments from [EMAIL PROTECTED] Tue Jan 8 02:36:55 +0000
2008 -------
Having spent 30 mins running through various test cases the Excel validation
business rule is:
(1) replace all non alphanumeric (+list of special chars) with _
(2) append _ to any names which would otherwise be valid sheet references.
Note that since (1) removes ! and : we only need to test a limit number of cases
in (2):
The validation process is therefore as follows:
1) Scan the name replacing any character other than as follows with “_”
Any Alphabetic Character (regardless of language)
Any Numeric Character
The Special Characters: "." (46), "?" (63), "\" (92), "ˆ" (95),
"_" (128), "¨" (136), "¯" (152), "´" (153), "¸" (168),
"˜" (175), "•" (180), "€" (183), "™" (184)
2) If any substitutions have occurred then trim any leading or trailing “_”
3) If the string matches /^([a-zA-Z]{1,2})(\d+)$/ and
uppercase($1) < ”IV” and $2 < 65537
Then append “_” to the name
3) If the string matches /^([Rr](\d*)$/ and $1 < 65537 or
/^([Cr](\d*)$/ and $1 < 257 or
/^([Rr](\d*) ([Cr](\d*)$/ and $1 < 65537 $2 < 257
Then append “_” to the name and flag an error on this name.
Don't ask my why that list of special chars, but that's what it is. Likewise
why it objects to RC1_ and not A1_ etc..
Given that the only change here is that Calc uses the . separator then then the
only change that you would need to do to drop "." from the special character
list.
Given that OOo already contains a perfectly good Regexp engine we could code
this up is about 30 lines of code rather than the current 1,000 or so, and even
put in a few comments to say what we are doing and why.
---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
