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

[IMP] Ricarica del valore di un cespite per restituzione #4263

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

SirAionTech
Copy link
Contributor

Implementa #4262 per 16.0.

Ho preso in prestito il refactoring dei test già proposto in #4166.

@SirAionTech SirAionTech force-pushed the 16.0-imp-l10n_it_asset_management-partial_recharge branch from 672f0fa to be3d2c3 Compare July 11, 2024 08:57
@SirAionTech SirAionTech marked this pull request as ready for review July 11, 2024 15:46
@francesco-ooops francesco-ooops linked an issue Jul 12, 2024 that may be closed by this pull request
1 task
@SirAionTech SirAionTech marked this pull request as draft July 17, 2024 08:34
@SirAionTech SirAionTech force-pushed the 16.0-imp-l10n_it_asset_management-partial_recharge branch from be3d2c3 to ba241f3 Compare July 17, 2024 09:20
@SirAionTech
Copy link
Contributor Author

SirAionTech commented Jul 17, 2024

Fatto rebase per includere #4261

@SirAionTech SirAionTech force-pushed the 16.0-imp-l10n_it_asset_management-partial_recharge branch from ba241f3 to f5a3577 Compare July 17, 2024 10:09
@SirAionTech SirAionTech marked this pull request as ready for review July 17, 2024 11:11
@SirAionTech SirAionTech force-pushed the 16.0-imp-l10n_it_asset_management-partial_recharge branch from 1bc3c8b to 8c3669b Compare September 17, 2024 13:49
@SirAionTech SirAionTech marked this pull request as draft September 17, 2024 13:49
@SirAionTech SirAionTech force-pushed the 16.0-imp-l10n_it_asset_management-partial_recharge branch from 8c3669b to c4b229d Compare September 17, 2024 14:37
@SirAionTech SirAionTech marked this pull request as ready for review September 20, 2024 09:01
@SirAionTech SirAionTech force-pushed the 16.0-imp-l10n_it_asset_management-partial_recharge branch from c4b229d to 0751cd8 Compare October 9, 2024 12:49
@SirAionTech
Copy link
Contributor Author

Fatto rebase per risolvere i conflitti

Copy link

@MaurizioConte MaurizioConte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dai test fatti non funziona cercando di collegare la fattura di vendita al cespite e dato che l'importo è superiore al valore del cespite non fa la rettifica negativa l'unica soluzione è di poter modificare l'importo prima di collegarlo al cespite.

@SirAionTech
Copy link
Contributor Author

Dai test fatti non funziona cercando di collegare la fattura di vendita al cespite e dato che l'importo è superiore al valore del cespite non fa la rettifica negativa l'unica soluzione è di poter modificare l'importo prima di collegarlo al cespite.

Grazie di aver provato, vedo in runboat i dati con cui hai provato: la fattura di acquisto image, il cespite image e la fattura di vendita image.

Quindi stai cercando di fare la dismissione parziale dalla fattura di vendita giusto? Qual è l'importo che non ti sta accettando in image?

@MaurizioConte
Copy link

MaurizioConte commented Oct 11, 2024 via email

@MaurizioConte
Copy link

MaurizioConte commented Oct 11, 2024

Dai test fatti non funziona cercando di collegare la fattura di vendita al cespite e dato che l'importo è superiore al valore del cespite non fa la rettifica negativa l'unica soluzione è di poter modificare l'importo prima di collegarlo al cespite.

Grazie di aver provato, vedo in runboat i dati con cui hai provato: la fattura di acquisto image, il cespite image e la fattura di vendita image.

Quindi stai cercando di fare la dismissione parziale dalla fattura di vendita giusto? Qual è l'importo che non ti sta accettando in image?

Ho fatto Fattura di Acquisto e collegata al cespite
Ho ammortizzato tutto il valore del cespite (100%)
Poi ho fatto vendita e collegata al cespite con dismissione parziale e indicato 1000 in valore di acquisto e 1000 in valore fondo ammortamento e collegato al cespite
Poi ho fatto nota credito vendita a storno totale e collegata al cespite con dismissione parziale e indicato 1000 in valore di acquisto e 1000 in valore fondo ammortamento e collegato al cespite.
Le righe sul cespite che sono state generate non mi convincono per esempio la nota di credito doveva reintegrare valore di acquisto e valore fondo ammortamento e invece li ha sommati ovvero è stata fatta la stessa registrazione e poi ci sono 2 righe q di lpus e l'altra di minus valenza per il tota indicato in fattura/nota ovvero 8000.
Adesso sto cercando di capire come si è comportato

Sul Cespite clicca su civilistico e vedi il dettaglio delle righe generate

image

@SirAionTech
Copy link
Contributor Author

Poi ho fatto nota credito vendita a storno totale e collegata al cespite con dismissione parziale e indicato 1000 in valore di acquisto e 1000 in valore fondo ammortamento e collegato al cespite.

Intendi ricarica parziale vero? O hai fatto due volte la dismissinoe?

Le righe sul cespite che sono state generate non mi convincono per esempio la nota di credito doveva reintegrare valore di acquisto e valore fondo ammortamento e invece li ha sommati ovvero è stata fatta la stessa registrazione e poi ci sono 2 righe q di lpus e l'altra di minus valenza per il tota indicato in fattura/nota ovvero 8000.

Magari hai fatto davvero due volte la dismissione, come hai scritto sopra

Adesso sto cercando di capire come si è comportato

Ok, fammi sapere

Copy link

@MaurizioConte MaurizioConte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sarebbe il caso che qualcun altro si faccia un giro perché magari io sbaglio qualche cosa di modo che poi ci possiamo confrontare e chiaro che c'è comunque bisogno di un funzionale che conosca i cespiti.

Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test funzionale: OK

@SirAionTech SirAionTech force-pushed the 16.0-imp-l10n_it_asset_management-partial_recharge branch from 0751cd8 to 8a42bfd Compare October 18, 2024 14:11
@SirAionTech
Copy link
Contributor Author

Fatto rebase per risolvere i conflitti

@SirAionTech SirAionTech force-pushed the 16.0-imp-l10n_it_asset_management-partial_recharge branch from 8a42bfd to 9c2d4ce Compare December 13, 2024 16:35
@SirAionTech
Copy link
Contributor Author

Fatto rebase per risolvere i conflitti.

@SirAionTech
Copy link
Contributor Author

Sarebbe il caso che qualcun altro si faccia un giro perché magari io sbaglio qualche cosa di modo che poi ci possiamo confrontare e chiaro che c'è comunque bisogno di un funzionale che conosca i cespiti.

Magari parlane con @MaurizioPellegrinet visto che per lui è ok

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

Successfully merging this pull request may close these issues.

Ricarica del valore di un cespite per restituzione
3 participants