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.

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

Custom themes may cause errors if they do not contain all required CSS/JS files #5041

Closed
jdcoats opened this issue Dec 5, 2022 · 8 comments
Labels
3rd Party Bug 3rd party bug bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Milestone

Comments

@jdcoats
Copy link

jdcoats commented Dec 5, 2022

What is it that changed and what do i need to do?

2022/12/05 12:49:46 - CMDPHP PHP ERROR WARNING Backtrace: (/host.php[174]:top_header(), /lib/functions.php[4595]:include_once(), /include/top_header.php[34]:html_common_header(), /lib/html.php[2567]:get_md5_include_css(), /lib/functions.php[6555]:get_md5_hash(), /lib/functions.php[6529]:md5_file(), CactiErrorHandler())
--
2022/12/05 12:49:46 - ERROR PHP WARNING: md5_file(/var/www/localhost/htdocs/cacti/lib/../include/themes/TMH/Diff.css): failed to open stream: No such file or directory in file: /var/www/localhost/htdocs/cacti/lib/functions.php on line: 6529
@TheWitness
Copy link
Member

TheWitness commented Dec 5, 2022

@jdcoats, just copy the Diff.css from the closest theme. We are not 100% ready for release, tree editing will be updated later today, and packages will be republished tonight maybe too, but test away.

@jdcoats
Copy link
Author

jdcoats commented Dec 5, 2022

10/4

@netniV netniV added the 3rd Party Bug 3rd party bug label Dec 5, 2022
@netniV
Copy link
Member

netniV commented Dec 5, 2022

@TheWitness I was going to close this as a third party bug, but in reality, I think we will need to have a default Diff.css to fall back upon since others may have custom themes that do not have this yet. Maybe log a warning when falling back?

@TheWitness
Copy link
Member

I would agree for any of these mandatory files.

@TheWitness
Copy link
Member

Suggesting that maybe we use the modern theme mandatory files if they're missing.

@TheWitness
Copy link
Member

We would like to likely want to debounce the error message so that it does not appear too often.

@TheWitness TheWitness changed the title md5_file error in custom theme after updating to 1.2.23 md5_file error in custom theme after updating to 1.2.23 Dec 8, 2022
TheWitness added a commit that referenced this issue Dec 8, 2022
md5_file error in custom theme after updating to 1.2.23
@TheWitness TheWitness added this to the v1.2.23 milestone Dec 8, 2022
@TheWitness TheWitness added bug Undesired behaviour resolved A fixed issue confirmed Bug is confirm by dev team labels Dec 8, 2022
@TheWitness
Copy link
Member

Okay, I've added a debounce message and an Admin Email for these file missing issues.

TheWitness added a commit that referenced this issue Dec 8, 2022
Wrong log class
TheWitness added a commit that referenced this issue Dec 8, 2022
md5_file error in custom theme after updating to 1.2.23
@TheWitness
Copy link
Member

Okay, this is resolved now.

@netniV netniV changed the title md5_file error in custom theme after updating to 1.2.23 Custom themes may cause errors if they do not contain all required CSS/JS files Dec 31, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Apr 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
3rd Party Bug 3rd party bug bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue
Projects
None yet
Development

No branches or pull requests

3 participants