mirror of
https://github.com/phpbb/phpbb.git
synced 2025-06-28 22:28:51 +00:00
[ticket/12273] Fix missing space in some Exceptions
PHPBB3-12273
This commit is contained in:
parent
08cce5fba5
commit
bc1ab3dc79
1 changed files with 7 additions and 6 deletions
|
@ -218,8 +218,9 @@ class php_exporter
|
||||||
|
|
||||||
if (isset($this->events[$this->current_event]))
|
if (isset($this->events[$this->current_event]))
|
||||||
{
|
{
|
||||||
throw new \LogicException('The event "' . $this->current_event . '" from file "' . $this->current_file
|
throw new \LogicException("The event '{$this->current_event}' from file "
|
||||||
. '" already exists in file "'. $this->events[$this->current_event]['file'] . '"', 10);
|
. "'{$this->current_file}:{$event_line_num}' already exists in file "
|
||||||
|
. "'{$this->events[$this->current_event]['file']}'", 10);
|
||||||
}
|
}
|
||||||
|
|
||||||
$this->events[$this->current_event] = array(
|
$this->events[$this->current_event] = array(
|
||||||
|
@ -308,7 +309,7 @@ class php_exporter
|
||||||
$found_vars_array = true;
|
$found_vars_array = true;
|
||||||
if (strlen($match[1]) > 90)
|
if (strlen($match[1]) > 90)
|
||||||
{
|
{
|
||||||
throw new \LogicException('Should use multiple lines for $vars definition'
|
throw new \LogicException('Should use multiple lines for $vars definition '
|
||||||
. "for event '{$this->current_event}' in file '{$this->current_file}:{$this->current_event_line}'", 3);
|
. "for event '{$this->current_event}' in file '{$this->current_file}:{$this->current_event_line}'", 3);
|
||||||
}
|
}
|
||||||
$vars_array = array_merge($vars_array, explode("', '", $match[1]));
|
$vars_array = array_merge($vars_array, explode("', '", $match[1]));
|
||||||
|
@ -365,7 +366,7 @@ class php_exporter
|
||||||
$doc_line = explode(' ', $var_line, 5);
|
$doc_line = explode(' ', $var_line, 5);
|
||||||
if (sizeof($doc_line) !== 5)
|
if (sizeof($doc_line) !== 5)
|
||||||
{
|
{
|
||||||
throw new \LogicException("Found invalid line '{$this->file_lines[$this->current_event_line - $current_doc_line]}'"
|
throw new \LogicException("Found invalid line '{$this->file_lines[$this->current_event_line - $current_doc_line]}' "
|
||||||
. "for event '{$this->current_event}' in file '{$this->current_file}:{$this->current_event_line}'", 1);
|
. "for event '{$this->current_event}' in file '{$this->current_file}:{$this->current_event_line}'", 1);
|
||||||
}
|
}
|
||||||
$doc_vars[] = $doc_line[3];
|
$doc_vars[] = $doc_line[3];
|
||||||
|
@ -550,7 +551,7 @@ class php_exporter
|
||||||
|
|
||||||
if ($event !== $event_name)
|
if ($event !== $event_name)
|
||||||
{
|
{
|
||||||
throw new \LogicException("Event name does not match '@event' tag for event"
|
throw new \LogicException("Event name does not match '@event' tag for event "
|
||||||
. "'{$this->current_event}' in file '{$this->current_file}:{$this->current_event_line}'", 2);
|
. "'{$this->current_event}' in file '{$this->current_file}:{$this->current_event_line}'", 2);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -573,7 +574,7 @@ class php_exporter
|
||||||
|
|
||||||
if ($sizeof_vars_array !== sizeof($vars_docblock) || $sizeof_vars_array !== sizeof(array_intersect($vars_array, $vars_docblock)))
|
if ($sizeof_vars_array !== sizeof($vars_docblock) || $sizeof_vars_array !== sizeof(array_intersect($vars_array, $vars_docblock)))
|
||||||
{
|
{
|
||||||
throw new \LogicException("\$vars array does not match the list of '@var' tags for event"
|
throw new \LogicException("\$vars array does not match the list of '@var' tags for event "
|
||||||
. "'{$this->current_event}' in file '{$this->current_file}:{$this->current_event_line}'");
|
. "'{$this->current_event}' in file '{$this->current_file}:{$this->current_event_line}'");
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Reference in a new issue