The following warnings occurred: | ||||||||||||
Warning [2] Undefined variable $unreadreports - Line: 26 - File: global.php(961) : eval()'d code PHP 8.2.25 (Linux)
|
admin_user delete bug - Printable Version +- UserSpice (https://userspice.com/forums) +-- Forum: Support Center (https://userspice.com/forums/forumdisplay.php?fid=23) +--- Forum: UserSpice 4.3 and Below (https://userspice.com/forums/forumdisplay.php?fid=26) +--- Thread: admin_user delete bug (/showthread.php?tid=238) |
admin_user delete bug - plb - 09-04-2016 When I go to users/admin_user.php and mark a user to delete, I get an error message that $validate variable doesn't exist. I have patched the code - here's a copy on pastebin: http://pastebin.com/YQzNN58E While I was in there I fixed some indentation issues just for presentation of the code - makes it more readable and standard with the rest of US. (I'd like to put a little border around the "create new user" section, but I'm still working on what CSS from bootstrap looks best there - there may be a second version in a bit...) admin_user delete bug - plb - 09-04-2016 I edited the pastebin with the new class - I put that section of the form in a "well well-sm" and I think it looks a lot nicer, setting off that section of the form from the rest of it. admin_user delete bug - mudmin - 09-06-2016 Thanks for that. That's one of those things that doesn't come up on most systems, but apparently it comes up sometimes. Your code looks good though. I'll check it on mine. I'm compiling a lot of stuff for the next update. admin_user delete bug - plb - 09-06-2016 I like having my error reporting turned up pretty verbosely as I find it helps me output cleaner code. That's probably why it came up on my system. admin_user delete bug - mudmin - 09-06-2016 Yep. Makes sense. admin_user delete bug - mudmin - 09-11-2016 For anyone looking at this in the future, this is admin_users.php not admin_user and the patch was added in 4.1.5 |