Fixing: "syntax error, unexpected token "{"" (#117)

luka4 · web-flow · commit 0b3d81150594 · 2023-08-18T17:07:38.000+05:30
* Fixing: "syntax error, unexpected token "{""

* Fixing: "array to string conversion error"

* Fixing: "call to undefined method crit()"

* Fixing: "call to undefined method err()"
diff --git a/Logger/Logger.php b/Logger/Logger.php
@@ -64,6 +64,9 @@ public function __construct(
     public function debug($message, array $context = []): void
     {
         if ($this->configProvider->isDebugEnabled()) {
+            if (is_array($message)) {
+                $message = json_encode($message);
+            }
             $this->addRecord(static::DEBUG, strval($message),$context);
         }
     }
diff --git a/Model/CurrentProductProvider.php b/Model/CurrentProductProvider.php
@@ -68,7 +68,7 @@ public function getProduct()
                 }
                 $this->currentProduct = $this->productRepository->getById($product->getId());
             } catch (Exception $e) {
-                $this->bvLogger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                $this->bvLogger->critical($e->getMessage()."\n".$e->getTraceAsString());
 
                 return false;
             }
diff --git a/Model/Feed/Feed.php b/Model/Feed/Feed.php
@@ -89,7 +89,7 @@ public function generateFeed()
                 break;
             }
         } catch (Exception $e) {
-            $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+            $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
         }
         $this->logger->debug('End Bazaarvoice '.$this->typeId.' Feed Generation');
     }
@@ -121,7 +121,7 @@ public function exportFeedByStoreGroup()
                     'Failed to export daily '.$this->typeId.' feed for store group: '
                     .$storeGroup->getName()
                 );
-                $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
             }
         }
     }
@@ -143,7 +143,7 @@ public function exportFeedByStore()
                 }
             } catch (Exception $e) {
                 $this->logger->error('Failed to export daily '.$this->typeId.' feed for store: '.$store->getCode());
-                $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
             }
         }
     }
@@ -164,7 +164,7 @@ public function exportFeedByWebsite()
                 }
             } catch (Exception $e) {
                 $this->logger->error('Failed to export daily '.$this->typeId.' feed for website: '.$website->getName());
-                $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
             }
         }
     }
@@ -181,7 +181,7 @@ public function exportFeedByGlobal()
             }
         } catch (Exception $e) {
             $this->logger->error('Failed to export daily '.$this->typeId.' feed.');
-            $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+            $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
         }
     }
 
@@ -310,7 +310,7 @@ protected function uploadFeed($sourceFile, $destinationFile, $store = null)
                 $ioObject->mv($sourceFile, $sentFile);
             }
         } catch (Exception $e) {
-            $this->logger->err($e->getMessage());
+            $this->logger->error($e->getMessage());
         }
     }
 }
diff --git a/Model/Indexer/Eav.php b/Model/Indexer/Eav.php
@@ -182,7 +182,7 @@ public function executeFull()
             $this->logger->debug(__('Bazaarvoice Product Feed Index is being rebuilt via cron.'));
             $this->execute();
         } catch (Exception $e) {
-            $this->logger->err($e->getMessage()."\n".$e->getTraceAsString());
+            $this->logger->error($e->getMessage()."\n".$e->getTraceAsString());
         }
 
         return true;
@@ -255,7 +255,7 @@ public function execute($ids = [])
                 $this->logStats();
             }
         } catch (Exception $e) {
-            $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+            $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
         }
 
         return true;
@@ -720,7 +720,7 @@ private function populateIndexLocaleData($productIds, Store $store): void
                         }
                     }
                 } catch (Exception $e) {
-                    $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                    $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
                 }
             }
         }
@@ -1093,10 +1093,10 @@ private function saveProductIndexes()
                     $existingIndex->addData($bvIndex->getData());
                     $this->indexRepository->save($existingIndex);
                 } catch (Exception $e) {
-                    $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                    $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
                 }
             } catch (Exception $e) {
-                $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
             }
         }
     }
