diff options
| author | Greg Roach <fisharebest@gmail.com> | 2015-10-23 08:03:44 +0100 |
|---|---|---|
| committer | Greg Roach <fisharebest@gmail.com> | 2015-10-23 08:03:59 +0100 |
| commit | a7e4e269b2c2fade5a6780850289bed4753f5ffb (patch) | |
| tree | b00aeabb505788d36be68160aff89ba1b1d7d0f2 | |
| parent | 4d05a5882553461e2cfd49ba3c94039e01f63942 (diff) | |
| download | webtrees-a7e4e269b2c2fade5a6780850289bed4753f5ffb.tar.gz webtrees-a7e4e269b2c2fade5a6780850289bed4753f5ffb.tar.bz2 webtrees-a7e4e269b2c2fade5a6780850289bed4753f5ffb.zip | |
DRY
| -rw-r--r-- | app/Census/AbstractCensusColumn.php | 2 | ||||
| -rw-r--r-- | app/Census/CensusColumnBirthPlaceSimple.php | 5 | ||||
| -rw-r--r-- | app/Census/CensusColumnFatherBirthPlaceSimple.php | 4 | ||||
| -rw-r--r-- | app/Census/CensusColumnFatherForeign.php | 2 | ||||
| -rw-r--r-- | app/Census/CensusColumnMotherBirthPlaceSimple.php | 4 | ||||
| -rw-r--r-- | app/Census/CensusColumnMotherForeign.php | 2 |
6 files changed, 8 insertions, 11 deletions
diff --git a/app/Census/AbstractCensusColumn.php b/app/Census/AbstractCensusColumn.php index 26afd7046b..40d27e0522 100644 --- a/app/Census/AbstractCensusColumn.php +++ b/app/Census/AbstractCensusColumn.php @@ -61,7 +61,7 @@ class AbstractCensusColumn { * * @return string - e.g. "England" */ - protected function country($place) { + protected function lastPartOfPlace($place) { $place = explode(', ', $place); return end($place); diff --git a/app/Census/CensusColumnBirthPlaceSimple.php b/app/Census/CensusColumnBirthPlaceSimple.php index 0a2af0dfd6..6bfcc429fd 100644 --- a/app/Census/CensusColumnBirthPlaceSimple.php +++ b/app/Census/CensusColumnBirthPlaceSimple.php @@ -31,9 +31,6 @@ class CensusColumnBirthPlaceSimple extends CensusColumnBirthPlace implements Cen * @return string */ public function generate(Individual $individual, Individual $head = null) { - $birth_place = parent::generate($individual, $head); - $tmp = explode(', ', $birth_place); - - return end($tmp); + return $this->lastPartOfPlace(parent::generate($individual, $head)); } } diff --git a/app/Census/CensusColumnFatherBirthPlaceSimple.php b/app/Census/CensusColumnFatherBirthPlaceSimple.php index 4df4d8566d..a16587bd79 100644 --- a/app/Census/CensusColumnFatherBirthPlaceSimple.php +++ b/app/Census/CensusColumnFatherBirthPlaceSimple.php @@ -21,7 +21,7 @@ use Fisharebest\Webtrees\Individual; /** * The individual's father's birth place. */ -class CensusColumnFatherBirthPlaceSimple extends CensusColumnBirthPlaceSimple implements CensusColumnInterface { +class CensusColumnFatherBirthPlaceSimple extends CensusColumnFatherBirthPlace implements CensusColumnInterface { /** * Generate the likely value of this census column, based on available information. * @@ -31,6 +31,6 @@ class CensusColumnFatherBirthPlaceSimple extends CensusColumnBirthPlaceSimple im * @return string */ public function generate(Individual $individual, Individual $head = null) { - return parent::generate($this->father($individual), $head); + return $this->lastPartOfPlace(parent::generate($individual, $head)); } } diff --git a/app/Census/CensusColumnFatherForeign.php b/app/Census/CensusColumnFatherForeign.php index c77b0b70a7..0e8398787f 100644 --- a/app/Census/CensusColumnFatherForeign.php +++ b/app/Census/CensusColumnFatherForeign.php @@ -32,7 +32,7 @@ class CensusColumnFatherForeign extends AbstractCensusColumn implements CensusCo public function generate(Individual $individual, Individual $head = null) { $father = $this->father($individual); - if ($father && $this->country($father->getBirthPlace()) !== $this->place()) { + if ($father && $this->lastPartOfPlace($father->getBirthPlace()) !== $this->place()) { return 'Y'; } else { return ''; diff --git a/app/Census/CensusColumnMotherBirthPlaceSimple.php b/app/Census/CensusColumnMotherBirthPlaceSimple.php index 2889cc6fd7..e87ef3d802 100644 --- a/app/Census/CensusColumnMotherBirthPlaceSimple.php +++ b/app/Census/CensusColumnMotherBirthPlaceSimple.php @@ -21,7 +21,7 @@ use Fisharebest\Webtrees\Individual; /** * The individual's mother's birth place. */ -class CensusColumnMotherBirthPlaceSimple extends CensusColumnBirthPlaceSimple implements CensusColumnInterface { +class CensusColumnMotherBirthPlaceSimple extends CensusColumnMotherBirthPlace implements CensusColumnInterface { /** * Generate the likely value of this census column, based on available information. * @@ -31,6 +31,6 @@ class CensusColumnMotherBirthPlaceSimple extends CensusColumnBirthPlaceSimple im * @return string */ public function generate(Individual $individual, Individual $head = null) { - return parent::generate($this->mother($individual), $head); + return $this->lastPartOfPlace(parent::generate($individual, $head)); } } diff --git a/app/Census/CensusColumnMotherForeign.php b/app/Census/CensusColumnMotherForeign.php index 64a344c6a9..2fb05952f1 100644 --- a/app/Census/CensusColumnMotherForeign.php +++ b/app/Census/CensusColumnMotherForeign.php @@ -32,7 +32,7 @@ class CensusColumnMotherForeign extends AbstractCensusColumn implements CensusCo public function generate(Individual $individual, Individual $head = null) { $mother = $this->mother($individual); - if ($mother && $this->country($mother->getBirthPlace()) !== $this->place()) { + if ($mother && $this->lastPartOfPlace($mother->getBirthPlace()) !== $this->place()) { return 'Y'; } else { return ''; |
