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

incluir lançamento com saldo anterior ao aplicar filtro por data #4

Open
rodrigopc opened this issue Oct 31, 2015 · 5 comments
Open

Comments

@rodrigopc
Copy link

Ao filtrar os lançamentos por data, o saldo acumulado não reflete a situação real, pois os valores anteriores ao mês inicial do filtro são desconsiderados. Exemplo:

06/2015 saldo inicial -100
07/2015 salário 1000
08/2015 aluguel -500

Ao filtrarmos os lançamentos entre 07/2015 e 08/2015 o saldo acumulado será 500, desconsiderando o saldo inicial negativo. Esta é, inclusive, uma das idéias cadastradas no UserVoice.

Para resolver este problema criei um PR que inclui uma opção para exibição do saldo anterior que, quando selecionada, cria um lançamento chamado "saldo anterior" com o valor do saldo acumulado anterior ao período filtrado.

@edimar
Copy link

edimar commented Nov 9, 2015

Opa, Rodrigo, tudo bem? Ótima ideia, estou passando exatamente pelo mesmo problema.

Só uma pergunta, quando vc diz que criou uma "PR", o que vem a ser está PR?

Abs.

@rodrigopc
Copy link
Author

PR = Pull Request; já implementei esta funcionalidade em um fork do moneylog e solicitei que fosse incluída no moneylog original.
Criei mais uma opção de filtro (Saldo Anterior) abaixo do filtro de meses, assim você pode ter tanto o comportamento novo (calcular o saldo anterior ao mês inicial) quanto o atual.

@edimar
Copy link

edimar commented Nov 9, 2015

Beleza, Obrigado;

Agora depende do Aurélio liberar a versão então?

E só por curiosidade como resolveu? Teria problemas em me enviar?

Eu estou quase resolvendo, basicamente somei o valor dentro do filterData
antes dele fazer aquele continue da comparação de data...
(Agora só estaria faltando incluir a linha com o saldo
anterior...teoricamente hehe).

Obrigado.
Em 09/11/2015 17:48, "Rodrigo Cunha" notifications@github.com escreveu:

PR = Pull Request; já implementei esta funcionalidade em um fork do
moneylog e solicitei que fosse incluída no moneylog original.


Reply to this email directly or view it on GitHub
#4 (comment)
.

@rodrigopc
Copy link
Author

Você pode conferir as alterações aqui, ou baixar o arquivo já gerado aqui

@edimar
Copy link

edimar commented Nov 27, 2015

Parabéns Rodrigo, Show de bola a solução mesmo!!

Era a mesma Lógica que eu estava seguindo, Já estava conseguindo incluir a linha de lançamento do saldo anterior, mas faltava ainda colocar um checkbox para marcar/desmarcar. Enfim você resolveu tudo de forma precisa e completa.

Obrigado.

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

2 participants