vitormicillo / pdf-fusion-laravel

src/Providers/PdfFusionServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      */
     public function register()
     {
-      $this->app->singleton('PdfFusionLaravel', function ($app) {
+      $this->app->singleton('PdfFusionLaravel', function($app) {
           $PdfFusion = new PdfFusionLaravel($app['files']);
           return $PdfFusion;
       });

src/PdfFusionLaravel.php

Indentation +219 added lines, -219 removed lines

@@ -6,226 +6,226 @@
 use Illuminate\Support\Str;
 
 class PdfFusionLaravel {
-	/**
-	 * Access the filesystem on an oop base
-	 *
-	 * @var Filesystem
-	 */
-	protected $filesystem = Filesystem::class;
-	/**
-	 * Hold all the files which will be merged
-	 *
-	 * @var Collection
-	 */
-	protected $files = Collection::class;
-	/**
-	 * Holds every tmp file so they can be removed during the deconstruction
-	 *
-	 * @var Collection
-	 */
-	protected $tmpFiles = Collection::class;
-	/**
-	 * The actual PDF Service
-	 *
-	 * @var FPDI
-	 */
-	protected $fpdi = Fpdi::class;
-	/**
-	 * The final file name
-	 *
-	 * @var string
-	 */
-	protected $fileName = 'undefined.pdf';
-	/**
-	 * Construct and initialize a new instance
-	 * @param Filesystem $Filesystem
-	 */
-	public function __construct(Filesystem $filesystem){
-		$this->filesystem = $filesystem;
-		$this->createDirectoryForTemporaryFiles();
-		$this->fpdi = new Fpdi();
-		$this->tmpFiles = collect([]);
-		$this->files = collect([]);
-	}
-	/**
-	 * The class deconstructor method
-	 */
-	public function __destruct() {
-		$filesystem = $this->filesystem;
-		$this->tmpFiles->each(function($filePath) use($filesystem){
-			$filesystem->delete($filePath);
-		});
-	}
-	/**
-	 * Initialize a new internal instance of FPDI in order to prevent any problems with shared resources
-	 * Please visit https://www.setasign.com/products/fpdi/manual/#p-159 for more information on this issue
-	 *
-	 * @return self
-	 */
-	public function init(){
-		return $this;
-	}
-	/**
-	 * Stream the merged PDF content
-	 *
-	 * @return string
-	 */
-	public function inline(){
-		return $this->fpdi->Output($this->fileName, 'I');
-	}
-	/**
-	 * Download the merged PDF content
-	 *
-	 * @return string
-	 */
-	public function download(){
-		return $this->fpdi->Output($this->fileName, 'D');
-	}
-	/**
-	 * Save the merged PDF content to the filesystem
-	 *
-	 * @return string
-	 */
-	public function save($filePath = null){
-		return $this->filesystem->put($filePath ? $filePath : $this->fileName, $this->string());
-	}
-	/**
-	 * Get the merged PDF content as binary string
-	 *
-	 * @return string
-	 */
-	public function string(){
-		return $this->fpdi->Output($this->fileName, 'S');
-	}
-	/**
-	 * Set the generated PDF fileName
-	 * @param string $fileName
-	 *
-	 * @return string
-	 */
-	public function setFileName($fileName){
-		$this->fileName = $fileName;
-		return $this;
-	}
-	/**
-	 * Add a PDF for inclusion in the merge with a binary string. Pages should be formatted: 1,3,6, 12-16.
-	 * @param string $string
-	 * @param mixed $pages
-	 * @param mixed $orientation
-	 *
-	 * @return void
-	 */
-	public function addPDFString($string, $pages = 'all', $orientation = null){
-		$filePath = storage_path('tmp/'.Str::random(16).'.pdf');
-		$this->filesystem->put($filePath, $string);
-		$this->tmpFiles->push($filePath);
-		return $this->addPathToPDF($filePath, $pages, $orientation);
-	}
-	/**
-	 * Add a PDF for inclusion in the merge with a valid file path. Pages should be formatted: 1,3,6, 12-16.
-	 * @param string $filePath
-	 * @param string $pages
-	 * @param string $orientation
-	 *
-	 * @return self
-	 *
-	 * @throws \Exception if the given pages aren't correct
-	 */
-	public function addPathToPDF($filePath, $pages = 'all', $orientation = null) {
-		if (file_exists($filePath)) {
-		$filePath = $this->convertPDFVersion($filePath);
-		if (!is_array($pages) && strtolower($pages) != 'all') {
-			throw new \Exception($filePath."'s pages could not be validated");
-		}
-		$this->files->push([
-			'name'  => $filePath,
-			'pages' => $pages,
-			'orientation' => $orientation
-		]);
-		} else {
-			throw new \Exception("Could not locate PDF on '$filePath'");
-		}
-		return $this;
-	}
-	/**
-	 * Merges your provided PDFs and outputs to specified location.
-	 * @param string $orientation
-	 *
-	 * @return void
-	 *
-	 * @throws \Exception if there are now PDFs to merge
-	 */
-	public function duplexMerge($orientation = 'P'){
-		$this->merge($orientation, true);
-	}
+ /**
+  * Access the filesystem on an oop base
+  *
+  * @var Filesystem
+  */
+ protected $filesystem = Filesystem::class;
+ /**
+  * Hold all the files which will be merged
+  *
+  * @var Collection
+  */
+ protected $files = Collection::class;
+ /**
+  * Holds every tmp file so they can be removed during the deconstruction
+  *
+  * @var Collection
+  */
+ protected $tmpFiles = Collection::class;
+ /**
+  * The actual PDF Service
+  *
+  * @var FPDI
+  */
+ protected $fpdi = Fpdi::class;
+ /**
+  * The final file name
+  *
+  * @var string
+  */
+ protected $fileName = 'undefined.pdf';
+ /**
+  * Construct and initialize a new instance
+  * @param Filesystem $Filesystem
+  */
+ public function __construct(Filesystem $filesystem){
+  $this->filesystem = $filesystem;
+  $this->createDirectoryForTemporaryFiles();
+  $this->fpdi = new Fpdi();
+  $this->tmpFiles = collect([]);
+  $this->files = collect([]);
+ }
+ /**
+  * The class deconstructor method
+  */
+ public function __destruct() {
+  $filesystem = $this->filesystem;
+  $this->tmpFiles->each(function($filePath) use($filesystem){
+   $filesystem->delete($filePath);
+  });
+ }
+ /**
+  * Initialize a new internal instance of FPDI in order to prevent any problems with shared resources
+  * Please visit https://www.setasign.com/products/fpdi/manual/#p-159 for more information on this issue
+  *
+  * @return self
+  */
+ public function init(){
+  return $this;
+ }
+ /**
+  * Stream the merged PDF content
+  *
+  * @return string
+  */
+ public function inline(){
+  return $this->fpdi->Output($this->fileName, 'I');
+ }
+ /**
+  * Download the merged PDF content
+  *
+  * @return string
+  */
+ public function download(){
+  return $this->fpdi->Output($this->fileName, 'D');
+ }
+ /**
+  * Save the merged PDF content to the filesystem
+  *
+  * @return string
+  */
+ public function save($filePath = null){
+  return $this->filesystem->put($filePath ? $filePath : $this->fileName, $this->string());
+ }
+ /**
+  * Get the merged PDF content as binary string
+  *
+  * @return string
+  */
+ public function string(){
+  return $this->fpdi->Output($this->fileName, 'S');
+ }
+ /**
+  * Set the generated PDF fileName
+  * @param string $fileName
+  *
+  * @return string
+  */
+ public function setFileName($fileName){
+  $this->fileName = $fileName;
+  return $this;
+ }
+ /**
+  * Add a PDF for inclusion in the merge with a binary string. Pages should be formatted: 1,3,6, 12-16.
+  * @param string $string
+  * @param mixed $pages
+  * @param mixed $orientation
+  *
+  * @return void
+  */
+ public function addPDFString($string, $pages = 'all', $orientation = null){
+  $filePath = storage_path('tmp/'.Str::random(16).'.pdf');
+  $this->filesystem->put($filePath, $string);
+  $this->tmpFiles->push($filePath);
+  return $this->addPathToPDF($filePath, $pages, $orientation);
+ }
+ /**
+  * Add a PDF for inclusion in the merge with a valid file path. Pages should be formatted: 1,3,6, 12-16.
+  * @param string $filePath
+  * @param string $pages
+  * @param string $orientation
+  *
+  * @return self
+  *
+  * @throws \Exception if the given pages aren't correct
+  */
+ public function addPathToPDF($filePath, $pages = 'all', $orientation = null) {
+  if (file_exists($filePath)) {
+  $filePath = $this->convertPDFVersion($filePath);
+  if (!is_array($pages) && strtolower($pages) != 'all') {
+   throw new \Exception($filePath."'s pages could not be validated");
+  }
+  $this->files->push([
+   'name'  => $filePath,
+   'pages' => $pages,
+   'orientation' => $orientation
+  ]);
+  } else {
+   throw new \Exception("Could not locate PDF on '$filePath'");
+  }
+  return $this;
+ }
+ /**
+  * Merges your provided PDFs and outputs to specified location.
+  * @param string $orientation
+  *
+  * @return void
+  *
+  * @throws \Exception if there are now PDFs to merge
+  */
+ public function duplexMerge($orientation = 'P'){
+  $this->merge($orientation, true);
+ }
 
-	public function merge($orientation = 'P', $duplex = false) {
-		if ($this->files->count() == 0) {
-			throw new \Exception("No PDFs to merge.");
-		}
-		$fpdi = $this->fpdi;
-		$files = $this->files;
-		foreach($files as $index => $file){
-			$file['orientation'] = is_null($file['orientation']) ? $orientation : $file['orientation'];
-			$count = $fpdi->setSourceFile($file['name']);
-			if($file['pages'] == 'all') {
-				$pages = $count;
-				for ($i = 1; $i <= $count; $i++) {
-					$template   = $fpdi->importPage($i);
-					$size       = $fpdi->getTemplateSize($template);
-					$fpdi->AddPage($file['orientation'], [$size['width'], $size['height']]);
-					$fpdi->useTemplate($template);
-				}
-			}else {
-				$pages = count($file['pages']);
-				foreach ($file['pages'] as $page) {
-					if (!$template = $fpdi->importPage($page)) {
-					throw new \Exception("Could not load page '$page' in PDF '".$file['name']."'. Check that the page exists.");
-					}
-					$size = $fpdi->getTemplateSize($template);
-					$fpdi->AddPage($file['orientation'], [$size['width'], $size['height']]);
-					$fpdi->useTemplate($template);
-				}
-			}
-			if ($duplex && $pages % 2 && $index < (count($files) - 1)) {
-				$fpdi->AddPage($file['orientation'], [$size['width'], $size['height']]);
-			}
-		}
-	}
+ public function merge($orientation = 'P', $duplex = false) {
+  if ($this->files->count() == 0) {
+   throw new \Exception("No PDFs to merge.");
+  }
+  $fpdi = $this->fpdi;
+  $files = $this->files;
+  foreach($files as $index => $file){
+   $file['orientation'] = is_null($file['orientation']) ? $orientation : $file['orientation'];
+   $count = $fpdi->setSourceFile($file['name']);
+   if($file['pages'] == 'all') {
+    $pages = $count;
+    for ($i = 1; $i <= $count; $i++) {
+     $template   = $fpdi->importPage($i);
+     $size       = $fpdi->getTemplateSize($template);
+     $fpdi->AddPage($file['orientation'], [$size['width'], $size['height']]);
+     $fpdi->useTemplate($template);
+    }
+   }else {
+    $pages = count($file['pages']);
+    foreach ($file['pages'] as $page) {
+     if (!$template = $fpdi->importPage($page)) {
+     throw new \Exception("Could not load page '$page' in PDF '".$file['name']."'. Check that the page exists.");
+     }
+     $size = $fpdi->getTemplateSize($template);
+     $fpdi->AddPage($file['orientation'], [$size['width'], $size['height']]);
+     $fpdi->useTemplate($template);
+    }
+   }
+   if ($duplex && $pages % 2 && $index < (count($files) - 1)) {
+    $fpdi->AddPage($file['orientation'], [$size['width'], $size['height']]);
+   }
+  }
+ }
 
-	/**
-	 * Converts PDF if version is above 1.4
-	 * @param string $filePath
-	 *
-	 * @return string
-	 */
-	protected function convertPDFVersion($filePath){
-		$pdf = fopen($filePath, "r");
-		$first_line = fgets($pdf);
-		fclose($pdf);
-		//extract version number
-		preg_match_all('!\d+!', $first_line, $matches);
-		$pdfversion = implode('.', $matches[0]);
-		if($pdfversion > "1.4"){
-			$newFilePath = storage_path('tmp/' . Str::random(16) . '.pdf');
-			//execute shell script that converts PDF to correct version and saves it to tmp folder
-			shell_exec('gs -dBATCH -dNOPAUSE -q -sDEVICE=pdfwrite -sOutputFile="'. $newFilePath . '" "' . $filePath . '"');
-			$this->tmpFiles->push($newFilePath);
-			$filePath = $newFilePath;
-		}
-		//return correct file path
-		return $filePath;
-	}
+ /**
+  * Converts PDF if version is above 1.4
+  * @param string $filePath
+  *
+  * @return string
+  */
+ protected function convertPDFVersion($filePath){
+  $pdf = fopen($filePath, "r");
+  $first_line = fgets($pdf);
+  fclose($pdf);
+  //extract version number
+  preg_match_all('!\d+!', $first_line, $matches);
+  $pdfversion = implode('.', $matches[0]);
+  if($pdfversion > "1.4"){
+   $newFilePath = storage_path('tmp/' . Str::random(16) . '.pdf');
+   //execute shell script that converts PDF to correct version and saves it to tmp folder
+   shell_exec('gs -dBATCH -dNOPAUSE -q -sDEVICE=pdfwrite -sOutputFile="'. $newFilePath . '" "' . $filePath . '"');
+   $this->tmpFiles->push($newFilePath);
+   $filePath = $newFilePath;
+  }
+  //return correct file path
+  return $filePath;
+ }
 
-	/**
-	 * Create a the temporary file directory if it doesn't exist.
-	 *
-	 * @return void
-	 */
-	protected function createDirectoryForTemporaryFiles(): void
-	{
-		if (! $this->filesystem->isDirectory(storage_path('tmp'))) {
-			$this->filesystem->makeDirectory(storage_path('tmp'));
-		}
-	}
+ /**
+  * Create a the temporary file directory if it doesn't exist.
+  *
+  * @return void
+  */
+ protected function createDirectoryForTemporaryFiles(): void
+ {
+  if (! $this->filesystem->isDirectory(storage_path('tmp'))) {
+   $this->filesystem->makeDirectory(storage_path('tmp'));
+  }
+ }
 }

Spacing +17 added lines, -17 removed lines

@@ -40,7 +40,7 @@ 
 	 * Construct and initialize a new instance
 	 * @param Filesystem $Filesystem
 	 */
-	public function __construct(Filesystem $filesystem){
+	public function __construct(Filesystem $filesystem) {
 		$this->filesystem = $filesystem;
 		$this->createDirectoryForTemporaryFiles();
 		$this->fpdi = new Fpdi();
@@ -62,7 +62,7 @@ 
 	 *
 	 * @return self
 	 */
-	public function init(){
+	public function init() {
 		return $this;
 	}
 	/**
@@ -70,7 +70,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function inline(){
+	public function inline() {
 		return $this->fpdi->Output($this->fileName, 'I');
 	}
 	/**
@@ -78,7 +78,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function download(){
+	public function download() {
 		return $this->fpdi->Output($this->fileName, 'D');
 	}
 	/**
@@ -86,7 +86,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function save($filePath = null){
+	public function save($filePath = null) {
 		return $this->filesystem->put($filePath ? $filePath : $this->fileName, $this->string());
 	}
 	/**
@@ -94,7 +94,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function string(){
+	public function string() {
 		return $this->fpdi->Output($this->fileName, 'S');
 	}
 	/**
@@ -103,7 +103,7 @@ 
 	 *
 	 * @return string
 	 */
-	public function setFileName($fileName){
+	public function setFileName($fileName) {
 		$this->fileName = $fileName;
 		return $this;
 	}
@@ -115,7 +115,7 @@ 
 	 *
 	 * @return void
 	 */
-	public function addPDFString($string, $pages = 'all', $orientation = null){
+	public function addPDFString($string, $pages = 'all', $orientation = null) {
 		$filePath = storage_path('tmp/'.Str::random(16).'.pdf');
 		$this->filesystem->put($filePath, $string);
 		$this->tmpFiles->push($filePath);
@@ -155,7 +155,7 @@ 
 	 *
 	 * @throws \Exception if there are now PDFs to merge
 	 */
-	public function duplexMerge($orientation = 'P'){
+	public function duplexMerge($orientation = 'P') {
 		$this->merge($orientation, true);
 	}
 
@@ -165,10 +165,10 @@ 
 		}
 		$fpdi = $this->fpdi;
 		$files = $this->files;
-		foreach($files as $index => $file){
+		foreach ($files as $index => $file) {
 			$file['orientation'] = is_null($file['orientation']) ? $orientation : $file['orientation'];
 			$count = $fpdi->setSourceFile($file['name']);
-			if($file['pages'] == 'all') {
+			if ($file['pages'] == 'all') {
 				$pages = $count;
 				for ($i = 1; $i <= $count; $i++) {
 					$template   = $fpdi->importPage($i);
@@ -176,7 +176,7 @@ 
 					$fpdi->AddPage($file['orientation'], [$size['width'], $size['height']]);
 					$fpdi->useTemplate($template);
 				}
-			}else {
+			} else {
 				$pages = count($file['pages']);
 				foreach ($file['pages'] as $page) {
 					if (!$template = $fpdi->importPage($page)) {
@@ -199,17 +199,17 @@ 
 	 *
 	 * @return string
 	 */
-	protected function convertPDFVersion($filePath){
+	protected function convertPDFVersion($filePath) {
 		$pdf = fopen($filePath, "r");
 		$first_line = fgets($pdf);
 		fclose($pdf);
 		//extract version number
 		preg_match_all('!\d+!', $first_line, $matches);
 		$pdfversion = implode('.', $matches[0]);
-		if($pdfversion > "1.4"){
-			$newFilePath = storage_path('tmp/' . Str::random(16) . '.pdf');
+		if ($pdfversion > "1.4") {
+			$newFilePath = storage_path('tmp/'.Str::random(16).'.pdf');
 			//execute shell script that converts PDF to correct version and saves it to tmp folder
-			shell_exec('gs -dBATCH -dNOPAUSE -q -sDEVICE=pdfwrite -sOutputFile="'. $newFilePath . '" "' . $filePath . '"');
+			shell_exec('gs -dBATCH -dNOPAUSE -q -sDEVICE=pdfwrite -sOutputFile="'.$newFilePath.'" "'.$filePath.'"');
 			$this->tmpFiles->push($newFilePath);
 			$filePath = $newFilePath;
 		}
@@ -224,7 +224,7 @@ 
 	 */
 	protected function createDirectoryForTemporaryFiles(): void
 	{
-		if (! $this->filesystem->isDirectory(storage_path('tmp'))) {
+		if (!$this->filesystem->isDirectory(storage_path('tmp'))) {
 			$this->filesystem->makeDirectory(storage_path('tmp'));
 		}
 	}

Braces +1 added lines, -1 removed lines

@@ -176,7 +176,7 @@
 					$fpdi->AddPage($file['orientation'], [$size['width'], $size['height']]);
 					$fpdi->useTemplate($template);
 				}
-			}else {
+			} else {
 				$pages = count($file['pages']);
 				foreach ($file['pages'] as $page) {
 					if (!$template = $fpdi->importPage($page)) {