search

mecachisenros / woocommerce_civicrm

GNU Affero General Public License v3.0
14 stars 16 forks source link

CiviCRM Data Caching Option #18

Closed rajeshrhino closed 5 years ago

rajeshrhino commented 6 years ago

Option to cache CiviCRM country/state list, Fetch CiviCRM financial types/other options only in related forms

bastienho commented 5 years ago

Hi @rajeshrhino,

caching huge datas is a good practice for perfs, but I'm not sure about the approach. Why a new table, almost replicating the main civicrm country table?

What about storing, countries in a variable or even a transient (which supports opcache)?

mecachisenros commented 5 years ago

@rajeshrhino I don't see the benefit of recreating the Country/States tables, I agree with @bastienho, Transients or WP_Object_Cache would be a better fit.