Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Título do PDF #602

Open
dubbows opened this issue May 20, 2024 · 10 comments
Open

Título do PDF #602

dubbows opened this issue May 20, 2024 · 10 comments

Comments

@dubbows
Copy link

dubbows commented May 20, 2024

Para incluir títulos no PDF é necessário essa implementação:

vendor/nfephp-org/sped-da/src/NFe/Danfe.php

class Danfe extends DaCommon
{
...
protected $title = '';

public function setTitle($title)
{
$this->title = $title;
}

protected function monta()
{
...
$this->pdf->setTitle($this->title);
...
}
...
}

Se puder incluir a implementação seria de grande valia.

@gustavokennedy
Copy link
Contributor

Olá @dubbows!

Abri um PR com a solicitação.

@f3lpz
Copy link
Contributor

f3lpz commented Nov 11, 2024

Boa tarde @gustavokennedy , esse pull request quebrou a impressão, pois na linha 500 esta atribuindo o valor da variavel pdf uma string vazia, isso da o erro Call to a member function setTitle() on string

@gustavokennedy
Copy link
Contributor

gustavokennedy commented Nov 11, 2024

Verdade @f3lpz , vou incluir o setTitle na linha 513, abaixo da instância $this->pdf = new Pdf($this->orientacao, 'mm', $this->papel);.

@gustavokennedy
Copy link
Contributor

Corrigido! @f3lpz @dubbows

@f3lpz
Copy link
Contributor

f3lpz commented Nov 11, 2024

Valeu @gustavokennedy !

@robmachado
Copy link
Contributor

@gustavokennedy sempre faça um teste local antes de subir um PR, como erá uma coisa muito simples nem me atentei ao local onde estava a implementação e autorizei (falha minha por não conferir)

@gustavokennedy
Copy link
Contributor

@robmachado Pode deixar! Foi tão simples que deixei passar também! Nas próximas farei, com certeza. 👏👏

@gustavokennedy
Copy link
Contributor

@robmachado Lembrei agora também! Não teriamos que alterar nada nas Documentações?

@robmachado
Copy link
Contributor

Poes é a documentação está "paupérrima" falta tudo kkkk vide https://github.com/nfephp-org/sped-da/blob/master/docs/Danfe.md

@robmachado
Copy link
Contributor

Eu estou fazendo uma refatoração da Danfe, por hora estou chamando de Danfe2, estou movendo as (ou a maioria das) estruturas para traits assim fica mais facil manter o codigo em blocos menores

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants