Opened 10 years ago

Closed 10 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 10 years ago.
mvc-13723.patch​ (564 bytes) - added by Ed Chatelain 10 years ago.
Update the test file.

Download all attachments as: .zip

Change History (8)

Changed 10 years ago by luch

Attachment: LoanWizardModel.patch​ added

comment:1 Changed 10 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 10 years ago by Adam Peller

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

comment:3 Changed 10 years ago by Ed Chatelain

Resolution: → invalid
Status: new → closed

comment:4 Changed 10 years ago by ben hockey

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

comment:5 Changed 10 years ago by Ed Chatelain

Resolution: invalid
Status: closed → reopened

ok, will do.

Changed 10 years ago by Ed Chatelain

Attachment: mvc-13723.patch​ added

Update the test file.

comment:6 Changed 10 years ago by cjolif

Resolution: → fixed
Status: reopened → closed

In [27743]:

fixes #13723. Thanks edchat.

Note: See TracTickets for help on using tickets.