Code review comment for lp:~synerpgy/openobject-addons/hr_payroll_french_localization

Revision history for this message
YannickB (YOLO consulting) (yannick-buron) wrote :

Well I'm afraid the work on the report take more time than planned... My skills is more on the functional part and I can't achieve to make the report working.
If you want to, maybe a little help could be useful. I already made a sxw (you can find it on l10n_fr_hr_payroll/report/fiche_paye_save.sxw), we just have to make it working.

If I have no other choice, I could certainly ask to my old developer to help me but not before the weekend.

As I can't anything else, I'll return working on the contributions declarations where I am way more efficient.

I also add a "type" field on hr.payroll.rule, which should work as your "Report Section"suggested. It can have as value : "normal", "employer" and "total"
     -A "normal" rule will work just like now. If it has a child rule with "employer" type, this child rule will be use for the employer part on the line of the report .
     -A "employer" rule, without being a child of an another rule, will only display his rate and amount field on the employer part on the report. The employee part will be leaved empty in that case.
     -A "Total" rule will be use in case we need to display a total line on the report. This kind of line should not have any impact on the total computed for the category, not like now. This is important as if I currently use total line the total computed for the category are all wrong.

« Back to merge proposal