Bug 18577 - Importing a batch using a framework not fully set up causes and endless...
authorNick Clemens <nick@bywatersolutions.com>
Fri, 23 Jun 2017 14:11:44 +0000 (14:11 +0000)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 10 Jul 2017 20:55:19 +0000 (17:55 -0300)
To test:
1 - Create a new framework - don't set it up
2 - Stage some records for import
3 - Manage the import, note your new framwork is in the list
4 - Apply patch
5 - Reload the page
6 - Note the framework is no longer an option

Followed test plan,  worked as intended
Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Koha/Schema/Result/BiblioFramework.pm
tools/manage-marc-import.pl

index 36c223b..3e7b70d 100644 (file)
@@ -62,6 +62,12 @@ __PACKAGE__->set_primary_key("frameworkcode");
 # Created by DBIx::Class::Schema::Loader v0.07025 @ 2013-10-14 20:56:21
 # DO NOT MODIFY THIS OR ANYTHING ABOVE! md5sum:KY1w7J/5cBsz9VV7QEBKPw
 
+__PACKAGE__->has_many(
+    "marc_tag_structure",
+    "Koha::Schema::Result::MarcTagStructure",
+    { "foreign.frameworkcode" => "self.frameworkcode"},
+    { cascade_copy => 0, cascade_delete => 0 },
+);
 
 # You can replace this text with custom content, and it will be preserved on regeneration
 1;
index 892ffe8..55707aa 100755 (executable)
@@ -63,7 +63,7 @@ my %cookies = parse CGI::Cookie($cookie);
 our $sessionID = $cookies{'CGISESSID'}->value;
 our $dbh = C4::Context->dbh;
 
-my $frameworks = Koha::BiblioFrameworks->search({}, { order_by => ['frameworktext'] });
+my $frameworks = Koha::BiblioFrameworks->search({ tagfield => { 'not' => undef } }, { join => 'marc_tag_structure', group_by=>'frameworkcode',order_by => ['frameworktext'] });
 $template->param( frameworks => $frameworks );
 
 if ($op eq "create_labels") {