Pianificazione fatturazione fa un doppio sconto

the_jolly01

Buongiorno, ho notato che andando a fare una pianificazione fatturazione su un contratto che ha già presente uno sconto quando vado a creare le rate ricalcola lo sconto applicandolo al prezzo già scontato. Mi spiego meglio con le immagini:

Qui vado ad aggiungere una nuova riga con prezzo unitario 2700, sconto di 1120 euro e 36 di quantità

Una volta creata la riga la vedo in questo modo corretto, facendo i calcoli [(2700-1120) + 22%] x 36 = 69393.6 euro ed è corretto.

Una volta generata invece la fatturazione ricorrente mi genera queste righe andando a mettere come prezzo unitario il prezzo 1580 che sarebbe il prezzo unitario meno lo sconto, quindi 2700-1120=1580. Quindi mi restituisce un importo di 460 che sarebbe il prezzo scontato meno lo sconto: 2700-1120 = 1580 e 1580-1120 = 460. Poi me lo riporta giustamente per 36 mesi e anzichè restituirmi 56880 come totale imponibile me lo restitusice come imponibile, cioè 158036 su cui poi ritoglie lo sconto che è di 112036 = 40320 portandomi il contratto dai 69393.6 a 20203.2, quindi un errore non da poco.

Per caso qualcuno sa dirmi se è stato già risolto il problema ufficialmente per la versione 2.4.48 di OpenSTAManager? Oppure sapreste dirmi se eventualmente sono io che sto sbagliando procedimento?

Purtroppo non posso aggiornare a versioni nuove in quanto utilizzo molto codice modificato e sarebbe un problema dover adattare tutto

P.S. Tutti i test sono stati eseguiti su una versione vanilla di OpenSTAManager, i problemi non sono dovuti quindi a modifiche di codice da parte mia

Valentina

Buongiorno the_jolly01,
può seguire gli aggiornamenti dell'issue da qui: devcode-it/openstamanager#1443
al momento non è ancora stato risolto, perchè è in programma la modifica dell'intera struttura del plugin.

the_jolly01

Ho risolto cambiando il codice dell'add nell'actions di pianificazione_fatturazione lo condivido qua ma devo ancora testarlo completamente:

`case 'add':
$contratto = Contratto::find($id_record);

    if (post('scadenza') == 'Mensile') {
        $timeing = '+1 month';
    }
    if (post('scadenza') == 'Bimestrale') {
        $timeing = '+2 month';
    }
    if (post('scadenza') == 'Trimestrale') {
        $timeing = '+3 month';
    }
    if (post('scadenza') == 'Quadrimestrale') {
        $timeing = '+4 month';
    }
    if (post('scadenza') == 'Semestrale') {
        $timeing = '+6 month';
    }
    if (post('scadenza') == 'Annuale') {
        $timeing = '+12 month';
    }

    $selezioni = collect(post('selezione_periodo'));
    $periodi = post('periodo');

    $numero_fatture = 0;
    $date_pianificazioni = [];
    $pianificazioni = [];
    foreach ($selezioni as $key => $selezione) {
        $date = new DateTime($periodi[$key]);

        if (post('cadenza_fatturazione') == 'Inizio') {
            $date->modify('first day of this month');
        } elseif (post('cadenza_fatturazione') == 'Giorno' && !empty(post('giorno_fisso'))) {
            $date->modify('last day of this month');
            $last_day = $date->format('d');
            $day = post('giorno_fisso') > $last_day ? $last_day : post('giorno_fisso');

            // Correzione data
            $date->setDate($date->format('Y'), $date->format('m'), $day);
        }

        // Comversione della data in stringa standard
        $data_scadenza = $date->format('Y-m-d');

        ++$numero_fatture;

        // Creazione pianificazione
        $pianificazione = Pianificazione::build($contratto, $data_scadenza);
        $date_pianificazioni[] = $data_scadenza;
        $pianificazioni[$numero_fatture] = $pianificazione->id;
    }

    if ($numero_fatture > 0) {
        $righe_contratto = $contratto->getRighe();
        $subtotale = [];
        $pianificata = [];
        $non_pianificata = [];

        // Creazione nuove righe
        $qta = post('qta');
        foreach ($righe_contratto as $r) {
            $qta_evasa = $r->qta_evasa;
            $data_scadenza = '';
            $inizio = $date_pianificazioni[0];
            $fine = date('Y-m-d', strtotime($inizio . ' ' . $timeing));
            $fine = date('Y-m-d', strtotime($fine . ' -1 days'));
            if ($cadenza_fatturazione == 'Fine') {
                $fine = Carbon\Carbon::parse($fine)->endOfMonth()->format("Y-m-d");
            }
            for ($rata = 1; $rata <= $numero_fatture; ++$rata) {
                if ($qta_evasa < $r->qta) {
                    $qta_riga = ($qta[$r->id] <= ($r->qta - $qta_evasa) ? $qta[$r->id] : ($r->qta - $qta_evasa));
                    $descrizione = post('descrizione')[$r->id];

                    $descrizione = variables($descrizione, $inizio, $fine, $rata, $numero_fatture)['descrizione'];

                    $inizio = $fine;
                    $inizio = date('Y-m-d', strtotime($inizio . ' +1 days'));

                    $fine = date('Y-m-d', strtotime($inizio . ' ' . $timeing));
                    $fine = date('Y-m-d', strtotime($fine . ' -1 days'));
                    if ($cadenza_fatturazione == 'Fine') {
                        $fine = Carbon\Carbon::parse($fine)->endOfMonth()->format("Y-m-d");
                    }

                    // Calcolo il prezzo unitario senza sottrarre lo sconto
                    $prezzo_imponibile = $r->subtotale / $r->qta;

                    if (!empty($r->idarticolo)) {
                        $articolo = ArticoloOriginale::find($r->idarticolo);
                        $riga = Articolo::build($contratto, $articolo);
                    } else {
                        $riga = Riga::build($contratto);
                    }

                    $riga->descrizione = $descrizione;
                    $riga->setPrezzoUnitario($prezzo_imponibile, $r->idiva);
                    $riga->setSconto(($r->tipo_sconto == 'PRC' ? $r->sconto_percentuale : $r->sconto_unitario), $r->tipo_sconto);
                    $riga->qta = $qta_riga;
                    $riga->idpianificazione = $pianificazioni[$rata];

                    $riga->save();

                    $qta_evasa += $qta_riga;
                    $pianificata[] = $pianificazioni[$rata];
                } else {
                    $non_pianificata[] = $pianificazioni[$rata];
                }
            }
            $r->delete();
        }
        $tot_non_pianificati = implode(', ', array_unique(array_diff($non_pianificata, $pianificata)));
        if (!empty($tot_non_pianificati)) {
            $dbo->query('DELETE FROM `co_fatturazione_contratti` WHERE `id` IN (' . $tot_non_pianificati . ')');
        }
    }

    break;`

Valentina

Buongiorno the_jolly01,
può trovare la fix al seguente link: devcode-it/openstamanager@07eb529