Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#3687 closed task (fixed)

use comma rather than semicolon as array separator

Reported by: bill Owned by: Adam Peller
Priority: high Milestone: 0.9
Component: Dijit Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description

parser.js [9519] changed the array delimiter in the parser from semi-colon to comma, so that you declare array parameters like:

   foo="1,2,3"

Should make same change for widget templates, so that dojoAttachEvent, etc. also uses commas:

dojoAttachEvent="onclick: _onClick, onkeyup: _onKeyUp"

Have to update the tests as well as the migration guide too.

Alternately/additionally arrays in the parser be specified as JS objects, like this?

   foo="[1,2,3]"

We don't use arrays much in the code so I wonder if it makes sense to have a special case in the parser.

Change History (4)

comment:1 Changed 12 years ago by bill

Talked about it in today's meeting. No vociferous objections.

comment:2 Changed 12 years ago by Adam Peller

Owner: changed from bill to Adam Peller

comment:3 Changed 12 years ago by Adam Peller

Resolution: fixed
Status: newclosed

(In [9679]) Use comma separator for dijit lists. Fixes #3687

comment:4 Changed 12 years ago by Adam Peller

Resolution: fixed

(In [9688]) Allow whitespace before and after comma-separated attrs. Ported a few more ";" references. Fixes #3687

Note: See TracTickets for help on using tickets.