php class and method exception

Question

Here is the public function "should create a virtual machine" with validation,

public function deployVirtualMachine($serviceOfferingId, $templateId, $zoneId, $account = "", $diskOfferingId = "") {

        if (empty($serviceOfferingId)) {
            throw new CloudStackClientException(sprintf(MISSING_ARGUMENT_MSG, "serviceOfferingId"), MISSING_ARGUMENT);
        }

        if (empty($templateId)) {
            throw new CloudStackClientException(sprintf(MISSING_ARGUMENT_MSG, "templateId"), MISSING_ARGUMENT);
        }

        if (empty($zoneId)) {
            throw new CloudStackClientException(sprintf(MISSING_ARGUMENT_MSG, "zoneId"), MISSING_ARGUMENT);
        }

        return $this->request("deployVirtualMachine", array(
            'serviceofferingid' => $serviceOfferingId,
            'templateid' => $templateId,
            'zoneid' => $zoneId,
            'account' => $account,
            'diskofferingid' => $diskOfferingId,
            'displayname' => $displayName,
    }

I'm trying to call this function, but I keep getting exception " $templateId" is missing. but I'm pretty sure its defined here in my array.

  $params = array(

           $serviceOfferingId => '85d06496-bb75-41fb-9358-4ab919e03fe4',
            $templateId => 'c0989cf6-2da5-11e4-a846-726c7bbb864f',
           $zoneId => '7cd483ab-5aad-458b-b5e1-0e270310f41c',
           $name => 'srv11'
 );


echo $cloudstack->deployVirtualMachine($params);

Any help would be highly appreciated

Thank you

Ayman Safadi · Accepted Answer · 2014-09-01 00:03:32Z

1

You're only passing 1 parameter, an array.

Try this instead:

$cloudstack->deployVirtualMachine('85d06496-bb75-41fb-9358-4ab919e03fe4', 'c0989cf6-2da5-11e4-a846-726c7bbb864f', '7cd483ab-5aad-458b-b5e1-0e270310f41c', 'srv11');

Your method is expecting 5 parameters (3 obligatory, 2 optional). $params is being received as $serviceOfferingId.

Either pass in the values individually, or change the signature of the method to accept a single parameter.

For example:

public function deployVirtualMachine($params) {

    if (!isset($params['serviceOfferingId'])) {
        throw new CloudStackClientException(sprintf(MISSING_ARGUMENT_MSG, "serviceOfferingId"), MISSING_ARGUMENT);
    }

    // etc...
}

One more thing, by using $serviceOfferingId as an array key, you're saying that the key should be the value of variable $serviceOfferingId. It doesn't seem that that's what you want. Use a string instead; 'serviceOfferingId'.

$serviceOfferingId = 'foo';

$params = array(
    $serviceOfferingId => 'bar'
);

Produces:

Array
(
    [foo] => bar
)

What you want is:

$params = array(
    'serviceOfferingId' => 'bar'
);

Which would produce:

Array
(
    [serviceOfferingId] => bar
)

edited Sep 1, 2014 at 0:03

answered Aug 31, 2014 at 23:50

Ayman Safadi

11.6k1 gold badge30 silver badges42 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Deano Over a year ago

Hey Ayman, I'm getting "PHP Catchable fatal error: Object of class stdClass could not be converted to string" any idea why?

Axel Amthor · Accepted Answer · 2014-08-31 23:52:39Z

1

Your method expects 5 single parameters of type string, in the method invocation you pass one single parameter of type array - what do you expect?

call it this way:

 echo $cloudstack->deployVirtualMachine('85d06496-bb75-41fb-9358-4ab919e03fe4',
           'c0989cf6-2da5-11e4-a846-726c7bbb864f',
               '7cd483ab-5aad-458b-b5e1-0e270310f41c',
                  null,
                   'srv11' );

answered Aug 31, 2014 at 23:52

Axel Amthor

11.1k1 gold badge28 silver badges45 bronze badges

Comments

Community · Accepted Answer · 2023-11-17 19:30:35Z

1

Another more weird solution would be, to pass the array $params into a closure:

      public $deployVirtualMachine = function ()
      {
          ....

Params Array (according to Aymans answer)

      $serviceOfferingId = 0,
      $templateId = 1;
      $zoneId = 2;
      $name = 3;

      $params = array(
        $serviceOfferingId => '85d06496-bb75-41fb-9358-4ab919e03fe4',
        $templateId => 'c0989cf6-2da5-11e4-a846-726c7bbb864f',
        $zoneId => '7cd483ab-5aad-458b-b5e1-0e270310f41c',
        $name => 'srv11'
      );

and then:

      call_user_func_array ( 'deployVirtualMachine ', $params );

That would pass the array, but renders as more or less unreadable code :/

edited Nov 17, 2023 at 19:30

CommunityBot

11 silver badge

answered Sep 2, 2014 at 17:41

Axel Amthor

11.1k1 gold badge28 silver badges45 bronze badges

Collectives™ on Stack Overflow

php class and method exception

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related