diff --git a/Model/Indexer/Flat.php b/Model/Indexer/Flat.php
@@ -196,7 +196,7 @@ public function executeFull()
             $this->logger->debug(__('Bazaarvoice Product Feed Index is being rebuilt.'));
             $this->execute();
         } catch (Exception $e) {
-            $this->logger->err($e->getMessage()."\n".$e->getTraceAsString());
+            $this->logger->error($e->getMessage()."\n".$e->getTraceAsString());
         }
 
         return true;
@@ -270,7 +270,7 @@ public function execute($ids = [])
                 $this->logStats();
             }
         } catch (Exception $e) {
-            $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+            $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
         }
 
         return true;
@@ -574,7 +574,7 @@ private function populateIndexStoreData($productIds, $store)
         try {
             $rows = $select->query();
         } catch (Exception $e) {
-            $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+            $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
             if (strpos($e->getMessage(), 'Column not found') !== false) {
                 $errorExplanation = 'The following "Column not found" error typically results from a product attribute missing from the flat product table. Please ensure that the attribute referenced in the error is set to Use In Product Listing = Yes, which should cause a reindex to add it to the product flat table that is being queried: ' . $e->getMessage();
                 throw new Exception($errorExplanation, 0, $e);
@@ -637,7 +637,7 @@ private function populateIndexStoreData($productIds, $store)
                 }
             }
 
-            if(!empty($indexData['category_external_id']) {
+            if(!empty($indexData['category_external_id'])) {
                 $indexData['category_external_id'] = str_replace('/', '-', $indexData['category_external_id']);
                 $indexData['category_external_id'] = str_replace('.html', '', $indexData['category_external_id']);
                 $indexData['category_external_id']
@@ -797,7 +797,7 @@ private function populateIndexLocaleData($productIds, $store)
                         }
                     }
                 } catch (Exception $e) {
-                    $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                    $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
                 }
             }
         }
@@ -1144,10 +1144,10 @@ private function saveProductIndexes()
                     $existingIndex->addData($bvIndex->getData());
                     $this->indexRepository->save($existingIndex);
                 } catch (Exception $e) {
-                    $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                    $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
                 }
             } catch (Exception $e) {
-                $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());
+                $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());
             }
         }
     }
diff --git a/ViewModel/Product.php b/ViewModel/Product.php
@@ -196,7 +196,7 @@ public function isConfigurable()
                 return $this->getProduct()->getTypeId() == Configurable::TYPE_CODE;
             }
         } catch (Exception $e) {
-            $this->bvLogger->crit($e->getMessage()."\n".$e->getTraceAsString());
+            $this->bvLogger->critical($e->getMessage()."\n".$e->getTraceAsString());
         }
 
         return false;

Original file line number	Diff line number	Diff line change
`@@ -64,6 +64,9 @@ public function __construct(`
`64`	`64`	`public function debug($message, array $context = []): void`
`65`	`65`	`{`
`66`	`66`	`if ($this->configProvider->isDebugEnabled()) {`
	`67`	`+ if (is_array($message)) {`
	`68`	`+ $message = json_encode($message);`
	`69`	`+ }`
