From 50ae86b75878d2d4043d8512389df4bd573dba9b Mon Sep 17 00:00:00 2001 From: Max Carrara Date: Thu, 14 Mar 2024 11:19:30 +0100 Subject: [PATCH] api: ceph monitor: rework some code style to modern Signed-off-by: Max Carrara [ TL: also improve if-expression wrapping ] Signed-off-by: Thomas Lamprecht --- PVE/API2/Ceph/MON.pm | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/PVE/API2/Ceph/MON.pm b/PVE/API2/Ceph/MON.pm index 057ca70d..00b8f503 100644 --- a/PVE/API2/Ceph/MON.pm +++ b/PVE/API2/Ceph/MON.pm @@ -147,14 +147,15 @@ my $assert_mon_prerequisites = sub { my $assert_mon_can_remove = sub { my ($monhash, $monlist, $monid, $mondir) = @_; - if (!(defined($monhash->{$monid}) || - grep { defined($_->{name}) && $_->{name} eq $monid } @$monlist)) - { + if ( + !defined($monhash->{$monid} || + grep { defined($_->{name}) && $_->{name} eq $monid } $monlist->@*) + ) { die "no such monitor id '$monid'\n" } die "monitor filesystem '$mondir' does not exist on this node\n" if ! -d $mondir; - die "can't remove last monitor\n" if scalar(@$monlist) <= 1; + die "can't remove last monitor\n" if scalar($monlist->@*) <= 1; }; my $remove_addr_from_mon_host = sub {