Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-16167

Temporary report tables cleared by one path of clearing caches but not another

          Details

          • Type: Bug
          • Status: Done/Fixed
          • Priority: Trivial
          • Resolution: Fixed/Completed
          • Affects Version/s: 4.4.13, 4.6
          • Fix Version/s: 4.4.14, 4.6
          • Component/s: None
          • Labels:
            None
          • Documentation Required?:
            None

            Description

            On digging it appears there are 2 separate functions for clearing cache tables which nearly replicate each other BUT one has a time clause and the other includes TABLE_NAME LIKE 'civicrm_report_temp%'

            I intend to merge them - although I'm a bit undecided about the time clause. It seems like the one on the UI clear caches maybe should still do the full clobber everything (ie. a force type param) but then again - it does mean that action risks ripping temp tables out from under people & as it is UI-available it could be seen as bad to allow that

            https://github.com/civicrm/civicrm-core/blob/4.6/CRM/Core/BAO/Cache.php#L344-344 would then call rather than duplicate

            https://github.com/civicrm/civicrm-core/blob/4.6/CRM/Core/Config.php#L807-807

              Attachments

                Activity

                  People

                  • Assignee:
                    eileen Eileen McNaughton
                    Reporter:
                    eileen Eileen McNaughton
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    1 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: