fixes #343 by reducing frequency of coupon load

Jon007 · Jon007 · commit edf0993e0a4e · 2018-08-19T20:59:49.000+01:00
- coupon id cache doesn’t need to be read on every admin screen, only
the polylang strings table screen
 - coupon ids don’t need to be cached per language
diff --git a/src/Hyyan/WPI/Coupon.php b/src/Hyyan/WPI/Coupon.php
@@ -138,9 +138,13 @@ public function translateMessage($value, $coupon)
     public function adminRegisterCouponStrings()
     {
         if (is_admin() && (!is_ajax())) {
-            $this->registerCouponStringsForTranslation();
+        global $pagenow;
+            if ( ($pagenow) && ( $pagenow == 'admin.php' ) && ($_GET[ 'page' ] == 'mlang_strings') ) {
+                  $this->registerCouponStringsForTranslation();
+            }
         }
-    }
+	  }
+
     /**
      * Register coupon titles adn descriptions in Polylang's Strings translations table.
      */
@@ -198,8 +202,7 @@ private function getCoupons()
     {
         global $woocommerce;
         
-        $locale = (function_exists('pll_current_language')) ? pll_current_language('locale') : get_locale();
-        $tKey = 'coupons-' . $locale;
+        $tKey	 = 'coupons-ids';
         
         $coupon_ids = get_transient($tKey);
         if ($coupon_ids) {
