dojo.date.stamp toISOString and fromISOString have uppercase acronyms

[skinner]

I think that back in 0.4, dojo.date had ISO functions like "setIso8601" and "fromIso8601". Now in 0.9, dojo.date.stamp has functions named "toISOString" and "fromISOString".

The dojo style guide says, "Abbreviations and acronyms SHOULD NOT be UPPERCASE when used as a name: getInnerHtml(), getXml(), XmlDocument?".

The old function names conformed to the dojo style guide, and the new names don't. Was this an intentional change, or just an oversight?

[Adam Peller]

It was done to match the proposed ECMA standard. I'm not crazy about this conflicting with our style guide, and I'm not even able to find a reference to this atm -- perhaps it's shifting around? What do you suggest we do?

[Adam Peller]

whatever we do, we should come to a resolution for 0.9

[skinner]

Got it. I didn't know that the names were chosen to match the proposed standard. A worthy goal. Bummer that it conflicts with the style guide. I have no suggestion for resolving that conflict.

This particular detail of the dojo style guide is maybe out of sync with the cultural norms in the rest of the JavaScript? world, outside of dojo. Personally I like the dojo convention of not using UPPERCASE for acronyms, but it does seem to lead to problems, since using UPPERCASE seems to be what everybody habitually does. If we could go back in time a year, I might reluctantly suggest changing that entry in the style guide, but I think we're now past the time when we could do that.

[Adam Peller]

well, not having any better ideas myself, I'm marking this wontfix. Please reopen if you have any suggestions.