Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-18105 Make deduping reversable
  3. CRM-18180

Log table lookup should include all declared tables, even if they don't start with civicrm_

    Details

    • Type: Sub-task
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.7.3
    • Fix Version/s: 4.7.7
    • Component/s: None
    • Documentation Required?:
      Developer Doc
    • Funding Source:
      Contributed Code

      Description

      While mostly the log tables DO start with log_civicrm we permit any table to be declared for logging by hook so we should allow for that possibility when looking up tables. Patch adds on a clause

      " OR TABLE_NAME IN (" . implode(',', $nonStandardTableLogs) . ")";

      in addition to

      TABLE_NAME LIKE 'log_civicrm'

      Adding this did not cause the query to stop using indexes in my testing

        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: