fisharebest / webtrees

app/GedcomRecordFactoriesInterface.php

Indentation +20 added lines, -20 removed lines

@@ -31,25 +31,25 @@
 
 interface GedcomRecordFactoriesInterface {
 
-  /**
-   * Add (or replace) a factory for a specific type, e.g. 'INDI', 'FAM' ...
-   */
-  public function setFactory(string $type, GedcomRecordFactory $factory): void;
-
-  /**
-   * Get an instance of a GedcomRecord object. For single records,
-   * we just receive the XREF. For bulk records (such as lists
-   * and search results) we can receive the GEDCOM data as well.
-   *
-   * @param string      $xref
-   * @param Tree        $tree
-   * @param string|null $gedcom
-   *
-   * @throws Exception
-   * @return GedcomRecord|Individual|Family|Source|Repository|Media|Note|null
-   */
-  public function getInstance(string $xref, Tree $tree, string $gedcom = null);
-
-  public function clearCache(): void;
+    /**
+     * Add (or replace) a factory for a specific type, e.g. 'INDI', 'FAM' ...
+     */
+    public function setFactory(string $type, GedcomRecordFactory $factory): void;
+
+    /**
+     * Get an instance of a GedcomRecord object. For single records,
+     * we just receive the XREF. For bulk records (such as lists
+     * and search results) we can receive the GEDCOM data as well.
+     *
+     * @param string      $xref
+     * @param Tree        $tree
+     * @param string|null $gedcom
+     *
+     * @throws Exception
+     * @return GedcomRecord|Individual|Family|Source|Repository|Media|Note|null
+     */
+    public function getInstance(string $xref, Tree $tree, string $gedcom = null);
+
+    public function clearCache(): void;
 
 }

Braces +2 added lines, -1 removed lines

@@ -29,7 +29,8 @@
 use Fisharebest\Webtrees\Repository;
 use Fisharebest\Webtrees\Source;
 
