Merge pull request #21199 from colemanw/fkDeleteBetter

[civicrm-core.git] / CRM / Financial / Page / BatchTransaction.php

diff --git a/CRM/Financial/Page/BatchTransaction.php b/CRM/Financial/Page/BatchTransaction.php
index cd32c0de9eafa683681eec579bc29919c4894f03..9f2221fb021711626519c7de0efea98d379f80f7 100644 (file)
--- a/CRM/Financial/Page/BatchTransaction.php
+++ b/CRM/Financial/Page/BatchTransaction.php
@@ -44,6 +44,14 @@ class CRM_Financial_Page_BatchTransaction extends CRM_Core_Page_Basic {
   /**
    * Get action Links.
    *
+   * @todo:
+   * While this function only references static self::$_links, we can't make
+   * the function static because we need to match CRM_Core_Page_Basic. Possibly
+   * the intent was caching, but there's nothing very time-consuming in here
+   * that needs it so do we even need $_links? The variable is public - a quick
+   * look doesn't seem like it's used on its own, but it's hard to fully check
+   * that.
+   *
    * @return array
    *   (reference) of action links
    */