Merge branch 'bugfix/notice' into 'master'

fix(notice): Notice error when cannot create log file

See merge request !15

CHANGELOG.md

+## [1.6.2] - 2020-07-21
+### Fixed
+- Notice error when cannot create log file
+
 ## [1.6.1] - 2020-05-25
 ### Fixed
 - WooCommerceCapture checks if WC exists before proceeding

src/WP/WPCapture.php

@@ -27,7 +27,7 @@ class WPCapture {
 				),
 				$dir
 			),
-			WPDesk\Notice\Notice::NOTICE_TYPE_ERROR
+			\WPDesk\Notice\Notice::NOTICE_TYPE_ERROR
 		);
 	}
 
@@ -46,7 +46,7 @@ class WPCapture {
 				),
 				$file
 			),
-			WPDesk\Notice\Notice::NOTICE_TYPE_ERROR
+			\WPDesk\Notice\Notice::NOTICE_TYPE_ERROR
 		);
 	}
 
@@ -87,6 +87,10 @@ class WPCapture {
 			}
 		}
 
+        if (!is_writable($log_file)) {
+            $this->add_notice_for_file($log_file);
+            return false;
+        }
         return true;
 	}
 

src/WPDeskLoggerFactory.php

@@ -114,8 +114,10 @@ class WPDeskLoggerFactory extends BasicLoggerFactory
      */
     private function appendMainLog($logger) {
         $wpCapture = $this->captureWPLog();
+        if (is_writable($wpCapture->get_log_file())) {
             $this->appendFileLog($logger, $wpCapture->get_log_file());
         }
+    }
 
     /**
      * @param Logger $logger