Opened 11 years ago
Closed 11 years ago
#12842 closed enhancement (fixed)
[patch] [cla] Quote certain properties that match external names
| Reported by: | Stephen Chung | Owned by: | Eugene Lazutkin |
|---|---|---|---|
| Priority: | lowest | Milestone: | 1.7 |
| Component: | General | Version: | 1.6.0 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
Properties of certain classes are designed to match with external names (e.g. colors) or to be matched by string (e.g. handleAs for dojo.xhr).
Attachments (1)
Change History (6)
Changed 11 years ago by
| Attachment: | dojo.quotes.patch added |
|---|
comment:1 Changed 11 years ago by
| Milestone: | 1.6.2 → tbd |
|---|---|
| Summary: | Quote certain properties that match external names → [patch] [cla] Quote certain properties that match external names |
comment:2 Changed 11 years ago by
comment:3 Changed 11 years ago by
| Milestone: | tbd → 1.6.2 |
|---|---|
| Owner: | set to Eugene Lazutkin |
| Status: | new → assigned |
comment:4 Changed 11 years ago by
| Milestone: | 1.6.2 → 1.7 |
|---|
comment:5 Changed 11 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Recommend: quote properties for classes that are designed to match external names, or to be matched by string. Reasons:
1) It allows the reader to see at a glance which classes contain properties with names that are significant as text (i.e. may break code if the properties change).
2) It allows addition of properties in the future with illegal characters that must be quoted without being inconsistent.
3) It does not impact gzipped size.
4) The Closure Compiler will not touch quoted properties, but will attempt to rename ones that are not quoted. This is not a very important use case, as this can also be handled via a post-processing pass.