Merge pull request #89 from creative-commoners/pulls/5/protect-hook

API Set extension hook implementation visibility to protected
silverstripe · May 21, 2024 · b98b0a7 · b98b0a7
2 parents 1c0ecca + 853c2b2
commit b98b0a7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/docs/en/developer-documentation.md b/docs/en/developer-documentation.md
@@ -21,7 +21,7 @@ For instance, in order to add a new Page field to the report you could add an ex
     use SilverStripe\SiteWideContentReport\SiteWideContentReport;
 
 	class MyReportExtension extends Extension {
-		public function updateColumns($itemType, &$columns) {
+		protected function updateColumns($itemType, &$columns) {
 			if(itemType !== 'Pages') {
 				return;
 			}
@@ -68,7 +68,7 @@ use SilverStripe\GridfieldQueuedExport\Forms\GridFieldQueuedExportButton;
 
 class SitewideContentReportQueuedExportExtension extends Extension
 {
-    public function updateExportButton(GridFieldComponent &$exportButton)
+    protected function updateExportButton(GridFieldComponent &$exportButton)
     {
         $exportButton = new GridFieldQueuedExportButton('buttons-after-left');
     }