Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fatal error when deleting logs #4830

Closed
Jimmi08 opened this issue Jul 28, 2022 · 1 comment · Fixed by #4831
Closed

Fatal error when deleting logs #4830

Jimmi08 opened this issue Jul 28, 2022 · 1 comment · Fixed by #4831
Labels
type: bug A problem that should not be happening

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented Jul 28, 2022 •
edited by Deltik
Loading

Bug Description

Fatal error: Uncaught Error: Object of class PDOStatement could not be converted to string in C:\xampp\htdocs\e107-2_3_3_git\e107_admin\admin_log.php:406
Stack trace:
 #0 C:\xampp\htdocs\e107-2_3_3_git\e107_admin\admin_log.php(276): admin_log_ui->maintenanceProcess()
 #1 C:\xampp\htdocs\e107-2_3_3_git\e107_handlers\admin_ui.php(2354): admin_log_ui->maintenancePage()
 #2 C:\xampp\htdocs\e107-2_3_3_git\e107_handlers\admin_ui.php(1375): e_admin_controller->dispatchPage()
 #3 C:\xampp\htdocs\e107-2_3_3_git\e107_admin\admin_log.php(864): e_admin_dispatcher->runPage()
 #4 {main} thrown in C:\xampp\htdocs\e107-2_3_3_git\e107_admin\admin_log.php on line 406

How to Reproduce

Steps to reproduce the behavior:

  1. Go to Logs
  2. Click on Options
  3. Click on Delete old entries (any button)

Expected Behavior

Screenshots

image

Server Information

PHP Operating System

Windows NT LAPTOP-C59ADQ1N 10.0 build 19044 (Windows 10) AMD64

PHP Version

PHP  8.1.4
@Jimmi08 Jimmi08 added the type: bug A problem that should not be happening label Jul 28, 2022
@tgtje
Copy link
Contributor

tgtje commented Jul 28, 2022

Just confirmation php 8.1.6 2.3.3 Win 11 Xampp

Deltik added a commit to Deltik/e107 that referenced this issue Jul 28, 2022
@Deltik
e107inc#4830: Sensible no delete log in `admin_log_ui::maintenancePro…
62b5c84 
…cess()`

Fixes: e107inc#4830
@Deltik Deltik mentioned this issue Jul 28, 2022
Merged
10 tasks
CaMer0n added a commit that referenced this issue Sep 29, 2022
@CaMer0n
Merge pull request #4831 from Deltik/fix/4830
2de2336 
Fixes #4830: Sensible no delete log in `admin_log_ui::maintenanceProcess()`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug A problem that should not be happening
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#4830: Sensible no delete log in admin_log_ui::maintenanceProcess() Deltik/e107
2 participants
@tgtje @Jimmi08