[Bug 56611] New: Adding notes fails
https://bugs.winehq.org/show_bug.cgi?id=56611 Bug ID: 56611 Summary: Adding notes fails Product: WineHQ Apps Database Version: unspecified Hardware: x86-64 OS: Linux Status: NEW Severity: normal Priority: P2 Component: appdb-unknown Assignee: wine-bugs(a)winehq.org Reporter: imwellcushtymelike(a)gmail.com Distribution: --- Created attachment 76366 --> https://bugs.winehq.org/attachment.cgi?id=76366 Screenshot When trying to add a note in the AppDB: “Ooops! Something has gone terribly wrong! Our monkey train has derailed! Worry not, a webmaster gopher help army has been dispatched and is on the way. If this error continues to be a problem, please report it to us on our Forums error details: Error Message: Database Error: Field 'linkedWith' doesn't have a default value Comment: File: query.php:149” Haven't added a note for a while so don't know when it started, but it's been at least 12 hours. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 --- Comment #1 from Ken Sharp <imwellcushtymelike(a)gmail.com> --- Similar when attempting to merge versions ("Move child objects"): “error details: Error Message: Database Error: Data truncated for column 'usedWorkaround' at row 1 Comment: File: query.php:149” -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 Ken Sharp <imwellcushtymelike(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Adding notes fails |Database errors in | |query.php -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 Ken Sharp <imwellcushtymelike(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Database errors in |AppDB reporting database |query.php |errors -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 Ken Sharp <imwellcushtymelike(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |49677 -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 --- Comment #2 from Ken Sharp <imwellcushtymelike(a)gmail.com> --- Also occurs when submitting a new application, but there are new applications in the queue so it must works sometimes. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 Rosanne DiMesio <dimesio(a)earthlink.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jnewman(a)codeweavers.com -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 --- Comment #3 from Jeremy Newman <jnewman(a)codeweavers.com> --- Pushed a commit: https://gitlab.winehq.org/winehq/appdb/-/commit/57cdb89927c69b4ca58a9de9d99d... This hopefully should fix the linkedWidth issue. For the usedWorkaround one, I think we can fix it by changing the table structure so the field has a default value. It is an enum with 'Yes' and 'No' as valid values. I might also need to update the code so it does not allow any other values. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 --- Comment #4 from Jeremy Newman <jnewman(a)codeweavers.com> --- patched testData.php create() to only allow defined values on usedWorkaround https://gitlab.winehq.org/winehq/appdb/-/commit/23b4c160942254f018566498748c... -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 --- Comment #5 from Ken Sharp <imwellcushtymelike(a)gmail.com> --- I thought the failure might be all down to the same issue but I guess not. I'll have to try and find where else it was failing. Right now it's down altogether (for me anyway). -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 --- Comment #6 from Ken Sharp <imwellcushtymelike(a)gmail.com> --- When merging versions: error details: Error Message: Database Error: Data truncated for column 'gpuMfr' at row 1 Comment: File: query.php:149 -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 --- Comment #7 from Ken Sharp <imwellcushtymelike(a)gmail.com> --- This one came as a surprise. When trying to add a download URL to VS 2022: error details: Error Message: Database Error: Field 'testedVersion' doesn't have a default value Comment: File: query.php:149 https://appdb.winehq.org/objectManager.php?sClass=version&iId=42073 -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 --- Comment #8 from Jeremy Newman <jnewman(a)codeweavers.com> --- These errors are coming from the move to a new MariaDB DB instance. This has a setting GLOBAL.sql_mode STRICT_TRANS_TABLES that is enabled compared to our old host. This causes the 'field X doesn't have a default value' errors and others. Normally I would just disable that setting, but the DB provider does not let me change those settings directly from their UI. I have filed a ticket with them to see if there is another way to do it, but I imagine they may say no. So the only other alternative is to continue patching the AppDB database and code so it works with this. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 Jeremy Newman <jnewman(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #9 from Jeremy Newman <jnewman(a)codeweavers.com> --- I figured out how to disable STRICT_TRANS_TABLES on the VPS mysql instance. This should resolve this issue. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 --- Comment #10 from Ken Sharp <imwellcushtymelike(a)gmail.com> --- Everything I tested looks good! -- 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.
https://bugs.winehq.org/show_bug.cgi?id=56611 Jeremy Newman <jnewman(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #11 from Jeremy Newman <jnewman(a)codeweavers.com> --- Closing. -- 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