profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/rvalyi/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Raphaël Valyi rvalyi Akretion Rio de Janeiro; Brazil http://www.akretion.com.br Akretion CEO

rvalyi/jripple 12

JRuby scripting step plugin for Kettle (Pentaho Data Integration): JRuby Rips Kettle = JRipple

akretion/sped-extractor 5

SPED (Sistema Público de Escrituração Digital) no Odoo ERP (ECD, ECF, EFD ICMS IPI, EFD PIS COFINS)

rvalyi/click-odoo 1

Run python scripts in a Odoo environment

rvalyi/geraldo 1

Geraldo is a reports engine for Python and Django applications. It uses the power of ReportLab and PIL to generate reports with page header and footer, child bands, report begin and summary bands, agreggation and graphic elements, etc.

rvalyi/google-spreadsheet-ruby 1

Ruby library to read/write Google Spreadsheet

rvalyi/jooor 1

Java RPC acceleration layer for OOOR

rvalyi/activeresource 0

Connects business objects and REST web services

rvalyi/anybox.recipe.odoo 0

Buildout recipe for Odoo 8, special branches for anybox.recipe.openerp handle Odoo 8 + OpenERP >= 5.0

rvalyi/buildstep 0

Buildstep uses Docker and Buildpacks to build applications like Heroku

issue commentOCA/l10n-brazil

[12][BUG] Problema com arredondamento do ISSQN

Tem esse tipo de recurso no Odoo no float_utils passando o método rounding_method='UP' https://github.com/odoo/odoo/blob/8f6430359b347d3669a536eca0a297c66ce99163/odoo/tools/float_utils.py#L37 agora eu não sei se o problema seria esse mesmo. Sendo que o Odoo usa floats e não decimals é fácil rolar erros de aredondamento. Até onde eu sei é sempre possível de resolver mas as vezes obriga a fazer coisas com rigor de cálculo científico com epsilons onde o cálculo seria trivial se fosse usar decimals. Mas enfim essa escolha pelos floats tá muito no sangue do Odoo e temos que assumir essa escolha TB e ter esse rigor nos cálculos então.

marcelsavegnago

comment created time in 2 days

pull request commentOCA/l10n-brazil

[12.0] Chave do documento fiscal sem prefixo

a questão do erpbrasil.base foi resolvida mas tem erros nos testes Travis...

renatonlima

comment created time in 2 days

issue commentOCA/l10n-brazil

[Q&A] Informações de Pagamento na Nota Fiscal Eletrônica

Eu não olhei bem, mas bem capaz que esteja faltando algo mesmo. De uma forma geral tem bastante melhoria a ser feito para mapear os pagamentos. O @mileo tinha embarcado numa coisa gigantesca https://github.com/OCA/l10n-brazil/pull/820 que foi revertida depois. Algumas coisa boa tinham, mas o problema mesmo era que o custo/beneficio disso tudo não batia. Basicamente, a gente assume no l10n_br_fiscal que se o cara quer usar pagamentos, ele vai instalar o modulo account. Então a gente quer usar a parte de pagamentos do account e não copiar ela dentro do modulo l10n_br_fiscal que ja ta obeso.

Sem falar que copiar código sem teste assim, cria problemas na migração, cria uma dependência artificial entre o fiscal e o account onde nao tinha (ja que se a Odoo SA mudasse algo pros pagamentos teriamos que adaptar no fiscal tb) e tb cria problema de licença porque o codigo da Odoo SA nao assinou o Contributor Licence Agreement da OCA e ela não tem a menor intençao de assinar. Dai tem que ser algo simples, umas 200 linhas no maximo que não atrapalha nas migrações. Sem alguem quiser usar pagamentos sem o modulo account, a gente ta super aberto para botar uns hooks para a pessoa puder plugar um modulo opcional para isso, porem ja que nem 1% de quem usa vai funcionar nesse cenario, não da para ser o padrão para geral. Pois é assim que se cria monstros impossível de manter depois.

cc @renatonlima

netosjb

comment created time in 3 days

issue commentOCA/l10n-brazil

[Q&A] Forma de pagamento - Leiaute NFe

Ola @netosjb eu nao parei para olhar bem esse seu issue, mas houve sim um select de pagamento que virou mesmo um campo txt nos ultimos esquemas, vé o que eu eu tinha comentado a respeito https://github.com/OCA/l10n-brazil/pull/1553#issue-705769360 "Se ha duvida: sim os tipo de bandeira e de pagamento ficaram mesmo com texto livre agora em vez da seleção de antes." Eu tb estranhei, mas vc pode abrir o XSD e verificar como eu verifiquei. Seria disso que se trata?

netosjb

comment created time in 3 days

issue commentOCA/l10n-brazil

[12.0][BUG] Problema em atualização de base (account.tax.group)

Então pessoal, a respeito do #1602 tem que atualizar com --update=l10n_br_coa dai ate onde eu testei em prod funciona bem. Normal, o esquema ligado ao modulo mudou, teve atualização minor ou major (major no caso), tem que dar update do modulo.

