Merge pull request #33 from The-3Labs-Team/params

Claudio-Emmolo · web-flow · commit 7ad87cb89519 · 2025-11-28T16:48:11.000+01:00
Fix ad rendering logic to prepend ads to current element's outertext
diff --git a/src/AdsPostParser.php b/src/AdsPostParser.php
@@ -54,7 +54,7 @@ public function appendAdvertising(array $params = []): string
                 }
 
                 try {
-                    $currentElement->outertext .= Blade::render('ads-post-parser::ads'.array_keys($thresholds)[$adsCount], ['params' => $params]);
+                    $currentElement->outertext = Blade::render('ads-post-parser::ads'.array_keys($thresholds)[$adsCount], ['params' => $params]).$currentElement->outertext;
                 } catch (\Exception $e) {
                     // Content without ADV
                 }

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ public function appendAdvertising(array $params = []): string`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`try {`
`57`		`- $currentElement->outertext .= Blade::render('ads-post-parser::ads'.array_keys($thresholds)[$adsCount], ['params' => $params]);`
	`57`	`+ $currentElement->outertext = Blade::render('ads-post-parser::ads'.array_keys($thresholds)[$adsCount], ['params' => $params]).$currentElement->outertext;`
`58`	`58`	`} catch (\Exception $e) {`
`59`	`59`	`// Content without ADV`
`60`	`60`	`}`