@@ -, +, @@ --- Koha/Schema/Result/Virtualshelve.pm | 8 ++++++++ t/db_dependent/Virtualshelves.t | 2 +- 2 files changed, 9 insertions(+), 1 deletion(-) --- a/Koha/Schema/Result/Virtualshelve.pm +++ a/Koha/Schema/Result/Virtualshelve.pm @@ -217,5 +217,13 @@ sub koha_objects_class { 'Koha::Virtualshelves'; } +__PACKAGE__->add_columns( + '+public' => { is_boolean => 1 }, +); + +__PACKAGE__->add_columns( + '+allow_change_from_staff' => { is_boolean => 1 }, +); + # You can replace this text with custom code or comments, and it will be preserved on regeneration 1; --- a/t/db_dependent/Virtualshelves.t +++ a/t/db_dependent/Virtualshelves.t @@ -327,7 +327,7 @@ subtest 'Shelf permissions' => sub { is( $public_shelf->can_be_deleted( $patron3->{borrowernumber} ), 0, 'Public list should not be deleted by someone with no special permissions' ); is( $public_shelf->can_be_deleted( $patron4->{borrowernumber} ), 0, 'Public list should not be deleted by someone with the edit_public_lists sub-permission checked' ); - is( $public_shelf->can_be_managed( $patron1->{borrowernumber} ), 1, 'The owner should be able to manage thier list' ); + is( $public_shelf->can_be_managed( $patron1->{borrowernumber} ), 1, 'The owner should be able to manage their list' ); is( $public_shelf->can_be_managed( $patron2->{borrowernumber} ), 0, 'Public list should not be managed by another staff member' ); is( $public_shelf->can_be_managed( $patron3->{borrowernumber} ), 0, 'Public list should not be managed by someone with no special permissions' ); is( $public_shelf->can_be_managed( $patron4->{borrowernumber} ), 1, 'Public list should be managed by someone with the edit_public_lists sub-permission checked' ); --