Skip to content

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

Merged
TheWitness merged 1 commit intoCacti:1.2.xfrom
xmacan:fix_ambigouous_column
Nov 12, 2023
Merged

When obtaining graph records, error messages may be recorded#5564
TheWitness merged 1 commit intoCacti:1.2.xfrom
xmacan:fix_ambigouous_column

Conversation

@xmacan
Copy link
Copy Markdown
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

@netniV
Copy link
Copy Markdown
Member

netniV commented Nov 17, 2023

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

@xmacan
Copy link
Copy Markdown
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
Copy Markdown
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