Merge pull request #19 from Codaone/Branch-1.1.1

Fix error error messages not disappearing after display
Codaone · May 5, 2017 · ab59761 · ab59761
2 parents b670a9f + f7186b9
commit ab59761
Showing 1 changed file with 11 additions and 14 deletions.
diff --git a/logitrail-woocommerce.php b/logitrail-woocommerce.php
@@ -3,7 +3,7 @@
 /*
     Plugin Name: Logitrail
     Description: Integrate checkout shipping with Logitrail
-    Version: 1.1.0
+    Version: 1.1.1
     Author: <a href="mailto:petri@codaone.fi">Petri Kanerva</a> | <a href="http://www.codaone.fi/">Codaone Oy</a>
 */
 
@@ -486,7 +486,7 @@ function logitrail_create_product($post_id) {
             $apic->setSecretKey( $settings['secret_key'] );
 
             if ( ! $product->get_sku() && !$product->get_type() == 'variable' ) {
-                $this->logitrail_set_error('SKU puuttuu tuotteesta "' . $product->get_title() . '". Tuotetta ei voitu viedä Logitrailin järjestelmään.');
+                self::logitrail_set_error('SKU puuttuu tuotteesta "' . $product->get_title() . '". Tuotetta ei voitu viedä Logitrailin järjestelmään.');
             } else {
                 if ($product->get_type() == 'variable') {
                     // Add variation products here
@@ -500,11 +500,11 @@ function logitrail_create_product($post_id) {
                         $child_title = $child->get_title() . ' - ' . $attributes;
 
                         if ( ! $child->get_sku() ) {
-                            $this->logitrail_set_error('SKU puuttuu tuotteesta "' . $child_title . '". Tuotetta ei voitu viedä Logitrailin järjestelmään.');
+                            self::logitrail_set_error('SKU puuttuu tuotteesta "' . $child_title . '". Tuotetta ei voitu viedä Logitrailin järjestelmään.');
                             continue;
                         }
                         if (in_array($child->get_sku(), $sku_array)) {
-                            $this->logitrail_set_error(
+                            self::logitrail_set_error(
                                 'Tuotteen "' . $child_title . '" SKU "'. $child->get_sku() .'" on jo lisätty Logitrailiin, 
                                 Tuotetta ei voitu viedä Logitrailin järjestelmään'
                             );
@@ -565,7 +565,7 @@ function logitrail_create_product($post_id) {
                         // separately, but just as a count
                         if ( count( $responses == 1 ) ) {
                             //wc_add_notice("Virhe siirrettäessä tuotetta Logitrailille", "notice");
-                            $this->logitrail_set_error('Virhe siirrettäessä tuotetta Logitrailille.');
+                            self::logitrail_set_error('Virhe siirrettäessä tuotetta Logitrailille.');
                         } else {
                             $errors ++;
                         }
@@ -637,12 +637,12 @@ public function export_products() {
                     $child_title = $child->get_title() . ' - ' . $attributes;
 
                     if ( ! $child->get_sku() ) {
-                        $this->logitrail_set_error('SKU puuttuu tuotteesta "' . $child_title . '". Tuotetta ei voitu viedä Logitrail-järjestelmään.');
+                        self::logitrail_set_error('SKU puuttuu tuotteesta "' . $child_title . '". Tuotetta ei voitu viedä Logitrail-järjestelmään.');
                         $errors++;
                         continue;
                     }
                     if (in_array($child->get_sku(), $sku_array)) {
-                        $this->logitrail_set_error('Tuotteen "' . $child_title . '" SKU "'. $child->get_sku() .'" on jo lisätty Logitrailiin. Tuotetta ei voitu viedä Logitrail-järjestelmään.');
+                        self::logitrail_set_error('Tuotteen "' . $child_title . '" SKU "'. $child->get_sku() .'" on jo lisätty Logitrailiin. Tuotetta ei voitu viedä Logitrail-järjestelmään.');
                         $errors++;
                         continue;
                     } else {
@@ -674,7 +674,7 @@ public function export_products() {
             } else {
                 if (!$this->logitrail_is_virtual($product) && $this->logitrail_shipping_enabled($product->get_id())) {
                     if (in_array($product->get_sku(), $sku_array)) {
-                        $this->logitrail_set_error('Tuotteen "' . $product->title . '" SKU "'. $product->get_sku() .'" on jo lisätty. Tuotetta ei voitu viedä Logitrail-järjestelmään.');
+                        self::logitrail_set_error('Tuotteen "' . $product->title . '" SKU "'. $product->get_sku() .'" on jo lisätty. Tuotetta ei voitu viedä Logitrail-järjestelmään.');
                         $errors++;
                         continue;
                     }
@@ -1018,9 +1018,6 @@ public static function logitrail_shipping_enabled($product_id) {
      * @return bool
      */
     public function logitrail_set_transient($key, $value, $identifier = '', $expiration = 604800) {
-        if (!$value) {
-            return false;
-        }
         if (!$identifier) {
             $identifier = WC()->session->get_customer_id();
         }
@@ -1041,15 +1038,15 @@ public static function logitrail_get_transient($key, $identifier = '') {
     }
 
     public static function logitrail_notifications() {
-        $notifications = self::logitrail_get_transient('notifications',  wp_get_current_user()->ID);
+        $notifications = self::logitrail_get_transient('notifications', wp_get_current_user()->ID);
         if (!$notifications) {
             return;
         }
 
         foreach($notifications as $notification) {
             printf( '<div class="%1$s"><p>%2$s</p></div>', $notification['class'], $notification['message'] );
         }
-        self::logitrail_set_transient('notifications', array());
+        self::logitrail_set_transient('notifications', array(), wp_get_current_user()->ID);
     }
 
     /**
@@ -1064,7 +1061,7 @@ public static function logitrail_set_error($message) {
                 'message' => $message
             );
 
-        self::logitrail_set_transient('notifications', $notifications);
+        self::logitrail_set_transient('notifications', $notifications, wp_get_current_user()->ID);
     }
 
     public static function get_debug_mode() {