ddp/photoassistant
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fixed BUG-143, disable next/prev as first thing we do when going next/prev Page

Browse Source
This commit is contained in:
Damien De Paoli
2025-10-28 23:04:33 +11:00
parent 89e8c5d9f7
commit 0ee55ee73d
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
BUGs
View File

@@ -1,5 +1,4 @@
### Next: 146 ### Next: 146
BUG-143: if I skip next fast enough in files_ip, it can get the warning about changes to entry list... (would have though I disabled next page until loaded)
BUG-140: When db is restarted underneath PA, it crashes job mgr... It should just accept timeouts, and keep trying to reconnect every 2? mins BUG-140: When db is restarted underneath PA, it crashes job mgr... It should just accept timeouts, and keep trying to reconnect every 2? mins
BUG-118: can move files from Bin path, but it leaves the del_file entry for it - need to remove it BUG-118: can move files from Bin path, but it leaves the del_file entry for it - need to remove it
BUG-117: when search returns files that can be deleted and/or restored, the icon stays as delete and tries to delete! BUG-117: when search returns files that can be deleted and/or restored, the icon stays as delete and tries to delete!

8
internal/js/files_support.js
View File

@@ -759,6 +759,10 @@ function resetNextPrevButtons()
// get list of eids for the next page, also make sure next/prev buttons make sense for page we are on // get list of eids for the next page, also make sure next/prev buttons make sense for page we are on
function nextPage(successCallback) function nextPage(successCallback)
{ {
// start with disabling more next presses until we are ready to process them
$('.prev').prop('disabled', true).addClass('disabled');
$('.next').prop('disabled', true).addClass('disabled');
// pageList[0] is the first entry on this page // pageList[0] is the first entry on this page
const currentPage=getPageNumberForId( pageList[0] ) const currentPage=getPageNumberForId( pageList[0] )
// should never happen / just return pageList unchanged // should never happen / just return pageList unchanged
@@ -774,6 +778,10 @@ function nextPage(successCallback)
// get list of eids for the prev page, also make sure next/prev buttons make sense for page we are on // get list of eids for the prev page, also make sure next/prev buttons make sense for page we are on
function prevPage(successCallback) function prevPage(successCallback)
{ {
// start with disabling more prev presses until we are ready to process them
$('.prev').prop('disabled', true).addClass('disabled');
$('.next').prop('disabled', true).addClass('disabled');
// pageList[0] is the first entry on this page // pageList[0] is the first entry on this page
const currentPage=getPageNumberForId( pageList[0] ) const currentPage=getPageNumberForId( pageList[0] )
// should never happen / just return pageList unchanged // should never happen / just return pageList unchanged