Swap the order, but also factor out the expression used in the active set call, in cases where it has side-effects which affected the value being used in the queue set call:
@@ -260,8 +260,8 @@ window.reorder = {};
return a - b;
});
var review = queue.splice((dataset.quick === 'r' ? 10 : $.jStorage.get('l/batchSize')));
- $.jStorage.set(dataset.active, queue);
$.jStorage.set(dataset.queue, review);
+ $.jStorage.set(dataset.active, queue);
dataset.updateVisual();
} catch (err) {
showerr(err);
@@ -378,8 +378,9 @@ window.reorder = {};
$('#option-wrap-up').click(function() {
if ($(this).attr('class') === 'wrap-up-selected') {
var fullQueue = $.jStorage.get(dataset.active).concat($.jStorage.get(dataset.queue));
- $.jStorage.set(dataset.active, fullQueue.splice(0, 10));
+ var active = fullQueue.splice(0, 10);
$.jStorage.set(dataset.queue, fullQueue);
+ $.jStorage.set(dataset.active, active);
} else {
if (ordered) {
sorter.reorder();
@@ -557,8 +558,9 @@ window.reorder = {};
});
}
}
- $.jStorage.set(dataset.active, (dataset.quick === 'r' ? fullQueue : fullQueue.splice(0, $.jStorage.get('l/batchSize'))));
+ var active = (dataset.quick === 'r' ? fullQueue : fullQueue.splice(0, $.jStorage.get('l/batchSize')));
$.jStorage.set(dataset.queue, (dataset.quick === 'r' ? [] : fullQueue));
+ $.jStorage.set(dataset.active, active);
dataset.updateVisual();
},
removeLevel: function(level, list) {