Refactor $ctype global as a parameter

1 files changed, 3 insertions, 5 deletions

diff --git a/app/Module/ReviewChangesModule.php b/app/Module/ReviewChangesModule.php
index f3fa99b144..c936e07940 100644
--- a/app/Module/ReviewChangesModule.php
+++ b/app/Module/ReviewChangesModule.php
@@ -52,15 +52,13 @@ class ReviewChangesModule extends AbstractModule implements ModuleBlockInterface
      *
      * @param Tree     $tree
      * @param int      $block_id
-     * @param bool     $template
+     * @param string   $ctype
      * @param string[] $cfg
      *
      * @return string
      */
-    public function getBlock(Tree $tree, int $block_id, bool $template = true, array $cfg = []): string
+    public function getBlock(Tree $tree, int $block_id, string $ctype = '', array $cfg = []): string
     {
-        global $ctype;
-
         $sendmail = $this->getBlockSetting($block_id, 'sendmail', '1');
         $days     = $this->getBlockSetting($block_id, 'days', '1');
 
@@ -140,7 +138,7 @@ class ReviewChangesModule extends AbstractModule implements ModuleBlockInterface
             }
             $content .= '</ul>';
 
-            if ($template) {
+            if ($ctype) {
                 if ($ctype === 'gedcom' && Auth::isManager($tree)) {
                     $config_url = route('tree-page-block-edit', [
                         'block_id' => $block_id,