`67`	`70`	`$this->addRecord(static::DEBUG, strval($message),$context);`
`68`	`71`	`}`
`69`	`72`	`}`
Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ public function getProduct()`
`68`	`68`	`}`
`69`	`69`	`$this->currentProduct = $this->productRepository->getById($product->getId());`
`70`	`70`	`} catch (Exception $e) {`
`71`		`- $this->bvLogger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`71`	`+ $this->bvLogger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`72`	`72`
`73`	`73`	`return false;`
`74`	`74`	`}`
Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ public function generateFeed()`
`89`	`89`	`break;`
`90`	`90`	`}`
`91`	`91`	`} catch (Exception $e) {`
`92`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`92`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`93`	`93`	`}`
`94`	`94`	`$this->logger->debug('End Bazaarvoice '.$this->typeId.' Feed Generation');`
`95`	`95`	`}`
`@@ -121,7 +121,7 @@ public function exportFeedByStoreGroup()`
`121`	`121`	`'Failed to export daily '.$this->typeId.' feed for store group: '`
`122`	`122`	`.$storeGroup->getName()`
`123`	`123`	`);`
`124`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`124`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`125`	`125`	`}`
`126`	`126`	`}`
`127`	`127`	`}`
`@@ -143,7 +143,7 @@ public function exportFeedByStore()`
`143`	`143`	`}`
`144`	`144`	`} catch (Exception $e) {`
`145`	`145`	`$this->logger->error('Failed to export daily '.$this->typeId.' feed for store: '.$store->getCode());`
`146`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`146`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`147`	`147`	`}`
`148`	`148`	`}`
`149`	`149`	`}`
`@@ -164,7 +164,7 @@ public function exportFeedByWebsite()`
`164`	`164`	`}`
`165`	`165`	`} catch (Exception $e) {`
`166`	`166`	`$this->logger->error('Failed to export daily '.$this->typeId.' feed for website: '.$website->getName());`
`167`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`167`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`168`	`168`	`}`
`169`	`169`	`}`
`170`	`170`	`}`
`@@ -181,7 +181,7 @@ public function exportFeedByGlobal()`
`181`	`181`	`}`
`182`	`182`	`} catch (Exception $e) {`
`183`	`183`	`$this->logger->error('Failed to export daily '.$this->typeId.' feed.');`
`184`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`184`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`185`	`185`	`}`
`186`	`186`	`}`
`187`	`187`
`@@ -310,7 +310,7 @@ protected function uploadFeed($sourceFile, $destinationFile, $store = null)`
`310`	`310`	`$ioObject->mv($sourceFile, $sentFile);`
`311`	`311`	`}`
`312`	`312`	`} catch (Exception $e) {`
`313`		`- $this->logger->err($e->getMessage());`
	`313`	`+ $this->logger->error($e->getMessage());`
