[Bug 59375] New: Filtering applications by Wine version results in 'Uncaught mysqli_sql_exception'
http://bugs.winehq.org/show_bug.cgi?id=59375 Bug ID: 59375 Summary: Filtering applications by Wine version results in 'Uncaught mysqli_sql_exception' Product: WineHQ Apps Database Version: unspecified Hardware: x86-64 OS: Linux Status: NEW Severity: normal Priority: P2 Component: appdb-unknown Assignee: wine-bugs@list.winehq.org Reporter: gyebro69@gmail.com Distribution: --- Created attachment 80325 --> http://bugs.winehq.org/attachment.cgi?id=80325 error message This used to work many months (maybe a 1 year) ago. To reproduce: click <Browse Apps>, select a Wine version from the drop-down list then click on <Update filter> button. I got 'Ooops! Something has gone terribly wrong!' and a long error message. I'm using Firefox 147.0.3 currently, but this has been an issue for quite a while with previous Firefox versions too. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59375 Ken Sharp <imwellcushtymelike@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |imwellcushtymelike@gmail.co | |m --- Comment #1 from Ken Sharp <imwellcushtymelike@gmail.com> --- get_bugzilla_versions() in util.php seems to be the function doing the work, but does versions.id exist in the Bugzilla DB, or was it renamed? I can find only three mentions of versions.id in the AppDB code, though there were more before the split. I can't find anything in the Bugzilla code. I don't know why. Probably user error. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59375 Stian Low <wineryyyyy@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |wineryyyyy@gmail.com --- Comment #2 from Stian Low <wineryyyyy@gmail.com> --- Apologies for missing this which seems caused by my merge to fix traffic issues by allowing Bugzilla and AppDB to run on separate servers: - https://bugs.winehq.org/show_bug.cgi?id=56706 (In reply to Ken Sharp from comment #1)
but does versions.id exist in the Bugzilla DB, or was it renamed?
versions.id refers to Bugzilla DB table which no longer exists on AppDB server so the field name needs to be corrected since it no longer applies for queries. My merge changed AppDB to instead retrieve Wine versions updates from Bugzilla server via periodic HTTP requests. I will push a fix for AppDB wine version filters soon. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59375 --- Comment #3 from Stian Low <wineryyyyy@gmail.com> --- Created attachment 81594 --> http://bugs.winehq.org/attachment.cgi?id=81594 Screenshot Browse Apps wine version filter fixed for my localhost instance Pending merge should fix: https://gitlab.winehq.org/winehq/appdb/-/merge_requests/34 Hopefully this wraps up remaining lingering bugs caused by my merge a year ago to support separating AppDB from Bugzilla server for traffic improvements. Sorry again for missing this and inconveniences it may have caused. I came into AppDB completely green and was unaware that my changes would not go through some testing server before being committed to production. I'll be more careful knowing changes are likely to immediately affect production going forward. I'm also much more familiar with AppDB now so mistakes will hopefully be less common. I'll review appdb bugs again to make sure nothing else needs to be fixed. Please let me know if I missed anything. If something is urgent and I don't respond to Bugzilla then please ping me (wineryyyyy@gmail.com) directly to raise it to my attention and I'll address more immediately. Also please let me know if there are any channels I should be watching to elevate my awareness of issues regarding the apps that I touch. Otherwise I'll be watching and review Bugzilla appdb more closely. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59375 --- Comment #4 from Stian Low <wineryyyyy@gmail.com> --- Created attachment 81599 --> http://bugs.winehq.org/attachment.cgi?id=81599 Screenshot mysql error fixed for appdb.wine.org after MR-34 merged Please let me know if the filter does not work as it did before. Otherwise this bug may be fixed. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59375 Béla Gyebrószki <gyebro69@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED --- Comment #5 from Béla Gyebrószki <gyebro69@gmail.com> --- (In reply to Stian Low from comment #4)
Created attachment 81599 [details] Screenshot mysql error fixed for appdb.wine.org after MR-34 merged
Please let me know if the filter does not work as it did before.
Otherwise this bug may be fixed.
Works as expected as of writing. Thank you for fixing this bug. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59375 Ken Sharp <imwellcushtymelike@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |92accbd6598a94c47ef2f13c1d3 | |c41b50f7da6d2 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=59375 Ken Sharp <imwellcushtymelike@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #6 from Ken Sharp <imwellcushtymelike@gmail.com> --- Closing bugs marked as fixed. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla