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

Remove deprecated Smarty elements #112

Open
cmaudoux opened this issue Jan 8, 2024 · 4 comments
Open

Remove deprecated Smarty elements #112

cmaudoux opened this issue Jan 8, 2024 · 4 comments
Assignees
@abpai94
Labels
enhancement New feature or request sponsor Developpement sponsorized by customers
Milestone
Backlog

Comments

@cmaudoux
Copy link
Contributor

cmaudoux commented Jan 8, 2024

Hi,

When enabling debug logs, lots of messages linked to warning and deprecated features are displayed polluting the screen.
image

Furthermore, I had to disable background to be able to read logs.
It could be nice to automaticaly remove the background when debug is set to True.

Cheers
@coudot coudot added the question Further information is requested label Jan 8, 2024
@coudot
Copy link
Member

coudot commented Jan 8, 2024

Hello @cmaudoux

you should configure your PHP settings to not display errors on page but on the error log file: https://www.php.net/manual/fr/errorfunc.configuration.php#ini.display-errors

@cmaudoux
Copy link
Contributor Author

cmaudoux commented Jan 8, 2024

Hi @coudot

Thx for your reply.
But the deprecated features and warning are still existing...

@coudot
Copy link
Member

coudot commented Jan 8, 2024

When debug is on, all messages are dumped into error_log. What is your need?

@cmaudoux
Copy link
Contributor Author

cmaudoux commented Jan 8, 2024

Fix the deprecated features

@coudot coudot added enhancement New feature or request sponsor Developpement sponsorized by customers and removed question Further information is requested labels Jul 12, 2024
@coudot coudot changed the title Improve smarty debug log Remove deprecated Smarty elements Jul 12, 2024
@coudot coudot added this to the Backlog milestone Jul 19, 2024
abpai94 added a commit to Worteks/service-desk that referenced this issue Jul 23, 2024
@abpai94
ltb-project#112: Upgraded to smarty4. Added configuration to ignore s…
252cf89 
…marty4 php warnings. Updated documentation.
@abpai94 abpai94 mentioned this issue Jul 23, 2024
Closed
@abpai94 abpai94 self-assigned this Jul 23, 2024
@coudot coudot linked a pull request Jul 26, 2024 that will close this issue
#112: Fixing deprecated error messages by upgrading to smarty4. #151
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abpai94 abpai94

Labels
enhancement New feature or request sponsor Developpement sponsorized by customers
Projects
None yet
Milestone
Backlog
Development

Successfully merging a pull request may close this issue.

#112: Fixing deprecated error messages by upgrading to smarty4. Worteks/service-desk
3 participants
@coudot @abpai94 @cmaudoux