`314`	`314`	`}`
`315`	`315`	`}`
`316`	`316`	`}`
Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ public function executeFull()`
`182`	`182`	`$this->logger->debug(__('Bazaarvoice Product Feed Index is being rebuilt via cron.'));`
`183`	`183`	`$this->execute();`
`184`	`184`	`} catch (Exception $e) {`
`185`		`- $this->logger->err($e->getMessage()."\n".$e->getTraceAsString());`
	`185`	`+ $this->logger->error($e->getMessage()."\n".$e->getTraceAsString());`
`186`	`186`	`}`
`187`	`187`
`188`	`188`	`return true;`
`@@ -255,7 +255,7 @@ public function execute($ids = [])`
`255`	`255`	`$this->logStats();`
`256`	`256`	`}`
`257`	`257`	`} catch (Exception $e) {`
`258`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`258`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`259`	`259`	`}`
`260`	`260`
`261`	`261`	`return true;`
`@@ -720,7 +720,7 @@ private function populateIndexLocaleData($productIds, Store $store): void`
`720`	`720`	`}`
`721`	`721`	`}`
`722`	`722`	`} catch (Exception $e) {`
`723`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`723`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`724`	`724`	`}`
`725`	`725`	`}`
`726`	`726`	`}`
`@@ -1093,10 +1093,10 @@ private function saveProductIndexes()`
`1093`	`1093`	`$existingIndex->addData($bvIndex->getData());`
`1094`	`1094`	`$this->indexRepository->save($existingIndex);`
`1095`	`1095`	`} catch (Exception $e) {`
`1096`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`1096`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`1097`	`1097`	`}`
`1098`	`1098`	`} catch (Exception $e) {`
`1099`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`1099`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`1100`	`1100`	`}`
`1101`	`1101`	`}`
`1102`	`1102`	`}`
Original file line number	Diff line number	Diff line change
`@@ -196,7 +196,7 @@ public function executeFull()`
`196`	`196`	`$this->logger->debug(__('Bazaarvoice Product Feed Index is being rebuilt.'));`
`197`	`197`	`$this->execute();`
`198`	`198`	`} catch (Exception $e) {`
`199`		`- $this->logger->err($e->getMessage()."\n".$e->getTraceAsString());`
	`199`	`+ $this->logger->error($e->getMessage()."\n".$e->getTraceAsString());`
`200`	`200`	`}`
`201`	`201`
`202`	`202`	`return true;`
`@@ -270,7 +270,7 @@ public function execute($ids = [])`
`270`	`270`	`$this->logStats();`
`271`	`271`	`}`
`272`	`272`	`} catch (Exception $e) {`
`273`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`273`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`274`	`274`	`}`
`275`	`275`
`276`	`276`	`return true;`
`@@ -574,7 +574,7 @@ private function populateIndexStoreData($productIds, $store)`
`574`	`574`	`try {`
`575`	`575`	`$rows = $select->query();`
`576`	`576`	`} catch (Exception $e) {`
`577`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`577`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`578`	`578`	`if (strpos($e->getMessage(), 'Column not found') !== false) {`
`579`	`579`	`$errorExplanation = 'The following "Column not found" error typically results from a product attribute missing from the flat product table. Please ensure that the attribute referenced in the error is set to Use In Product Listing = Yes, which should cause a reindex to add it to the product flat table that is being queried: ' . $e->getMessage();`
`580`	`580`	`throw new Exception($errorExplanation, 0, $e);`
`@@ -637,7 +637,7 @@ private function populateIndexStoreData($productIds, $store)`
`637`	`637`	`}`
`638`	`638`	`}`
`639`	`639`
`640`		`- if(!empty($indexData['category_external_id']) {`
	`640`	`+ if(!empty($indexData['category_external_id'])) {`
`641`	`641`	`$indexData['category_external_id'] = str_replace('/', '-', $indexData['category_external_id']);`
`642`	`642`	`$indexData['category_external_id'] = str_replace('.html', '', $indexData['category_external_id']);`
`643`	`643`	`$indexData['category_external_id']`
`@@ -797,7 +797,7 @@ private function populateIndexLocaleData($productIds, $store)`
`797`	`797`	`}`
`798`	`798`	`}`
`799`	`799`	`} catch (Exception $e) {`
`800`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`800`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`801`	`801`	`}`
`802`	`802`	`}`
`803`	`803`	`}`
`@@ -1144,10 +1144,10 @@ private function saveProductIndexes()`
`1144`	`1144`	`$existingIndex->addData($bvIndex->getData());`
`1145`	`1145`	`$this->indexRepository->save($existingIndex);`
`1146`	`1146`	`} catch (Exception $e) {`
`1147`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`1147`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`1148`	`1148`	`}`
`1149`	`1149`	`} catch (Exception $e) {`
`1150`		`- $this->logger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`1150`	`+ $this->logger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`1151`	`1151`	`}`
`1152`	`1152`	`}`
`1153`	`1153`	`}`
Original file line number	Diff line number	Diff line change
`@@ -196,7 +196,7 @@ public function isConfigurable()`
`196`	`196`	`return $this->getProduct()->getTypeId() == Configurable::TYPE_CODE;`
`197`	`197`	`}`
`198`	`198`	`} catch (Exception $e) {`
`199`		`- $this->bvLogger->crit($e->getMessage()."\n".$e->getTraceAsString());`
	`199`	`+ $this->bvLogger->critical($e->getMessage()."\n".$e->getTraceAsString());`
`200`	`200`	`}`
`201`	`201`
`202`	`202`	`return false;`