Skip to content

Commit

Permalink
bugfix for regression in change upload target selector behavior
Browse files Browse the repository at this point in the history
  • Loading branch information
jebeck committed Jun 1, 2015
1 parent d3dee47 commit 498931b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions lib/state/appActions.js
Original file line number Diff line number Diff line change
Expand Up @@ -141,9 +141,7 @@ appActions.bindApp = function(app) {
return this;
};

appActions.changeGroup = function(e) {
var userid = e.target.value;

appActions.changeGroup = function(userid) {
// reset upload state when changing group
for(var i in this.app.state.uploads) {
appActions.reset(i);
Expand Down
2 changes: 1 addition & 1 deletion test/ui/testAppActions.js
Original file line number Diff line number Diff line change
Expand Up @@ -833,7 +833,7 @@ describe('appActions', function() {

it('updates user id for uploading', function() {
app.state.targetId = 'foo';
appActions.changeGroup({target: {value: 'bar'}});
appActions.changeGroup('bar');
expect(app.state.targetId).to.equal('bar');
});

Expand Down

0 comments on commit 498931b

Please sign in to comment.