Accordion widget refresh method sets active option to 0 when it was set to false

[StevenE]

Dear,

The accordion widget refresh method sets the active option to 0 if it was set to false. I had a look at the code and I think the solution is easy: just change the "if" on line 6235 to an "else if" and the problem is solved.

Kind regards, Steven Engels.

[tj.vantoll]

Using the boolean version of the active option requires you to set the collapsible option to true. If you pass {active: false} without setting collapsible, it is indeed changed to 0. But that is because false is an invalid value.

This appears to be working just fine: ​http://jsfiddle.net/tj_vantoll/RxaqD/. If you are still having issues let us know.

[StevenE]

Replying to tj.vantoll:

Using the boolean version of the active option requires you to set the collapsible option to true. If you pass {active: false} without setting collapsible, it is indeed changed to 0. But that is because false is an invalid value.

This appears to be working just fine: ​http://jsfiddle.net/tj_vantoll/RxaqD/. If you are still having issues let us know.

It seems there is a mismatch between the github version and the distributed version. jsfiddle references to the github version and there this issue has indeed been fixed by the fix I suggested. If you change the reference in jsfiddle to ​http://code.jquery.com/ui/1.10.2/jquery-ui.js you can see the problem still exists there. I updated your jsfiddle ​http://jsfiddle.net/RxaqD/2/ But if it is solved on github I assume it will get solved in the distributed version too next time it gets released. thx, for the help.

[tj.vantoll]

Replying to StevenE:

Replying to tj.vantoll:

Using the boolean version of the active option requires you to set the collapsible option to true. If you pass {active: false} without setting collapsible, it is indeed changed to 0. But that is because false is an invalid value.

This appears to be working just fine: ​http://jsfiddle.net/tj_vantoll/RxaqD/. If you are still having issues let us know.

It seems there is a mismatch between the github version and the distributed version. jsfiddle references to the github version and there this issue has indeed been fixed by the fix I suggested. If you change the reference in jsfiddle to ​http://code.jquery.com/ui/1.10.2/jquery-ui.js you can see the problem still exists there. I updated your jsfiddle ​http://jsfiddle.net/RxaqD/2/ But if it is solved on github I assume it will get solved in the distributed version too next time it gets released. thx, for the help.

Ah, yeah you're right. This is a duplicate of #9189 which was fixed in 1.10.3 and will be released soon.

[tj.vantoll]

Duplicate of #9189.