MT#57801 /api/phonebookentries fix get_form()

* address an issue where get_form() logic did not return
  any $form value because of the if ($params) block was
  falsely used.

Change-Id: I77f03ac7914c1b4a8f072269cf2c98b99657fa52
(cherry picked from commit c2fe092004)
mr11.3
Kirill Solomko 3 years ago
parent 33b78a4098
commit 0f70c8bb05

@ -33,24 +33,28 @@ sub _item_rs {
sub get_form {
my ($self, $c) = @_;
my $params = $c->request->query_params;
my $form;
if ($c->user->roles eq "admin") {
$form = NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::ResellerAPI", $c);
} elsif ($c->user->roles eq "reseller") {
$form = NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::ResellerAPI", $c);
} elsif ($c->user->roles eq 'subscriber' ||
$c->user->roles eq 'subscriberadmin') {
$form = NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::SubscriberAPI", $c);
}
if ($params) {
if ($params->{reseller_id}) {
return NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::ResellerAPI", $c);
$form = NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::ResellerAPI", $c);
} elsif ($params->{customer_id}) {
return NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::CustomerAPI", $c);
$form = NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::CustomerAPI", $c);
} elsif ($params->{subscriber_id}) {
return NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::SubscriberAPI", $c);
$form = NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::SubscriberAPI", $c);
}
} elsif ($c->user->roles eq "admin") {
return NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::ResellerAPI", $c);
} elsif ($c->user->roles eq "reseller") {
return NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::ResellerAPI", $c);
} elsif ($c->user->roles eq 'subscriber' ||
$c->user->roles eq 'subscriberadmin') {
return NGCP::Panel::Form::get("NGCP::Panel::Form::Phonebook::SubscriberAPI", $c);
}
return;
return $form;
}
sub process_hal_resource {

Loading…
Cancel
Save