src/sbbs3 listfile.cpp 1.61 1.62
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3

Modified Files:
listfile.cpp
Log Message:
Fix bug introduced in rev 1.60: when searching for a file and a single file matches the search spec, the 'D' batch-flagging command would invoke the downloadfile() method which confusingly, doesn't actually download (send)
the file, but instead performs all the post-download functions needed. So
the user would just see "<filename> sent." and possibly lose some credits without actually receiving the file.



---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/sbbs/sbbs/-/commit/a3c0f031ac017b95093a21f7
Modified Files:
src/sbbs3/listfile.cpp
Log Message:
Deleting a file's extended description would try to delete some other file too

This is a "forever bug". I guess at one time I stored each file's extended description in a separate file? I don't know, but this remove() call is
clearly wrong.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/a9214ac450a5a8d71daa8494
Modified Files:
src/sbbs3/listfile.cpp
Log Message:
Change CRLF before batch flag prompt to a conditional new-line.

If a sysop wants a *blank* line before the batch flag prompt, they can put that (or a conditional blank line, even better) in their text.dat (BatchFlagPrompt). For now, I'm thinking no blank line before the prompt is better use of the screen.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/fd649bf8c381fa6d7d4394b6
Modified Files:
src/sbbs3/listfile.cpp
Log Message:
Make it a conditional *blank* line before the batch flag prompt

It's very tricky to defeat the auto-pause to have your own custom prompt at rows-1. So going back to having a blank line before the batch prompt.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/cdedcb5a2eab52a861108026
Modified Files:
src/sbbs3/listfile.cpp
Log Message:
Use correct string for file path when removing file for non dir-ops

Fix issue reported by Jay Crutti.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/caed3600410b1a8a74a68c95
Modified Files:
src/sbbs3/listfile.cpp
Log Message:
Simplify the file remove/delete logic here.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/be0c953e10cb6d3485093400
Modified Files:
src/sbbs3/listfile.cpp
Log Message:
Don't attempt to remove files that have already been removed

This is the likely cause of Phil's reported errors when removing files: access=-110 info=smb_removefile name not found

The only way I was able to reproduce these errors was to attempt to remove the same file multiple times in the same file listing. In other worse, use 'R', remove a file, then 'P'revious back to the first file and try to remove it again. Or to remove it a file from the batch flag prompt and then attempt to remove it again. <shrug>

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/df49b43613693fd8b710972c
Modified Files:
src/sbbs3/listfile.cpp
Log Message:
Beautify the file list files (e.g. NEWFILES.TXT) a bit

Use the file size estimates rather than actual byte values (to keep lines < 80 chars).
Use file name file summary/description is blank or doesn't exist.

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/9fcce522c52d7972f15b9fc8
Modified Files:
src/sbbs3/listfile.cpp
Log Message:
Create (export to text) file lists faster

- Don't read/load extended descriptions
- Don't query each file size (use file cost from database)

---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net