From 2b22d9cca8bc574f39dd7d18e4c5b68453fdb956 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 5 Feb 2018 17:48:03 -0300 Subject: [PATCH] Bug 20144: [sql_modes] Add default value for subscription_numberpatterns.description in tests Fix for: Field 'description' doesn't have a default value t/db_dependent/AdditionalField.t Signed-off-by: Josef Moravec Signed-off-by: Julian Maurice --- C4/Serials/Numberpattern.pm | 1 + t/db_dependent/AdditionalField.t | 1 + 2 files changed, 2 insertions(+) diff --git a/C4/Serials/Numberpattern.pm b/C4/Serials/Numberpattern.pm index 210b8dd487..3c35b628dc 100644 --- a/C4/Serials/Numberpattern.pm +++ b/C4/Serials/Numberpattern.pm @@ -139,6 +139,7 @@ sub AddSubscriptionNumberpattern { return; } + # FIXME label, description and numberingmethod must be mandatory my @keys; my @values; foreach (qw/ label description numberingmethod displayorder diff --git a/t/db_dependent/AdditionalField.t b/t/db_dependent/AdditionalField.t index 59620b8a44..93121db687 100644 --- a/t/db_dependent/AdditionalField.t +++ b/t/db_dependent/AdditionalField.t @@ -125,6 +125,7 @@ my $budget_id = AddBudget({ my $frequency_id = AddSubscriptionFrequency({ description => "Test frequency 1" }); my $pattern_id = AddSubscriptionNumberpattern({ label => 'Test numberpattern 1', + description => 'Description for numberpattern 1', numberingmethod => '{X}' }); -- 2.14.2