README.md

@@ -32,9 +32,8 @@ JASPER_SERVER_PASSWORD="<Password de acesso>"
 ```
 
 ### Fazendo requisições ao servidor
-Para efetuar as requisições ao servidor jasper, 
-utilize o serviço [**JasperServerRequestServiceContract**](src/app/Services/Contracts/JasperServerRequestServiceContract.php). Este serviço, além de conter 
-o método **call(uri)** para efetuar a requisição e os métodos auxiliares de configuração.
+Para efetuar as requisições ao servidor jasper, utilize o serviço **JasperServerRequestServiceContract**. Este serviço, além de conter 
+o método **call(uri)** para efetuar a requisição, e os métodos auxiliares de configuração.
 
 Segue abaixo exemplo de requisição para um relatório chamado **relatorio_teste**, contendo um parâmetro chamado **parametro_um**:
 
@@ -45,52 +44,14 @@ $report = $this->jasperServerRequestService
     ->addInputControl('parametro_um', 'valor_qualquer')
     ->setRequestTimeout(300)
     ->call($uri);
+
+echo $report;
 ```
 
 Por padrão os relatórios são retornados no formato **html**. Para especificar outro formato, basta utilizar o método
-**setFormat** disponível no [serviço](src/app/Services/JasperServerRequestService.php) e o enum [ReportFormatEnum](src/app/Enums/ReportFormatEnum.php) com os formatos disponíveis:
+**setFormat** disponível no serviço:
 ```shell
 $this->jasperServerRequestService
-  ->setFormat(ReportFormatEnum::PDF)
+  ->setFormat('pdf')
   ...
-```
-
-#### Métodos inline() e download()
-Caso queira forçar a renderização inline do relatório, 
-basta utilizar a função ```inline()``` após o método ```call()```. Veja o exemplo abaixo:
-```php
-$uri = '/relatorio_teste';
-
-$this->jasperServerRequestService
-    ->addInputControl('parametro_um', 'valor_qualquer')
-    ->setRequestTimeout(300)
-    ->call($uri)
-    ->inline();
-```
-Apenas arquivos que o navegador consegue renderizar serão 
-exibidos. Caso contrário, o arquivo será baixado.
-
-Caso queira forçar o download do arquivo, basta utilizar o método ```download('nome_do_arquivo')```, veja o exemplo abaixo:
-```php
-$uri = '/relatorio_teste';
-
-$this->jasperServerRequestService
-    ->addInputControl('parametro_um', 'valor_qualquer')
-    ->setRequestTimeout(300)
-    ->call($uri)
-    ->download();
-```
-O nome do arquivo é opcional e, caso não seja informado, 
-terá um valor randômico atribuído.
-
-Caso você necessite obter o valor binário do relatório, pode utilizar
-o método getReport(). Exemplo:
-```php
-$uri = '/relatorio_teste';
-
-$report = $this->jasperServerRequestService
-->addInputControl('parametro_um', 'valor_qualquer')
-->setRequestTimeout(300)
-->call($uri)
-->getReport();
 ```

src/app/Enums/ContentDispositionEnum.php

@@ -1,20 +0,0 @@
-<?php
-
-namespace Ae3\JasperServer\Laravel\Integrator\app\Enums;
-
-class ContentDispositionEnum
-{
-    public const INLINE = "inline";
-    public const ATTACHMENT = "attachment";
-
-    /**
-     * @return array
-     */
-    public static function all(): array
-    {
-        return [
-            self::INLINE,
-            self::ATTACHMENT
-        ];
-    }
-}

src/app/Enums/ReportFormatEnum.php

@@ -1,73 +0,0 @@
-<?php
-
-namespace Ae3\JasperServer\Laravel\Integrator\app\Enums;
-
-class ReportFormatEnum
-{
-    public const PDF = "pdf";
-    public const HTML = "html";
-    public const XML = "xml";
-    public const CSV = "csv";
-    public const DOCX = "docx";
-    public const ODT = "odt";
-    public const ODS = "ods";
-    public const XLSX = "xlsx";
-
-    /**
-     * @return array
-     */
-    public static function all(): array
-    {
-        return [
-            self::PDF,
-            self::HTML,
-            self::XML,
-            self::CSV,
-            self::DOCX,
-            self::ODT,
-            self::ODS,
-            self::XLSX,
-        ];
-    }
-
-    /**
-     * @return string[]
-     */
-    public static function downloadable(): array
-    {
-        return [
-            self::PDF,
-            self::XML,
-            self::CSV,
-            self::DOCX,
-            self::ODT,
-            self::ODS,
-            self::XLSX,
-        ];
-    }
-
-    /**
-     * @param string $format
-     * @return string
-     */
-    public static function getMimeType(string $format): string
-    {
-        switch ($format) {
-            case self::PDF:
-                return 'application/pdf';
-            case self::XML:
-                return 'application/xml';
-            case self::CSV:
-                return 'text/csv';
-            case self::DOCX:
-                return 'application/vnd.openxmlformats-officedocument.wordprocessingml.document';
-            case self::ODT:
-                return 'application/vnd.oasis.opendocument.text';
-            case self::XLSX:
-            case self::ODS:
-                return 'application/vnd.oasis.opendocument.spreadsheet';
-            default:
-                return 'text/html';
-        }
-    }
-}

src/app/Services/Contracts/JasperServerRequestServiceContract.php

@@ -73,25 +73,10 @@ interface JasperServerRequestServiceContract
      */
     public function setContentDisposition(string $contentDisposition): JasperServerRequestService;
 
-    /**
-     * @return string
-     */
-    public function getReport(): string;
-
     /**
      * @param string $uri
-     * @return $this
-     */
-    public function call(string $uri): JasperServerRequestService;
-
-    /**
-     * @return void
-     */
-    public function inline(): void;
-
-    /**
-     * @param string|null $filename
-     * @return void
+     * @param string|null $pdfFilename
+     * @return string
      */
-    public function download(?string $filename = null): void;
+    public function call(string $uri, ?string $pdfFilename): string;
 }

src/app/Services/JasperServerRequestService.php

@@ -2,7 +2,6 @@
 
 namespace Ae3\JasperServer\Laravel\Integrator\app\Services;
 
-use Ae3\JasperServer\Laravel\Integrator\app\Enums\ContentDispositionEnum;
 use Ae3\JasperServer\Laravel\Integrator\app\Enums\ReportFormatEnum;
 use Ae3\JasperServer\Laravel\Integrator\app\Services\Contracts\JasperServerRequestServiceContract;
 use Jaspersoft\Client\Client;
@@ -67,12 +66,7 @@ class JasperServerRequestService implements JasperServerRequestServiceContract
     /**
      * @var string
      */
-    protected $contentDisposition;
-
-    /**
-     * @var string
-     */
-    protected $report;
+    protected $contentDisposition = 'inline';
 
     /**
      * @param Client $client
@@ -81,8 +75,7 @@ class JasperServerRequestService implements JasperServerRequestServiceContract
         Client $client
     ) {
         $this->client = $client;
-        $this->setFormat(ReportFormatEnum::HTML);
-        $this->setContentDisposition(ContentDispositionEnum::INLINE);
+        $this->format = ReportFormatEnum::HTML;
     }
 
     /**
@@ -199,21 +192,14 @@ class JasperServerRequestService implements JasperServerRequestServiceContract
         return $this;
     }
 
-    /**
-     * @return string Binary data of report
-     */
-    public function getReport(): string
-    {
-        return $this->report;
-    }
-
     /**
      * @param string $uri
-     * @return $this
+     * @param string|null $pdfFilename
+     * @return string
      */
-    public function call(string $uri): self
+    public function call(string $uri, ?string $pdfFilename = null): string
     {
-        $this->report = $this->client->reportService()->runReport(
+        $report = $this->client->reportService()->runReport(
             $uri,
             $this->format,
             $this->pages,
@@ -226,61 +212,30 @@ class JasperServerRequestService implements JasperServerRequestServiceContract
             $this->transformerKey
         );
 
-        return $this;
-    }
-
-    /**
-     * @return void
-     */
-    public function inline(): void
-    {
-        $this->setContentDisposition(ContentDispositionEnum::INLINE);
-        $this->setHeader($this->report);
-        echo $this->getReport();
-    }
-
-    /**
-     * @param string|null $filename
-     * @return void
-     */
-    public function download(?string $filename = null): void
-    {
-        if (in_array($this->format, ReportFormatEnum::downloadable())) {
-            $this->setContentDisposition(ContentDispositionEnum::ATTACHMENT);
-            $this->setHeader($this->report, $filename);
+        if ($this->format === 'pdf') {
+            $this->setPdfHeader($report, $pdfFilename);
         }
 
-        echo $this->getReport();
+        return $report;
     }
 
     /**
      * @param string $report
-     * @param string|null $filename
+     * @param string|null $pdfFilename
      * @return void
      */
-    private function setHeader(string $report, ?string $filename = null)
+    private function setPdfHeader(string $report, ?string $pdfFilename = null)
     {
-        if (!$filename) {
-            $filename = uniqid();
+        if (!$pdfFilename) {
+            $pdfFilename = uniqid() . '.pdf';
         }
 
-        $filename = $this->appendFormatExtension($filename);
-
         header('Cache-Control: must-revalidate');
         header('Pragma: public');
         header('Content-Description: File Transfer');
-        header("Content-Disposition: $this->contentDisposition;filename=$filename");
+        header("Content-Disposition: $this->contentDisposition;filename=$pdfFilename");
         header('Content-Transfer-Encoding: binary');
         header('Content-Length: ' . strlen($report));
-        header('Content-Type: ' . ReportFormatEnum::getMimeType($this->format));
+        header('Content-Type: application/pdf');
     }
-
-    /**
-     * @param string $filename
-     * @return string
-     */
-        private function appendFormatExtension(string $filename): string
-        {
-            return $filename . '.' . $this->format;
-        }
 }