Add a couple of soft deprecations
authorEileen McNaughton <emcnaughton@wikimedia.org>
Fri, 27 Oct 2023 04:25:36 +0000 (17:25 +1300)
committerEileen McNaughton <emcnaughton@wikimedia.org>
Fri, 27 Oct 2023 04:25:36 +0000 (17:25 +1300)
CRM/Invoicing/Utils.php

index 70de2c7d4d6535bd1b30e1a66ebd0fdb3cf33390..382bc4177d37feb55264b95256c0e6f17f178116 100644 (file)
@@ -48,11 +48,9 @@ class CRM_Invoicing_Utils {
   /**
    * Function to call to determine if invoicing is enabled.
    *
-   * Historically the invoicing was declared as a setting but actually
-   * set within contribution_invoice_settings (which stores multiple settings
-   * as an array in a non-standard way).
+   * Use Civi::settings()->get('invoicing') instead.
    *
-   * We check both here. But will deprecate the latter in time.
+   * @deprecated since 5.68 expected removal time to be added when we add noisy deprecation.
    */
   public static function isInvoicingEnabled() {
     return Civi::settings()->get('invoicing');
@@ -61,8 +59,9 @@ class CRM_Invoicing_Utils {
   /**
    * Function to get the tax term.
    *
-   * The value is nested in the contribution_invoice_settings setting - which
-   * is unsupported. Here we have a wrapper function to make later cleanup easier.
+   * Use Civi::settings()->get('tax_term') instead.
+   *
+   * @deprecated since 5.68 expected removal time to be added when we add noisy deprecation.
    */
   public static function getTaxTerm() {
     return Civi::settings()->get('tax_term');