https://gitlab.synchro.net/main/sbbs/-/commit/a53829d34393689015add034
Modified Files:
src/sbbs3/useredit.cpp
Log Message:
Stop beeping at the sysop when user-searches are successful.

If anything, I suppose would be beep if a search fails, but really, I think beeps are kind of annoying these days. Not changing the currently selected/viewed user is likely all that's really needed to indicate a search failure.

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

https://gitlab.synchro.net/main/sbbs/-/commit/d8c36d9d898830fa0b8456f9
Modified Files:
src/sbbs3/useredit.cpp
Log Message:
Remove unnecessary current user (co-sysop) level/flag checks

As Andre pointed out, these checks perform no function because a user with a level lower than the user being edited cannot enter the related command-key anyway.

This was just effectively dead code that was held-over from ancient SBBS days, seemingly before I learned to effectively use the || operator:
if(!(atoi(str)>useron.level && console&CON_R_INPUT))

:-)

Fixes issue #361

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

https://gitlab.synchro.net/main/sbbs/-/commit/c7eb313c0a749c22975e5a37
Modified Files:
src/sbbs3/useredit.cpp
Log Message:
Don't assign to unused variable

CID 433272

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

https://gitlab.synchro.net/main/sbbs/-/commit/d83001e8cff767fcd80261d6
Modified Files:
src/sbbs3/useredit.cpp
Log Message:
Better handling of Quit/Ctrl-C at default protocol selection

IF user hits 'Q' (or whatever the "Quit" key is), set the default protocol field in the user record to " " (instead of an empty string).
If user hits abort (Ctrl-C), don't make any change to the default protocol.

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

https://gitlab.synchro.net/main/sbbs/-/commit/8d60770fa44d0ea8dd09d48b
Modified Files:
src/sbbs3/useredit.cpp
Log Message:
Change external editor yes/no prompt default to match current user setting

Also, although not a bug (because we re-read/parse the user's record every
menu cycle), don't decrement user.xedit before calling uselect()
- just not a good practice to not modify variables unnecessarily.
See the corresponding change to exec.cpp, which was a bug.

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

https://gitlab.synchro.net/main/sbbs/-/commit/d71cce5ffce2d856b8b0f783
Modified Files:
src/sbbs3/useredit.cpp
Log Message:
Clear line counter before starting the user settings/config menu.

Fixes unnecessary [Hit a key] prompt.

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

https://gitlab.synchro.net/main/sbbs/-/commit/0bd7a0af71cf7e7b13edc6bc
Modified Files:
src/sbbs3/useredit.cpp
Log Message:
Fix CID 32913

getkeys() could return -1 if user disconnects (and SS_ABORT not set), so this appears to be a valid bug.

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