| Summary: | Field stockrotationrotas.description should be NOT NULL, title UNIQUE | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Marcel de Rooy <m.de.rooy> |
| Component: | Database | Assignee: | Marcel de Rooy <m.de.rooy> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | kyle, martin.renvoize, tomascohen, victor |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | Trivial patch |
| Documentation contact: | Martin Renvoize | Documentation submission: | |
| Text to go in the release notes: | Version(s) released in: |
22.05.00,21.11.06
|
|
| Circulation function: | |||
| Bug Depends on: | |||
| Bug Blocks: | 30486 | ||
| Attachments: |
Bug 30565: Correct kohastructure
Bug 30565: Database revision for stockrotationrotas Bug 30565: DBIx schema change for stockrotationrotas Bug 30565: Correct kohastructure Bug 30565: Database revision for stockrotationrotas Bug 30565: DBIx schema change for stockrotationrotas |
||
|
Description
Marcel de Rooy
2022-04-19 14:48:02 UTC
Compare this $DBversion = '18.06.00.040'; CONSTRAINT `stockrotationrotas_title` UNIQUE (`title`) EN `title` varchar(100) NOT NULL, -- Title for this rota $DBversion = '18.06.00.048'; $dbh->do( "ALTER TABLE stockrotationrotas CHANGE COLUMN description description text" ); #Bug 21682 with kohastructure.sql. Created attachment 133457 [details] [review] Bug 30565: Correct kohastructure The unique key was added in a dbrev and is referred to in code. But forgotten in kohastructure. Test plan: Run new install or kohastructure.sql on empty db. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 133458 [details] [review] Bug 30565: Database revision for stockrotationrotas Add unique key if needed. Adjust description column to make it NOT NULL. Test plan: (Optionally:) Drop unique key, make description nullable. Run dbrev. Check if unique key present, and description not-nullable. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 133459 [details] [review] Bug 30565: DBIx schema change for stockrotationrotas No test plan. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 133766 [details] [review] Bug 30565: Correct kohastructure The unique key was added in a dbrev and is referred to in code. But forgotten in kohastructure. Test plan: Run new install or kohastructure.sql on empty db. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Created attachment 133767 [details] [review] Bug 30565: Database revision for stockrotationrotas Add unique key if needed. Adjust description column to make it NOT NULL. Test plan: (Optionally:) Drop unique key, make description nullable. Run dbrev. Check if unique key present, and description not-nullable. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Created attachment 133768 [details] [review] Bug 30565: DBIx schema change for stockrotationrotas No test plan. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Trivial fix, straight QA for me. Great work, Marcel! Pushed to master for 22.05, thanks to everybody involved [U+1F984] Pushed to 21.11.x for 21.11.06 Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed. Architectural fix, no documentation changes required. |