-interface GedcomRecordFactoriesInterface {
+interface GedcomRecordFactoriesInterface
+{
 
   /**
    * Add (or replace) a factory for a specific type, e.g. 'INDI', 'FAM' ...

app/GedcomRecordFactory.php

Indentation +1 added lines, -1 removed lines

@@ -24,6 +24,6 @@
 
 interface GedcomRecordFactory {
 
-  public function createRecord(string $xref, string $gedcom, ?string $pending, Tree $tree): GedcomRecord;
+    public function createRecord(string $xref, string $gedcom, ?string $pending, Tree $tree): GedcomRecord;
   
 }
\ No newline at end of file

Braces +2 added lines, -1 removed lines

@@ -22,7 +22,8 @@
 use Fisharebest\Webtrees\GedcomRecord;
 use Fisharebest\Webtrees\Tree;
 
-interface GedcomRecordFactory {
+interface GedcomRecordFactory
+{
 
   public function createRecord(string $xref, string $gedcom, ?string $pending, Tree $tree): GedcomRecord;
   

app/GedcomRecordFactories.php

Indentation +56 added lines, -56 removed lines

@@ -35,50 +35,50 @@ 
 
 class GedcomRecordFactories implements GedcomRecordFactoriesInterface {
 
-  /** @var GedcomRecord[][] Allow getInstance() to return references to existing objects */
-  protected $gedcom_record_cache;
+    /** @var GedcomRecord[][] Allow getInstance() to return references to existing objects */
+    protected $gedcom_record_cache;
 
-  /** @var stdClass[][] Fetch all pending edits in one database query */
-  protected $pending_record_cache;
+    /** @var stdClass[][] Fetch all pending edits in one database query */
+    protected $pending_record_cache;
   
-  protected $factories = [];
+    protected $factories = [];
 
-  public function __construct() {
-  }
+    public function __construct() {
+    }
   
-  public function setFactory(string $type, GedcomRecordFactory $factory): void 
-  {
-      $this->factories[$type] = $factory;
-  }
-
-  /**
-   * Get an instance of a GedcomRecord object. For single records,
-   * we just receive the XREF. For bulk records (such as lists
-   * and search results) we can receive the GEDCOM data as well.
-   *
-   * @param string      $xref
-   * @param Tree        $tree
-   * @param string|null $gedcom
-   *
-   * @throws Exception
-   * @return GedcomRecord|Individual|Family|Source|Repository|Media|Note|null
-   */
-  public function getInstance(string $xref, Tree $tree, string $gedcom = null) {
+    public function setFactory(string $type, GedcomRecordFactory $factory): void 
+    {
+        $this->factories[$type] = $factory;
+    }
+
+    /**
+     * Get an instance of a GedcomRecord object. For single records,
+     * we just receive the XREF. For bulk records (such as lists
+     * and search results) we can receive the GEDCOM data as well.
+     *
+     * @param string      $xref
+     * @param Tree        $tree
+     * @param string|null $gedcom
+     *
+     * @throws Exception
+     * @return GedcomRecord|Individual|Family|Source|Repository|Media|Note|null
+     */
+    public function getInstance(string $xref, Tree $tree, string $gedcom = null) {
     $tree_id = $tree->id();
 
     // Is this record already in the cache?
     if (isset($this->gedcom_record_cache[$xref][$tree_id])) {
-      return $this->gedcom_record_cache[$xref][$tree_id];
+        return $this->gedcom_record_cache[$xref][$tree_id];
     }
 
     // Do we need to fetch the record from the database?
     if ($gedcom === null) {
-      $gedcom = GedcomRecord::retrieveGedcomRecord($xref, $tree_id);
+        $gedcom = GedcomRecord::retrieveGedcomRecord($xref, $tree_id);
     }
 
     // If we can edit, then we also need to be able to see pending records.
     if (Auth::isEditor($tree)) {
-      if (!isset($this->pending_record_cache[$tree_id])) {
+        if (!isset($this->pending_record_cache[$tree_id])) {
         // Fetch all pending records in one database query
         $this->pending_record_cache[$tree_id] = [];
         $rows = DB::table('change')
@@ -89,70 +89,70 @@ 
                 ->get();
 
         foreach ($rows as $row) {
-          $this->pending_record_cache[$tree_id][$row->xref] = $row->new_gedcom;
+            $this->pending_record_cache[$tree_id][$row->xref] = $row->new_gedcom;
+        }
         }
-      }
 
-      $pending = $this->pending_record_cache[$tree_id][$xref] ?? null;
+        $pending = $this->pending_record_cache[$tree_id][$xref] ?? null;
     } else {
-      // There are no pending changes for this record
-      $pending = null;
+        // There are no pending changes for this record
+        $pending = null;
     }
 
     // No such record exists
     if ($gedcom === null && $pending === null) {
-      return null;
+        return null;
     }
 
     // No such record, but a pending creation exists
     if ($gedcom === null) {
-      $gedcom = '';
+        $gedcom = '';
     }
 
     // Create the object
     if (preg_match('/^0 @(' . Gedcom::REGEX_XREF . ')@ (' . Gedcom::REGEX_TAG . ')/', $gedcom . $pending, $match)) {
-      $xref = $match[1]; // Collation - we may have requested I123 and found i123
-      $type = $match[2];
+        $xref = $match[1]; // Collation - we may have requested I123 and found i123
+        $type = $match[2];
     } elseif (preg_match('/^0 (HEAD|TRLR)/', $gedcom . $pending, $match)) {
-      $xref = $match[1];
-      $type = $match[1];
+        $xref = $match[1];
+        $type = $match[1];
     } elseif ($gedcom . $pending) {
-      throw new Exception('Unrecognized GEDCOM record: ' . $gedcom);
+        throw new Exception('Unrecognized GEDCOM record: ' . $gedcom);
     } else {
-      // A record with both pending creation and pending deletion
-      $type = static::RECORD_TYPE;
+        // A record with both pending creation and pending deletion
+        $type = static::RECORD_TYPE;
     }
 
     $factory = $this->factories[$type] ?? null;
     if ($factory !== null) {
-      $record = $factory->createRecord($xref, $gedcom, $pending, $tree);
+        $record = $factory->createRecord($xref, $gedcom, $pending, $tree);
 
-      // Store it in the cache
-      $this->gedcom_record_cache[$xref][$tree_id] = $record;
+        // Store it in the cache
+        $this->gedcom_record_cache[$xref][$tree_id] = $record;
 
-      return $record;
+        return $record;
     }
 
     switch ($type) {
-      case 'INDI':
+        case 'INDI':
         $record = new Individual($xref, $gedcom, $pending, $tree);
         break;
-      case 'FAM':
+        case 'FAM':
         $record = new Family($xref, $gedcom, $pending, $tree);
         break;
-      case 'SOUR':
+        case 'SOUR':
         $record = new Source($xref, $gedcom, $pending, $tree);
         break;
-      case 'OBJE':
+        case 'OBJE':
         $record = new Media($xref, $gedcom, $pending, $tree);
         break;
-      case 'REPO':
+        case 'REPO':
         $record = new Repository($xref, $gedcom, $pending, $tree);
         break;
-      case 'NOTE':
+        case 'NOTE':
         $record = new Note($xref, $gedcom, $pending, $tree);
         break;
-      default:
+        default:
         $record = new GedcomRecord($xref, $gedcom, $pending, $tree);
         break;
     }
@@ -161,12 +161,12 @@ 
     $this->gedcom_record_cache[$xref][$tree_id] = $record;
 
     return $record;
-  }
+    }
 
-  public function clearCache(): void {
+    public function clearCache(): void {
     // Clear the cache
     $this->gedcom_record_cache = [];
     $this->pending_record_cache = [];
-  }
+    }
 
 }

Braces +6 added lines, -3 removed lines

@@ -33,7 +33,8 @@ 
 use Fisharebest\Webtrees\Source;
 use stdClass;
 
-class GedcomRecordFactories implements GedcomRecordFactoriesInterface {
+class GedcomRecordFactories implements GedcomRecordFactoriesInterface
+{
 
   /** @var GedcomRecord[][] Allow getInstance() to return references to existing objects */
   protected $gedcom_record_cache;
@@ -43,7 +44,8 @@ 
   
   protected $factories = [];
 
-  public function __construct() {
+  public function __construct()
+  {
   }
   
   public function setFactory(string $type, GedcomRecordFactory $factory): void 
@@ -63,7 +65,8 @@ 
    * @throws Exception
    * @return GedcomRecord|Individual|Family|Source|Repository|Media|Note|null
    */
-  public function getInstance(string $xref, Tree $tree, string $gedcom = null) {
+  public function getInstance(string $xref, Tree $tree, string $gedcom = null)
+  {
     $tree_id = $tree->id();
 
     // Is this record already in the cache?

app/GedcomRecord.php

Indentation +1 added lines, -1 removed lines

@@ -190,7 +190,7 @@
 
     public static function retrieveGedcomRecord(string $xref, int $tree_id): ?string
     {
-      return self::fetchGedcomRecord($xref, $tree_id);
+        return self::fetchGedcomRecord($xref, $tree_id);
     }
 
     /**