Agora, ao contrario de que talvez vcs pensaram, o l10n_br_coa NAO DEPENDE do l10n_br_base, portanto, atualizar o l10n_br_base NAO VAI mandar atualizar o l10n_br_coa (sem falar que nao teria porque atualizar o l10n_br_base no caso). Podem confirmar se ta OK e se podemos fechar o issue?

marcelsavegnago

comment created time in 3 days

pull request commentOCA/l10n-brazil

[12.0] Ao Excluir uma Fatura o documento fiscal deve ser excluido

(no caso das linhas de documento fiscal o problema ja tinha sido solucionado https://github.com/OCA/l10n-brazil/blob/12.0/l10n_br_account/models/account_invoice_line.py#L235 )

renatonlima

comment created time in 4 days

PullRequestReviewEvent
PullRequestReviewEvent

pull request commentOCA/l10n-brazil

[12.0][FIX] l10n_br_nfse: federal withholdings

/ocabot merge minor

marcelsavegnago

comment created time in 4 days

PullRequestReviewEvent

pull request commentOCA/l10n-brazil

[12.0][ADD] l10n_br_nfse: Law indication in DANFSe

/ocabot merge minor

marcelsavegnago

comment created time in 4 days

pull request commentOCA/l10n-brazil

[12.0][FIX] l10n_br_fiscal e l10n_br_nfse: regeração do PDF do documento fiscal

@marcelsavegnago valeu pelo trabalho. Vc poderia por favor dar um rebase para a gente conferir os testes? Pois parece conflitar com meu workaround temporário para resolver a questão da cagada da Odoo SA com os logs e que foi resolvido uns dias atrás.

marcelsavegnago

comment created time in 4 days

pull request commentOCA/l10n-brazil

[12.0][IMP] BR fields in the Sale Report

/ocabot merge nobump

mbcosta

comment created time in 4 days

pull request commentOCA/l10n-brazil

[12.0][IMP] BR fields in the Sale Report

/ocabot merge nobump

mbcosta

comment created time in 4 days

PullRequestReviewEvent
PullRequestReviewEvent
PullRequestReviewEvent

Pull request review commentOCA/l10n-brazil

[12.0] Ao Excluir uma Fatura o documento fiscal deve ser excluido

 def create(self, values):             values["document_date"] = self._date_server_format()         return super().create(values) +    def unlink(self):+        forbidden_states_unlink = [+            SITUACAO_EDOC_AUTORIZADA,+            SITUACAO_EDOC_CANCELADA,+            SITUACAO_EDOC_DENEGADA,+            SITUACAO_EDOC_INUTILIZADA,+        ]++        for record in self.filtered(+            lambda d: d != self.env.user.company_id.fiscal_dummy_id+            and d.edoc_state in forbidden_states_unlink+        ):

OK, pode ser eu nâo tinha visto.

renatonlima

comment created time in 5 days

Pull request review commentOCA/l10n-brazil

[12.0] Ao Excluir uma Fatura o documento fiscal deve ser excluido

 def create(self, values):             values["document_date"] = self._date_server_format()         return super().create(values) +    def unlink(self):+        forbidden_states_unlink = [+            SITUACAO_EDOC_AUTORIZADA,+            SITUACAO_EDOC_CANCELADA,+            SITUACAO_EDOC_DENEGADA,+            SITUACAO_EDOC_INUTILIZADA,+        ]++        for record in self.filtered(+            lambda d: d != self.env.user.company_id.fiscal_dummy_id+            and d.edoc_state in forbidden_states_unlink+        ):

É meio deselegante fazer um for com um raise dentro assim. Que tal escrever algo como if any(record for record in self.filtered(...):

renatonlima

comment created time in 5 days

PullRequestReviewEvent
PullRequestReviewEvent
PullRequestReviewEvent

pull request commentOCA/l10n-brazil

[12.0] Devoluções e Retornos de Materiais

/ocabot merge major

renatonlima

comment created time in 5 days

pull request commentOCA/l10n-brazil

[13.0][FORWARD PORT] ref group tax

/ocabot merge major

rvalyi

comment created time in 5 days

PR closed OCA/l10n-brazil

[14.0][FORWARD PORT] ref group tax

forward por de #1602 via os cherry-pick automatico dos commits na 13.0 https://github.com/OCA/l10n-brazil/pull/1623 Aplicando apenas os 2 commits dos 2 modulos l10n_br_coa e l10n_br_coa_simple ja migrado. O outro commit foi aplicado no PR https://github.com/OCA/l10n-brazil/pull/1589

cc @netosjb

+47 -245

1 comment

8 changed files

rvalyi

pr closed time in 6 days

pull request commentOCA/l10n-brazil

[14.0][FORWARD PORT] ref group tax

fechando porque foi incluido no https://github.com/OCA/l10n-brazil/pull/1589

rvalyi

comment created time in 6 days

pull request commentOCA/l10n-brazil

[14.0] mig l10n_br_coa_generic

/ocabot merge major

rvalyi

comment created time in 6 days

pull request commentOCA/l10n-brazil

[12.0] refactor account tax group

parabens de novo @netosjb pela excelente contribuição!

netosjb

comment created time in 6 days

pull request commentOCA/l10n-brazil

[12.0] refactor account tax group

/ocabot merge major

netosjb

comment created time in 6 days