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

When obtaining graph records, error messages may be recorded #5564

Merged
merged 1 commit into from
Nov 12, 2023
Merged

When obtaining graph records, error messages may be recorded #5564

merged 1 commit into from
Nov 12, 2023

Conversation

xmacan
Copy link
Member

@xmacan xmacan commented Nov 12, 2023

2023-11-12 21:34:13 - CMDPHP SQL Backtrace: (/host.php[176]:host(), /host.php[1797]:get_device_records(), /host.php[1498]:db_fetch_assoc(), /lib/database.php[684]:db_fetch_assoc_prepared(), /lib/database.php[704]:db_execute_prepared())
2023-11-12 21:34:13 - CMDPHP ERROR: A DB Row Failed!, Error: Column 'graphs' in field list is ambiguous

@TheWitness TheWitness merged commit 6b737c7 into Cacti:1.2.x Nov 12, 2023
6 checks passed
@netniV
Copy link
Member

netniV commented Nov 17, 2023

Was this fix for a current release bug, or was the CHANGELOG not updated?

@xmacan
Copy link
Member Author

xmacan commented Nov 17, 2023

I didn't find this in open issues. This problem was there long time, maybe it only shows up on some versions of mysql. I forgot update CHANGELOG

@TheWitness
Copy link
Member

This is more a problem when a user upgrades to develop and then goes back to 1.2.x using the develop database. So, it was a good catch overall, mainly to keep us from answering a bunch of questions after the fact.

@netniV netniV changed the title fix SQL query - Column 'graphs' in field list is ambiguous When obtaining graph records, error messages may be recorded Dec 16, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Mar 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants