Changeset 3074869

Timestamp:

04/22/2024 08:58:21 AM (21 months ago)

Author:

brandonpirk

Message:

Check Logfile data is not empty for logging

Location:

kassa-at-for-woocommerce/trunk

Files:

• api-connection.php (modified) (1 diff)
• kassa-at-for-woocommerce.php (modified) (5 diffs)
• stock-syncro.php (modified) (1 diff)

kassa-at-for-woocommerce/trunk/api-connection.php

@@ -35 +35 @@
         $host = 'https://' . kaw_get_subdomain() . '.kassa.at';
     } elseif ( kaw_get_envirement_mode() === 'dev' ) {
-        $host = 'http://' . kaw_get_subdomain() . '.kassa.at.10.0.0.13.nip.io:3000';
+        $host = 'http://' . kaw_get_subdomain() . '.kassa.at.192.168.165.45.nip.io:3000';
     }
     return $host;

kassa-at-for-woocommerce/trunk/kassa-at-for-woocommerce.php

@@ -5 +5 @@
      * Domain Path: /languages
      * Description: An API Plugin which corresponds with a KASSA.AT account to synchronize online and offline stock-changes.
-     * Version:     1.1.0
+     * Version:     1.1.1
      * Author:      Faxonline GmbH
      * Author URI:  https://www.kassa.at
@@ -47 +47 @@
 
 /**
+ * Check if variable is null or empty
+ */
+function IsNullOrEmptyString($var){
+    return ($var === null || trim($var) === '');
+}
+
+
+/**
  * Asks for the log file and writes whatever is in $data to it.
  *
@@ -58 +66 @@
         $filestring = file_get_contents( $logfile ); /* phpcs:ignore */
         $filejson   = json_decode( $filestring );
-        array_unshift(
-            $filejson,
-            array(
-                'datetime' => gmdate( 'd-m-Y H:i:s' ),
-                'type'     => $type,
-                'data'     => kaw_log_data_formating( $type, $data ),
-            )
-        );
-        $filejson   = kaw_cut_log( $filejson );
-        $fileresult = wp_json_encode( $filejson );
-        file_put_contents( $logfile, $fileresult ); /* phpcs:ignore */
+
+        if( !empty($data) && !empty($filestring) && !empty($logfile) && !empty($filejson) ) {
+            array_unshift(
+                $filejson,
+                array(
+                    'datetime' => gmdate( 'd-m-Y H:i:s' ),
+                    'type'     => $type,
+                    'data'     => kaw_log_data_formating( $type, $data ),
+                )
+            );
+            $filejson   = kaw_cut_log( $filejson );
+            $fileresult = wp_json_encode( $filejson );
+            file_put_contents( $logfile, $fileresult ); /* phpcs:ignore */
+        }
     }
 }
@@ -149 +160 @@
 function kaw_enable_log() {
     if ( get_option( 'kaw-logging' ) ) {
+
         if ( get_option( 'kaw-logging' ) === 'enabled' ) {
             return true;
@@ -355 +367 @@
     $bt     = debug_backtrace(); /* phpcs:ignore */
     $caller = array_shift( $bt );
+
     if ( count( explode( 'wp-content', $caller['file'] ) ) === 2 ) {
         $path = '/wp-content' . explode( 'wp-content', $caller['file'] )[1];

kassa-at-for-woocommerce/trunk/stock-syncro.php

@@ -73 +73 @@
         )
     );
+
     if ( 1 !== $inventory->length ) {
         kaw_log_data(