Bug 6993 - Incorrect data type in auth_subfield_structure
Summary: Incorrect data type in auth_subfield_structure
Status: RESOLVED DUPLICATE of bug 22919
Alias: None
Product: Koha
Classification: Unclassified
Component: Database (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor
Assignee: Galen Charlton
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks: 17258
  Show dependency tree / graph
 
Reported: 2011-10-09 20:15 UTC by Chris Nighswonger
Modified: 2019-06-19 15:15 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Nighswonger 2011-10-09 20:15:00 UTC 
kohastructure.sql makes auth_subfield_structure.linkid a tinyint(1) NOT NULL, but then authorities_normal_marc21.sql tries to insert an null string value for that field.

From kohastructure.sql:

DROP TABLE IF EXISTS `auth_subfield_structure`;
CREATE TABLE `auth_subfield_structure` (

...

  `linkid` tinyint(1) NOT NULL default 0,

...

From authorities_normal_marc21.sql :

INSERT INTO `auth_subfield_structure` (`authtypecode`, `tagfield`, `tagsubfield`, `liblibrarian`, `libopac`, `repeatable`, `mandatory`, `tab`, `authorised_value`, `value_builder`, `seealso`, `isurl`, `hidden`, `linkid`, `kohafield`, `frameworkcode`) VALUES
		('', '000', '@', 'fixed length control field', 'fixed length control field', 0, 1, 0, NULL, 'marc21_leader_authorities.pl', NULL, 0, 0, '', '', '')

...
Comment 1 Paul Poulain 2011-10-26 14:45:12 UTC 
Bug still valid in 3.6
I think the NOT NULL is not necessary, so I propose we change to NULL
Comment 2 Owen Leonard 2014-07-10 17:32:35 UTC 
The files described here appear to remain unchanged from when this bug was reported but obviously there are not critical consequences since the bug has not been touched in years. 

Is this bug really valid? If so, is it really a critical bug?
Comment 3 Jonathan Druart 2015-03-26 16:13:45 UTC 
For information:

MariaDB [koha]> INSERT INTO `auth_subfield_structure` (`authtypecode`, `tagfield`, `tagsubfield`, `liblibrarian`, `libopac`, `repeatable`, `mandatory`, `tab`, `authorised_value`, `value_builder`, `seealso`, `isu
rl`, `hidden`, `linkid`, `kohafield`, `frameworkcode`) VALUES
    -> ('', '000', '@', 'fixed length control field', 'fixed length control field', 0, 1, 0, NULL, 'marc21_leader_authorities.pl', NULL, 0, 0, '', '', '');         
Query OK, 1 row affected, 1 warning (0.08 sec)

MariaDB [koha]> show warnings;
+---------+------+----------------------------------------------------------+
| Level   | Code | Message                                                  |
+---------+------+----------------------------------------------------------+
| Warning | 1366 | Incorrect integer value: '' for column 'linkid' at row 1 |
+---------+------+----------------------------------------------------------+
1 row in set (0.00 sec)

MariaDB [koha]> select linkid from auth_subfield_structure where tagfield='000';
+--------+
| linkid |
+--------+
|      0 |
+--------+
1 row in set (0.00 sec)
Comment 4 Jonathan Druart 2015-03-26 16:14:46 UTC 
Not critical at all, considered as minor. Feel free to change it if you disagree.
Comment 5 Jonathan Druart 2019-06-19 15:15:07 UTC 

*** This bug has been marked as a duplicate of bug 22919 ***