Bug : Coût Locataire

[bv74]

Bonjour,
dans les charges locatives, le coût par locataire n'est pas renseigné. Aucune valeur n'est affichée. Seul le texte est présent.
Cdlt.
BVA

[mikygee]

Bonjour,
J'ai l'impression qu'il manque dans le code une requête du type
> SELECT fk_renter AS id_locataire, ROUND(SUM(case when MONTH(lo.date_echeance)=1 then lo.chargesamount else 0 end),2) AS month_1, ROUND(SUM(case when MONTH(lo.date_echeance)=2 then lo.chargesamount else 0 end),2) AS month_2, ROUND(SUM(case when MONTH(lo.date_echeance)=3 then lo.chargesamount else 0 end),2) AS month_3, ROUND(SUM(case when MONTH(lo.date_echeance)=4 then lo.chargesamount else 0 end),2) AS month_4, ROUND(SUM(case when MONTH(lo.date_echeance)=5 then lo.chargesamount else 0 end),2) AS month_5, ROUND(SUM(case when MONTH(lo.date_echeance)=6 then lo.chargesamount else 0 end),2) AS month_6, ROUND(SUM(case when MONTH(lo.date_echeance)=7 then lo.chargesamount else 0 end),2) AS month_7, ROUND(SUM(case when MONTH(lo.date_echeance)=8 then lo.chargesamount else 0 end),2) AS month_8, ROUND(SUM(case when MONTH(lo.date_echeance)=9 then lo.chargesamount else 0 end),2) AS month_9, ROUND(SUM(case when MONTH(lo.date_echeance)=10 then lo.chargesamount else 0 end),2) AS month_10, ROUND(SUM(case when MONTH(lo.date_echeance)=11 then lo.chargesamount else 0 end),2) AS month_11, ROUND(SUM(case when MONTH(lo.date_echeance)=12 then lo.chargesamount else 0 end),2) AS month_12 FROM llx_ultimateimmo_immoreceipt as lo , llx_ultimateimmo_immoproperty as ll , llx_ultimateimmo_building as ii WHERE lo.date_echeance >= '2022-01-01 00:00:00' AND lo.date_echeance <= '2022-12-31 23:59:59' AND lo.fk_property = ll.rowid AND ll.fk_property = ii.fk_property GROUP BY fk_renter;

[mikygee]

Dans cost/cost_renter.php ligne 156

Peux-tu remplacer toute la section sql par cela et me dire si c'est le résultat que tu attends

$sql = "SELECT CONCAT(irer.firstname, ' ', irer.lastname) AS nom_locataire";
foreach( $months_list as $month_num => $month_name )
{
        $sql .= ', ROUND(SUM(case when MONTH(ic.date_start)='.$month_num.' then ic.amount else 0 end),2) AS month_'.$month_num;
}
$sql .= " FROM " . MAIN_DB_PREFIX . "ultimateimmo_immocost as ic";
$sql .= " , " . MAIN_DB_PREFIX . "ultimateimmo_immoproperty as ll";
$sql .= " , " . MAIN_DB_PREFIX . "ultimateimmo_immorent as ir";
$sql .= " , " . MAIN_DB_PREFIX . "ultimateimmo_immorenter as irer";
$sql .= " WHERE ic.date_start >= '" . $db->idate ( dol_get_first_day ( $y, 1, false ) ) . "'";
$sql .= "  AND ic.date_start <= '" . $db->idate ( dol_get_last_day ( $y, 12, false ) ) . "'";
$sql .= "  AND ic.fk_property = ll.rowid ";
$sql .= "  AND ir.fk_property = ll.rowid ";
$sql .= "  AND ir.fk_renter = irer.rowid ";
$sql .= " GROUP BY irer.lastname";

[bv74]

Bonjour,

ce que j'ai dans ce fichier ressemble très fortement aux lignes de code du commit 724189c

$sql = "SELECT ii.label AS nom_immeuble";
foreach( $months_list as $month_num => $month_name )

[mikygee]

Bonjour,

Je ne comprends pas trop, car il semble que c'est moi qui fût à l'origine de ce commit.
Ce commit porte principalement sur des éléments de traduction.

La requête que je propose de modifier n'a rien à voir avec ce commit. L'as-tu fait ? As-tu vu si le résultat te convenait ?

[bv74]

Bonjour @mikygee,

1. j'ai réinstallé la dernière version connue de ultimateImmo.

2. j'ai remplacé toute les section à partir de la ligne 156.

3. Je n'ai que les charges réelles du locataire
   

4. Le résultat est meilleur. Je vérifie que les valeurs sont OK.

5. conséquence du point 3, je n'ai pas le différentiel.

6. Au passage corriger charges réel locataire par charges réelles locataires

[mikygee]

Je n'ai pas exactement compris ton dernier retour.

• Est-ce que tu me dis dans ton derniers message que la section Charges réelles locataire est maintenant correcte ?
  Moi j'ai ceci
  
  Toujours sur le même point, est-ce qu'en colonne 1 tu penses qu'il faut afficher le nom/prénom du locataire comme je l'ai fait ou le nom de l'appartement.
  Je ne suis pas sûr de savoir ce que Darkjeff souhaitait afficher à l'origine ou ergonomiquement si un utilisateur préfèrerait voir un nom de locataire ou un appartement. A mon avis, on préfèrerait voir le nom de l'appartement, surtout si le locataire change en cours d'année.

• Et tu m'indiques que la section différentiel charge n'est pas correcte, c'est cela ?

[Darkjeff]

Il faut le nom du locataire car les charges sont pour le locataire
mais les charges sont sur l'immeuble
c'est une réflexion intéressantes

et il y a encore quelques corrections a faire sur cette page quand on est en multi propriétaire

[bv74]

Bonjour,

@mikygee
Est-ce plus clair avec la copie écran ? J'ai mis en rouge les sections OK ou KO. Je suis prudent en mettant a priori, il faut que je vérifie s'il y a bien des données et si les sommes sont justes.

@Darkjeff : Oui je pense que par locataire c'est préférable.
cdlt.

[mikygee]

Oui c'est plus clair.
Je pense qu'il faudrait que tu vérifies les données, car mon jeu de tests n'est peut être pas complet.

@Darkjeff J'ai du mal à comprendre comment tu avais un résultat dans Charges réelles car ni moi ni @bv74 n'avions de données.
On voit bien d'après ta capture écran que tous les calculs sont corrects.

[bv74]

@mikygee
Oui bien entendu OK pour compléter le jeu de données, mais probablement pas avant fin septembre.

[bv74]

Bonjour,
avec la version 96dfc20, j'ai remis la requête SQL à la ligne 156 dans le fichier cost/cost_renter.php. Je retrouve ce que j'ai décrit ci-dessus.
Les charges réelles par locataire sont justes.