@@ -78,8 +78,7 @@ discard block |
||
78 | 78 | public function startAction($message, $codeLine = null) |
79 | 79 | { |
80 | 80 | // do we have an open action? |
81 | - if (!$this->action || !$this->action->getIsOpen()) |
|
82 | - { |
|
81 | + if (!$this->action || !$this->action->getIsOpen()) { |
|
83 | 82 | $openItem = $this->action = new Action_LogItem($this->injectables, 1); |
84 | 83 | } |
85 | 84 | else { |
@@ -93,8 +92,7 @@ discard block |
||
93 | 92 | public function closeAllOpenActions() |
94 | 93 | { |
95 | 94 | // do we have any empty log items? |
96 | - if (!$this->action) |
|
97 | - { |
|
95 | + if (!$this->action) { |
|
98 | 96 | return; |
99 | 97 | } |
100 | 98 |
@@ -60,7 +60,8 @@ |
||
60 | 60 | /** |
61 | 61 | * @param string $err |
62 | 62 | */ |
63 | - public function __construct($err) { |
|
63 | + public function __construct($err) |
|
64 | + { |
|
64 | 65 | $msg = "Invalid config - $err"; |
65 | 66 | parent::__construct(500, $msg, $msg); |
66 | 67 | } |
@@ -61,7 +61,8 @@ |
||
61 | 61 | /** |
62 | 62 | * @param string $key |
63 | 63 | */ |
64 | - public function __construct($key) { |
|
64 | + public function __construct($key) |
|
65 | + { |
|
65 | 66 | $msg = "No such data '{$key}' in the checkpoint"; |
66 | 67 | parent::__construct(500, $msg, $msg); |
67 | 68 | } |
@@ -61,7 +61,8 @@ |
||
61 | 61 | /** |
62 | 62 | * @param string $className |
63 | 63 | */ |
64 | - public function __construct($className) { |
|
64 | + public function __construct($className) |
|
65 | + { |
|
65 | 66 | $msg = "Class '{$className}' does not inherit from Prose\\Prose"; |
66 | 67 | parent::__construct(500, $msg, $msg); |
67 | 68 | } |
@@ -68,7 +68,7 @@ |
||
68 | 68 | // |
69 | 69 | // we always search the generic 'Phases' namespace first, in case |
70 | 70 | // users don't want to uniquely namespace their Phase classes |
71 | - $this->namespaces = array ("Phases"); |
|
71 | + $this->namespaces = array("Phases"); |
|
72 | 72 | |
73 | 73 | // add in any additional namespaces we've been asked to search |
74 | 74 | foreach ($namespaces as $namespace) { |
@@ -78,7 +78,7 @@ discard block |
||
78 | 78 | public function playPhases($activity, StoryTeller $st, Injectables $injectables, $phases, $thingBeingPlayed) |
79 | 79 | { |
80 | 80 | // shorthand |
81 | - $output = $st->getOutput(); |
|
81 | + $output = $st->getOutput(); |
|
82 | 82 | |
83 | 83 | // we are going to need something to help us load each of our |
84 | 84 | // phases |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | |
99 | 99 | // the result of playing this group of phases |
100 | 100 | $groupResult = null; |
101 | - if ($thingBeingPlayed){ |
|
101 | + if ($thingBeingPlayed) { |
|
102 | 102 | $groupResult = $thingBeingPlayed->getResult(); |
103 | 103 | $groupResult->setActivity($activity); |
104 | 104 | } |
@@ -98,7 +98,7 @@ discard block |
||
98 | 98 | |
99 | 99 | // the result of playing this group of phases |
100 | 100 | $groupResult = null; |
101 | - if ($thingBeingPlayed){ |
|
101 | + if ($thingBeingPlayed) { |
|
102 | 102 | $groupResult = $thingBeingPlayed->getResult(); |
103 | 103 | $groupResult->setActivity($activity); |
104 | 104 | } |
@@ -110,8 +110,7 @@ discard block |
||
110 | 110 | // |
111 | 111 | // 1. all listed phases have been executed, or |
112 | 112 | // 2. one of the phases says that the story has failed |
113 | - foreach ($phasesToPlay as $phaseName => $phaseData) |
|
114 | - { |
|
113 | + foreach ($phasesToPlay as $phaseName => $phaseData) { |
|
115 | 114 | // shorthand |
116 | 115 | $phase = $phaseData['phase']; |
117 | 116 | $isActive = $phaseData['isActive']; |
@@ -128,8 +127,7 @@ discard block |
||
128 | 127 | |
129 | 128 | // now, what do we do? |
130 | 129 | $nextAction = $phaseResult->getNextAction(); |
131 | - switch ($nextAction) |
|
132 | - { |
|
130 | + switch ($nextAction) { |
|
133 | 131 | case self::NEXT_SKIP: |
134 | 132 | // why? |
135 | 133 | if ($phaseResult->getPhaseIsBlacklisted()) { |
@@ -78,37 +78,37 @@ |
||
78 | 78 | * |
79 | 79 | * @var PhaseResult |
80 | 80 | */ |
81 | - public $failedPhase = null; |
|
81 | + public $failedPhase = null; |
|
82 | 82 | |
83 | 83 | /** |
84 | 84 | * did the story pass, fail, or otherwise go horribly wrong? |
85 | 85 | * @var integer |
86 | 86 | */ |
87 | - public $resultCode = 0; |
|
87 | + public $resultCode = 0; |
|
88 | 88 | |
89 | 89 | /** |
90 | 90 | * when did we start playing this story? |
91 | 91 | * @var float |
92 | 92 | */ |
93 | - public $startTime = null; |
|
93 | + public $startTime = null; |
|
94 | 94 | |
95 | 95 | /** |
96 | 96 | * when did we finish playing this story? |
97 | 97 | * @var float |
98 | 98 | */ |
99 | - public $endTime = null; |
|
99 | + public $endTime = null; |
|
100 | 100 | |
101 | 101 | /** |
102 | 102 | * how long did the story take to play? |
103 | 103 | * @var float |
104 | 104 | */ |
105 | - public $durationTime = null; |
|
105 | + public $durationTime = null; |
|
106 | 106 | |
107 | 107 | /** |
108 | 108 | * which file is this PhaseGroup_Result associated with? |
109 | 109 | * @var string |
110 | 110 | */ |
111 | - public $filename = null; |
|
111 | + public $filename = null; |
|
112 | 112 | |
113 | 113 | const UNKNOWN = 0; |
114 | 114 | const OKAY = 1; |
@@ -223,8 +223,7 @@ discard block |
||
223 | 223 | */ |
224 | 224 | public function getPhaseGroupFailed() |
225 | 225 | { |
226 | - switch ($this->resultCode) |
|
227 | - { |
|
226 | + switch ($this->resultCode) { |
|
228 | 227 | case self::FAIL: |
229 | 228 | case self::ERROR: |
230 | 229 | case self::INCOMPLETE: |
@@ -240,8 +239,7 @@ discard block |
||
240 | 239 | */ |
241 | 240 | public function getPhaseGroupSkipped() |
242 | 241 | { |
243 | - switch ($this->resultCode) |
|
244 | - { |
|
242 | + switch ($this->resultCode) { |
|
245 | 243 | case self::BLACKLISTED: |
246 | 244 | case self::SKIPPED: |
247 | 245 | return true; |
@@ -71,7 +71,7 @@ |
||
71 | 71 | // |
72 | 72 | // we always search the generic 'Prose' namespace first, in case |
73 | 73 | // users don't want to uniquely namespace their Prose classes |
74 | - $this->namespaces = array ("Prose"); |
|
74 | + $this->namespaces = array("Prose"); |
|
75 | 75 | |
76 | 76 | // append the namespaces we've been asked to search |
77 | 77 | foreach ($namespaces as $namespace) { |
@@ -68,7 +68,7 @@ |
||
68 | 68 | // |
69 | 69 | // we always search the generic 'Reports' namespace first, in case |
70 | 70 | // users don't want to uniquely namespace their Report classes |
71 | - $this->namespaces = array ("Reports"); |
|
71 | + $this->namespaces = array("Reports"); |
|
72 | 72 | |
73 | 73 | // add in any additional namespaces we've been asked to search |
74 | 74 | foreach ($namespaces as $namespace) { |