CRM-13823 - Remove unncessary settings
authorColeman Watts <coleman@civicrm.org>
Wed, 18 Nov 2015 21:45:00 +0000 (16:45 -0500)
committerColeman Watts <coleman@civicrm.org>
Wed, 18 Nov 2015 21:45:00 +0000 (16:45 -0500)
Now that status messages can be snoozed and disabled there is no need for settings specific to version checks.

CRM/Admin/Form/Setting/Miscellaneous.php
CRM/Utils/Check/Env.php
CRM/Utils/VersionCheck.php
api/v3/examples/Setting/GetFields.php
settings/Core.setting.php
templates/CRM/Admin/Form/Setting/Miscellaneous.tpl
templates/CRM/common/civicrm.settings.php.template

index 6daba5212b526af583ad7626e9a58c3998d1089c..4a4645a628d647b8d690f3f19a68e71d07fd6838 100644 (file)
@@ -40,10 +40,7 @@ class CRM_Admin_Form_Setting_Miscellaneous extends CRM_Admin_Form_Setting {
     'max_attachments' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
     'contact_undelete' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
     'dashboardCacheTimeout' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
-    'versionAlert' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
-    'securityUpdateAlert' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
     'versionCheck' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
-    'versionCheckIgnoreDate' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
     'empoweredBy' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
     'logging' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
     'maxFileSize' => CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME,
index b01f5db17d706e7da66c7dff68a810bdeb6e026d..5927d7b5de5328493063325d03c98420587df41c 100644 (file)
@@ -292,7 +292,7 @@ class CRM_Utils_Check_Env {
   public function checkVersion() {
     $messages = array();
 
-    if (CRM_Core_BAO_Setting::getItem(CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME, 'versionAlert', NULL, 1)) {
+    if (Civi::settings()->get('versionCheck')) {
       $vc = CRM_Utils_VersionCheck::singleton();
       $newerVersion = $vc->isNewerVersionAvailable();
 
@@ -348,7 +348,7 @@ class CRM_Utils_Check_Env {
     else {
       $messages[] = new CRM_Utils_Check_Message(
         __FUNCTION__,
-        ts('The check for new versions of CiviCRM has been disabled.'),
+        ts('The check for new versions of CiviCRM has been disabled. <a %1>Re-enable the setting</a> to receive important security update notifications.', array(1 => CRM_Utils_System::url('civicrm/admin/setting/misc', 'reset=1'))),
         ts('Update Check Disabled'),
         \Psr\Log\LogLevel::NOTICE,
         'fa-times-circle-o'
index 380c4b61dba191d7c1b643e9ee239adcec0b6536..583c5d115304c07c7c7de9d124c14b5006d515b6 100644 (file)
@@ -64,13 +64,6 @@ class CRM_Utils_VersionCheck {
    */
   public $localMajorVersion;
 
-  /**
-   * User setting to skip updates prior to a certain date
-   *
-   * @var string
-   */
-  public $ignoreDate;
-
   /**
    * Info about available versions
    *
@@ -120,7 +113,6 @@ class CRM_Utils_VersionCheck {
         // Get the latest version and send site info
         $this->pingBack();
       }
-      $this->ignoreDate = CRM_Core_BAO_Setting::getItem(CRM_Core_BAO_Setting::SYSTEM_PREFERENCES_NAME, 'versionCheckIgnoreDate');
 
       // Sort version info in ascending order for easier comparisons
       ksort($this->versionInfo, SORT_NUMERIC);
@@ -257,11 +249,9 @@ class CRM_Utils_VersionCheck {
     if (!empty($majorVersion['releases'])) {
       foreach ($majorVersion['releases'] as $release) {
         if (version_compare($this->localVersion, $release['version']) < 0) {
-          if (!$this->ignoreDate || $this->ignoreDate < $release['date']) {
-            $newerVersion['newest'] = $release['version'];
-            if (CRM_Utils_Array::value('security', $release)) {
-              $newerVersion['security'] = $release['version'];
-            }
+          $newerVersion['newest'] = $release['version'];
+          if (CRM_Utils_Array::value('security', $release)) {
+            $newerVersion['security'] = $release['version'];
           }
         }
       }
index c797b6fe9548be5f324a560001c382749ddc1b2a..2f1b5d0ff8b85705329edb0392866c84ba0021a1 100644 (file)
@@ -527,48 +527,6 @@ function setting_getfields_expectedresult() {
         'description' => 'Allow Permanent Delete for contacts who are linked to live financial transactions',
         'help_text' => '',
       ),
-      'versionAlert' => array(
-        'group_name' => 'CiviCRM Preferences',
-        'group' => 'core',
-        'name' => 'versionAlert',
-        'type' => 'Integer',
-        'quick_form_type' => 'Element',
-        'html_type' => 'select',
-        'option_values' => array(
-          '0' => 'Disabled',
-          '1' => 'Display In Page Footer',
-          '2' => 'Display As Popup Alert',
-          '3' => 'Page Footer + Popup Alert',
-        ),
-        'default' => 1,
-        'add' => '4.3',
-        'title' => 'New Version Alerts',
-        'is_domain' => 1,
-        'is_contact' => 0,
-        'description' => '',
-        'help_text' => '',
-      ),
-      'securityUpdateAlert' => array(
-        'group_name' => 'CiviCRM Preferences',
-        'group' => 'core',
-        'name' => 'securityUpdateAlert',
-        'type' => 'Integer',
-        'quick_form_type' => 'Element',
-        'html_type' => 'select',
-        'option_values' => array(
-          '0' => 'Disabled',
-          '1' => 'Display In Page Footer',
-          '2' => 'Display As Popup Alert',
-          '3' => 'Page Footer + Popup Alert',
-        ),
-        'default' => 3,
-        'add' => '4.6',
-        'title' => 'Security Update Alerts',
-        'is_domain' => 1,
-        'is_contact' => 0,
-        'description' => '',
-        'help_text' => '',
-      ),
       'versionCheck' => array(
         'group_name' => 'CiviCRM Preferences',
         'group' => 'core',
@@ -583,24 +541,6 @@ function setting_getfields_expectedresult() {
         'description' => '',
         'help_text' => '',
       ),
-      'versionCheckIgnoreDate' => array(
-        'group_name' => 'CiviCRM Preferences',
-        'group' => 'core',
-        'name' => 'versionCheckIgnoreDate',
-        'type' => 'String',
-        'quick_form_type' => 'Element',
-        'html_type' => 'text',
-        'html_attributes' => array(
-          'placeholder' => 'YYYY-MM-DD',
-        ),
-        'default' => '',
-        'add' => '4.6',
-        'title' => 'Ignore Updates Prior to',
-        'is_domain' => 1,
-        'is_contact' => 0,
-        'description' => '',
-        'help_text' => '',
-      ),
       'securityAlert' => array(
         'group_name' => 'CiviCRM Preferences',
         'group' => 'core',
index d7d817c13b85fb2fbef1a10ee197b50d6f8b7ed8..a09ffbde7bdbea205b9019837f2fdb1a45ed65ac 100644 (file)
@@ -361,20 +361,6 @@ return array(
     'description' => 'Allow Permanent Delete for contacts who are linked to live financial transactions',
     'help_text' => NULL,
   ),
-  'versionAlert' => array(
-    'group_name' => 'CiviCRM Preferences',
-    'group' => 'core',
-    'name' => 'versionAlert',
-    'type' => 'Boolean',
-    'quick_form_type' => 'YesNo',
-    'default' => 1,
-    'add' => '4.3',
-    'title' => 'New Version Alerts',
-    'is_domain' => 1,
-    'is_contact' => 0,
-    'description' => "",
-    'help_text' => NULL,
-  ),
   'versionCheck' => array(
     'group_name' => 'CiviCRM Preferences',
     'group' => 'core',
@@ -389,22 +375,6 @@ return array(
     'description' => "",
     'help_text' => NULL,
   ),
-  'versionCheckIgnoreDate' => array(
-    'group_name' => 'CiviCRM Preferences',
-    'group' => 'core',
-    'name' => 'versionCheckIgnoreDate',
-    'type' => 'String',
-    'quick_form_type' => 'Element',
-    'html_type' => 'text',
-    'html_attributes' => array('placeholder' => 'YYYY-MM-DD'),
-    'default' => '',
-    'add' => '4.6',
-    'title' => 'Ignore Updates Prior to',
-    'is_domain' => 1,
-    'is_contact' => 0,
-    'description' => "",
-    'help_text' => NULL,
-  ),
   'securityAlert' => array(
     'group_name' => 'CiviCRM Preferences',
     'group' => 'core',
@@ -609,15 +579,9 @@ return array(
     'group' => 'core',
     'name' => 'resCacheCode',
     'type' => 'String',
-    'quick_form_type' => 'Element',
-    'html_attributes' => array(
-      'size' => 16,
-      'maxlength' => 16,
-    ),
-    'html_type' => 'Text',
     'default' => NULL,
     'add' => '4.3',
-    'title' => 'Resource Cache Code',
+    'title' => NULL,
     'is_domain' => 1,
     'is_contact' => 0,
     'description' => 'Code appended to resource URLs (JS/CSS) to coerce HTTP caching',
index fe742e72d0377d3b2dac4b7e378ace6896259458..5af743be029205fbc45747d6e469f227b68d791c 100644 (file)
             <td>{$form.versionCheck.html}<br />
                 <p class="description">{ts}When enabled, statistics about your CiviCRM installation are reported anonymously to the CiviCRM team to assist in prioritizing ongoing development efforts. The following information is gathered: CiviCRM version, versions of PHP, MySQL and framework (Drupal/Joomla/standalone), and default language. Counts (but no actual data) of the following record types are reported: contacts, activities, cases, relationships, contributions, contribution pages, contribution products, contribution widgets, discounts, price sets, profiles, events, participants, tell-a-friend pages, grants, mailings, memberships, membership blocks, pledges, pledge blocks and active payment processor types.{/ts}</p></td>
         </tr>
-        <tr class="crm-miscellaneous-form-block-versionCheckIgnoreDate">
-            <td class="label">{$form.versionCheckIgnoreDate.label}</td>
-            <td>{$form.versionCheckIgnoreDate.html|crmReplace:type:date}<br />
-                <p class="description">{ts}If you wish to stop receiving alerts about a current update (for example, if you have applied a security patch manually), set this to today's date.{/ts}</p></td>
-        </tr>
-        <tr class="crm-miscellaneous-form-block-securityUpdateAlert">
-          <td class="label">{$form.securityUpdateAlert.label}</td>
-          <td>{$form.securityUpdateAlert.html}<br />
-            <p class="description">{ts}Alert site admins about security advisories. <strong>Strongly recommended.</strong>{/ts}</p></td>
-        </tr>
-        <tr class="crm-miscellaneous-form-block-versionAlert">
-          <td class="label">{$form.versionAlert.label}</td>
-          <td>{$form.versionAlert.html}<br />
-            <p class="description">{ts}Alert site admins about all new CiviCRM releases.{/ts}</p></td>
-        </tr>
         <tr class="crm-miscellaneous-form-block-empoweredBy">
             <td class="label">{$form.empoweredBy.label}</td>
             <td>{$form.empoweredBy.html}<br />
           })
       }
     });
-    $('select[name=securityUpdateAlert]').change(function() {
-      if ($(this).val() == '0') {
-        CRM.confirm({message: {/literal}"{ts escape='js'}Disabling this option will prevent CiviCRM from checking for important security updates. Are you sure?{/ts}"{literal}})
-          .on('crmConfirm:no', function() {
-            $('select[name=securityUpdateAlert]').val('3');
-          })
-      }
-    });
   });
 </script>
 {/literal}
\ No newline at end of file
index 1c0e64fe54829674b7080aa2b3b167266ee885b7..10fbe26b91dd92ce8fa610806c1d7a8f97123cb8 100644 (file)
@@ -218,10 +218,6 @@ if (!defined('CIVICRM_UF_BASEURL')) {
  // Disable automatic download / installation of extensions
  // $civicrm_setting['Extension Preferences']['ext_repo_url'] = false;
 
- // Disable version update alerts.
- // $civicrm_setting['CiviCRM Preferences']['versionAlert']; = false;
-
-
 /**
  * If you are using any CiviCRM script in the bin directory that
  * requires authentication, then you also need to set this key.