And TestBuilder.t is failing because of this.
Created attachment 143622 [details] [review] Bug 32162: Add primar key to erm_eholdings_packages_agreements Test plan: Confirm that this change fixes the related TestBuilder.t failure Confirm that it does not break the eRM app, try to attach an agreement to a package.
Created attachment 143645 [details] [review] Bug 32162: Add primar key to erm_eholdings_packages_agreements Test plan: Confirm that this change fixes the related TestBuilder.t failure Confirm that it does not break the eRM app, try to attach an agreement to a package.
Created attachment 143682 [details] [review] Bug 32162: Add primar key to erm_eholdings_packages_agreements Test plan: Confirm that this change fixes the related TestBuilder.t failure Confirm that it does not break the eRM app, try to attach an agreement to a package. Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Created attachment 143771 [details] [review] Bug 32162: Add primar key to erm_eholdings_packages_agreements Test plan: Confirm that this change fixes the related TestBuilder.t failure Confirm that it does not break the eRM app, try to attach an agreement to a package. Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Trivial correction, works as designed, no regressions. Passing QA
Pushed to master for 22.11. Nice work everyone, thanks!
Cannot recreate in dex system, but saw error in upgrading an install from 22.05 to 22.11: Upgrade to 22.06.00.084 [08:06:49]: Bug 31162 - Add primary key to erm_eholdings_packages_agreements ERROR - {UNKNOWN}: DBI Exception: DBD::mysql::db do failed: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'CONSTRAINT erm_eholdings_packages_agreements_uniq, ADD PRIMARY K' at line 3 at /usr/share/koha/lib/C4/Installer.pm line 739 Bug number also wrong in the update file
What's the version of the DBMS, Nick?
(In reply to Jonathan Druart from comment #8) > What's the version of the DBMS, Nick? Server version: 5.7.12 MySQL Community Server (GPL)
(In reply to Nick Clemens from comment #9) > (In reply to Jonathan Druart from comment #8) > > What's the version of the DBMS, Nick? > > Server version: 5.7.12 MySQL Community Server (GPL) If this is a MySQL 5.7 thing, are we still supporting it? See also bug 29805.
(In reply to Jonathan Druart from comment #10) > (In reply to Nick Clemens from comment #9) > > (In reply to Jonathan Druart from comment #8) > > > What's the version of the DBMS, Nick? > > > > Server version: 5.7.12 MySQL Community Server (GPL) > > If this is a MySQL 5.7 thing, are we still supporting it? > > See also bug 29805. See bug 35338.
(In reply to Nick Clemens from comment #7) > Bug number also wrong in the update file Oops, fixed on bug 35698.