summaryrefslogtreecommitdiff
path: root/app/Census/AbstractCensusColumn.php
diff options
context:
space:
mode:
authorRico Sonntag <mail@ricosonntag.de>2018-09-23 00:32:55 +0200
committerGreg Roach <fisharebest@gmail.com>2018-09-22 23:32:55 +0100
commitb2ce94c6833c25934b40a7e6bc15985d50b5f42a (patch)
tree62b41467db3d26698ff21ed862461402d2febd1d /app/Census/AbstractCensusColumn.php
parenta0cead57f9defbc64fdc8dfbbfb2cf01c516be57 (diff)
downloadwebtrees-b2ce94c6833c25934b40a7e6bc15985d50b5f42a.tar.gz
webtrees-b2ce94c6833c25934b40a7e6bc15985d50b5f42a.tar.bz2
webtrees-b2ce94c6833c25934b40a7e6bc15985d50b5f42a.zip
Reduced if/else complexity, removed not require else conditions (#1866)
* Reduced if/else complexity, removed not require else conditions * Fix indentation * Fix indentation * Use null-coalesce to simplify logic * Use null-coalesce to simplify logic * Use null-coalesce to simplify logic * Use null-coalesce to simplify logic * Use null-coalesce to simplify logic * Use null-coalesce to simplify logic * Admin theme no longer exists * Code style * Use null-coalesce operator to simplify logic
Diffstat (limited to 'app/Census/AbstractCensusColumn.php')
-rw-r--r--app/Census/AbstractCensusColumn.php24
1 files changed, 12 insertions, 12 deletions
diff --git a/app/Census/AbstractCensusColumn.php b/app/Census/AbstractCensusColumn.php
index 61af235270..062a1e135d 100644
--- a/app/Census/AbstractCensusColumn.php
+++ b/app/Census/AbstractCensusColumn.php
@@ -71,9 +71,9 @@ class AbstractCensusColumn
if ($family) {
return $family->getHusband();
- } else {
- return null;
}
+
+ return null;
}
/**
@@ -89,9 +89,9 @@ class AbstractCensusColumn
if ($family) {
return $family->getWife();
- } else {
- return null;
}
+
+ return null;
}
/**
@@ -113,13 +113,13 @@ class AbstractCensusColumn
if (empty($families)) {
return null;
- } else {
- usort($families, function (Family $x, Family $y): int {
- return Date::compare($x->getMarriageDate(), $y->getMarriageDate());
- });
-
- return end($families);
}
+
+ usort($families, function (Family $x, Family $y): int {
+ return Date::compare($x->getMarriageDate(), $y->getMarriageDate());
+ });
+
+ return end($families);
}
/**
@@ -169,9 +169,9 @@ class AbstractCensusColumn
if (end($parts) === $this->place()) {
return implode(', ', array_slice($parts, 0, -1));
- } else {
- return $place;
}
+
+ return $place;
}
/**