Opened 8 years ago

Closed 8 years ago

#13723 closed defect (fixed)

dojo mvc tests LoanWizardModel minor fix

Reported by: luch Owned by: Ed Chatelain
Priority: low Milestone: 1.8
Component: DojoX MVC Version:
Keywords: Cc: rahul
Blocked By: Blocking:

Description

Minor fix to tests/models/LoanWizardModel.js

Attachments (2)

LoanWizardModel.patch​ (489 bytes) - added by luch 8 years ago.
mvc-13723.patch​ (564 bytes) - added by Ed Chatelain 8 years ago.
Update the test file.

Download all attachments as: .zip

Change History (8)

Changed 8 years ago by luch

Attachment: LoanWizardModel.patch​ added

comment:1 Changed 8 years ago by ben hockey

actually... constructors are already chained so that line should be removed. also arguments, rather than args, is actually right so the format of the call was right to begin with but its just not needed.

comment:2 Changed 8 years ago by Adam Peller

Cc: rahul added
Owner: changed from rahul to Ed Chatelain

comment:3 Changed 8 years ago by Ed Chatelain

Resolution: → invalid
Status: new → closed

comment:4 Changed 8 years ago by ben hockey

ed, you should remove the line. it's wrong (not needed). constructors are already chained.

comment:5 Changed 8 years ago by Ed Chatelain

Resolution: invalid
Status: closed → reopened

ok, will do.

Changed 8 years ago by Ed Chatelain

Attachment: mvc-13723.patch​ added

Update the test file.

comment:6 Changed 8 years ago by cjolif

Resolution: → fixed
Status: reopened → closed

In [27743]:

fixes #13723. Thanks edchat.

Note: See TracTickets for help on using tickets.