search

nfe / client-php

Official NFe.io API Client for PHP
https://nfe.io
MIT License
40 stars 18 forks source link

Conflito da Classe NFe #11

Closed renatonascalves closed 6 years ago

renatonascalves commented 7 years ago

@gblmarquez A livraria tem um problema que acho que poderia ser alterado, é relacionado ao #9.

Tenho um cliente que irá usar o plugin WooCommerce NFe, que usa a livraria client-php, para emissão de notas de serviços, e também irá usar o plugin Nota Fiscal Eletrônica WooCommerce para emissão de nota de produtos.

O problema é que o segundo plugin usa uma classe chamada "NFe", com a mesma nomenclatura do seu plugin, que também se chama "NFe". Essa coincidência dá conflito, não permitindo usar os dois plugins ao mesmo tempo.

As opções que sugiro são:

  1. Alterar a classe de NFe para outra diferente, única (fix temporário);
  2. Melhorar o plugin adicionando suporte ao PSR, relacionado ao #9.

No momento, para esse projeto estou usando a opção 1. Esse issue é apenas para chamar a atenção a esse problema. Pois outras livrarias podem usar o mesmo nome dessa classe e ter o mesmo problema, se eles não usarem namespace, claro.

renatonascalves commented 7 years ago

Adicionei em outra branch, https://github.com/nfe/client-php/tree/new-class-name, o fix que mencionei. Ele é temporário, apenas para resolver meu problema.