ENH Allow disabling of help_links

creative-commoners · Sep 4, 2023 · 325d87c · 325d87c
1 parent 714c773
commit 325d87c
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 7 deletions.
diff --git a/code/LeftAndMain.php b/code/LeftAndMain.php
@@ -1889,13 +1889,15 @@ public function getHelpLinks()
             }
         });
 
-        foreach ($helpLinks as $key => $value) {
-            $translationKey = str_replace(' ', '', $key ?? '');
-
-            $formattedLinks[] = [
-                'Title' => _t(__CLASS__ . '.' . $translationKey, $key),
-                'URL' => $value
-            ];
+        if ($helpLinks) {
+            foreach ($helpLinks as $key => $value) {
+                $translationKey = str_replace(' ', '', $key ?? '');
+
+                $formattedLinks[] = [
+                    'Title' => _t(__CLASS__ . '.' . $translationKey, $key),
+                    'URL' => $value
+                ];
+            }
         }
 
         return ArrayList::create($formattedLinks);

diff --git a/tests/php/LeftAndMainTest.php b/tests/php/LeftAndMainTest.php
@@ -215,6 +215,13 @@ public function testGetHelpLinks()
         $this->assertEquals('www.silverstripe.org', $silverstripeLink['URL']);
     }
 
+    public function testDisableHelpLinks()
+    {
+        Config::modify()->set(LeftAndMain::class, 'help_links', false);
+        $helpLinks = LeftAndMain::singleton()->getHelpLinks();
+        $this->assertCount(0, $helpLinks);
+    }
+
     /**
      * @dataProvider provideTestCMSVersionNumber
      */