gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 28207: Fix biblio_framework.frameworktext isn't in GROUP BY 

When a new framework is created and "Marc structure" is opened, a SQL
error happened if sql_strict_mode is on.

Test plan:
Create a 2 new biblio frameworks
Click "Marc structure" for one of them
=> You see the list of all frameworks with an existing structure

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
(cherry picked from commit f3b28bfe44)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
Jonathan Druart 2021-04-26 10:49:12 +02:00 committed by Fridolin Somers
parent 0216a66aca
commit 65064e1cdb
2 changed files with 12 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
admin/marctagstructure.pl
View File

@ -187,18 +187,16 @@ if ($op eq 'add_form') {
################## ITEMTYPE_CREATE ##################################
# called automatically if an unexisting frameworkis selected
} elsif ($op eq 'framework_create') {
$sth = $dbh->prepare("select count(*),marc_tag_structure.frameworkcode,frameworktext from marc_tag_structure,biblio_framework where biblio_framework.frameworkcode=marc_tag_structure.frameworkcode group by marc_tag_structure.frameworkcode");
$sth->execute;
my @existingframeworkloop;
while (my ($tot,$thisframeworkcode,$frameworktext) = $sth->fetchrow) {
if ($tot>0) {
push @existingframeworkloop, {
value => $thisframeworkcode,
frameworktext => $frameworktext,
};
}
}
$template->param( existingframeworkloop => \@existingframeworkloop );
my $frameworks = Koha::BiblioFrameworks->search(
{
'marc_tag_structure.frameworkcode' => { '!=' => undef }
},
{
join => 'marc_tag_structure',
distinct => 1
}
);
$template->param( existing_frameworks => $frameworks );
################## DEFAULT ##################################
} else { # DEFAULT

4
koha-tmpl/intranet-tmpl/prog/en/modules/admin/marctagstructure.tt
View File

@ -147,8 +147,8 @@
Create framework for [% framework.frameworkcode | html %] ([% framework.frameworktext | html %]) using
<select name="existingframeworkcode">
<option value="">Default</option>
[% FOREACH existingframeworkloo IN existingframeworkloop %]
<option value="[% existingframeworkloo.value | html %]">[% existingframeworkloo.frameworktext | html %]</option>
[% FOREACH fw IN existing_frameworks%]
<option value="[% fw.frameworkcode | html %]">[% fw.frameworktext | html %]</option>
[% END %]
</select>
<input type="submit" value="OK" />