@@ -77,7 +77,7 @@ |
||
77 | 77 | } |
78 | 78 | |
79 | 79 | // yes, it really is a form |
80 | - $this->formId = $formId; |
|
80 | + $this->formId = $formId; |
|
81 | 81 | $this->setTopElement($formElement); |
82 | 82 | } |
83 | 83 |
@@ -55,7 +55,8 @@ |
||
55 | 55 | */ |
56 | 56 | class E5xx_UnknownDomElementType extends E5xx_ProseException |
57 | 57 | { |
58 | - public function __construct($elementType) { |
|
58 | + public function __construct($elementType) |
|
59 | + { |
|
59 | 60 | $msg = "Unknown DOM element type '{$elementType}'"; |
60 | 61 | parent::__construct(500, $msg, $msg); |
61 | 62 | } |
@@ -57,7 +57,8 @@ |
||
57 | 57 | */ |
58 | 58 | class ObsoleteModuleException extends RuntimeException |
59 | 59 | { |
60 | - public function __construct($oldModule, $newModule) { |
|
60 | + public function __construct($oldModule, $newModule) |
|
61 | + { |
|
61 | 62 | $msg = "Support for '$oldModule' has been removed; please use $newModule instead"; |
62 | 63 | parent::__construct($msg); |
63 | 64 | } |
@@ -57,7 +57,8 @@ |
||
57 | 57 | */ |
58 | 58 | class E5xx_NotImplemented extends E5xx_ProseException |
59 | 59 | { |
60 | - public function __construct($methodName) { |
|
60 | + public function __construct($methodName) |
|
61 | + { |
|
61 | 62 | $msg = "Method '{$methodName}' has not been implemented yet"; |
62 | 63 | parent::__construct(500, $msg, $msg); |
63 | 64 | } |
@@ -55,7 +55,8 @@ |
||
55 | 55 | */ |
56 | 56 | class E5xx_ActionFailed extends E5xx_ProseException |
57 | 57 | { |
58 | - public function __construct($actionName, $reason = '', $params = array()) { |
|
58 | + public function __construct($actionName, $reason = '', $params = array()) |
|
59 | + { |
|
59 | 60 | $msg = "Action '$actionName' failed"; |
60 | 61 | if (strlen($reason) > 0) { |
61 | 62 | $msg .= "; reason is '{$reason}'"; |
@@ -57,7 +57,8 @@ |
||
57 | 57 | */ |
58 | 58 | class E4xx_StoryShouldFail extends Exxx_Exception |
59 | 59 | { |
60 | - public function __construct() { |
|
60 | + public function __construct() |
|
61 | + { |
|
61 | 62 | $msg = "Story should fail"; |
62 | 63 | parent::__construct(400, $msg, $msg); |
63 | 64 | } |
@@ -55,7 +55,8 @@ |
||
55 | 55 | */ |
56 | 56 | class E5xx_ExpectFailed extends E5xx_ProseException |
57 | 57 | { |
58 | - public function __construct($actionName, $expected, $found) { |
|
58 | + public function __construct($actionName, $expected, $found) |
|
59 | + { |
|
59 | 60 | $msg = "Action '$actionName' failed; expected '$expected', found '$found'"; |
60 | 61 | parent::__construct(500, $msg, $msg); |
61 | 62 | } |
@@ -61,14 +61,14 @@ |
||
61 | 61 | // get the hosts table, if we have one |
62 | 62 | $hostsTable = $this->getTable(); |
63 | 63 | if (!$hostsTable) { |
64 | - return; |
|
64 | + return; |
|
65 | 65 | } |
66 | 66 | |
67 | 67 | // remove any empty test environments |
68 | 68 | foreach ($hostsTable as $testEnv => $hosts) { |
69 | - if (count(get_object_vars($hosts)) == 0) { |
|
70 | - unset($hostsTable->$testEnv); |
|
71 | - } |
|
69 | + if (count(get_object_vars($hosts)) == 0) { |
|
70 | + unset($hostsTable->$testEnv); |
|
71 | + } |
|
72 | 72 | } |
73 | 73 | |
74 | 74 | // cleanup the tables |
@@ -501,7 +501,7 @@ discard block |
||
501 | 501 | $data = $hostDetails->appSettings->getData($path); |
502 | 502 | |
503 | 503 | // all done |
504 | - $log->endAction([ "setting is", $data ]); |
|
504 | + $log->endAction(["setting is", $data]); |
|
505 | 505 | return $data; |
506 | 506 | } |
507 | 507 | |
@@ -573,7 +573,7 @@ discard block |
||
573 | 573 | $data = $hostDetails->storySettings->getData($path); |
574 | 574 | |
575 | 575 | // all done |
576 | - $log->endAction([ "setting is", $data ]); |
|
576 | + $log->endAction(["setting is", $data]); |
|
577 | 577 | return $data; |
578 | 578 | } |
579 | 579 |
@@ -318,6 +318,7 @@ |
||
318 | 318 | |
319 | 319 | /** |
320 | 320 | * @deprecated since v2.4.0 |
321 | + * @param string $sessionName |
|
321 | 322 | */ |
322 | 323 | public function getScreenIsRunning($sessionName) |
323 | 324 | { |