LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

fix duplicated history bug

Browse source
This commit is contained in:
Baltazar Gomez 2017-08-26 20:12:45 -06:00 • committed by GitHub
parent fb2b37c4f1
commit b3c4f4369e
1 changed files with 5 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
ui/js/reducers/app.js
View file

@ -172,27 +172,19 @@ reducers[types.WINDOW_FOCUSED] = function(state, action) {
};
reducers[types.HISTORY_NAVIGATE] = (state, action) => {
let page = false;
let location = false;
// Get history from state
const { history } = state;
if (action.data.page) {
// Get page
page = action.data.page;
} else if (action.data.location) {
// Get new location
location = action.data.location;
}
let { location, page } = action.data;
// Add new location to stack
if (location) {
const lastItem = history.stack.length - 1;
const lastIndex = history.stack.length - 1;
const lastPage = history.stack[lastIndex].location;
// Check for duplicated
let is_duplicate = lastItem > -1
? history.stack[lastItem].location === location
let is_duplicate = lastIndex > -1
? lastPage.replace(/[?]$/, "") === location.replace(/[?]$/, "")
: false;
if (!is_duplicate) {