-
Notifications
You must be signed in to change notification settings - Fork 12
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
Adding new module hr_payslip_compute_sheet #9
Adding new module hr_payslip_compute_sheet #9
Conversation
Issues : #6 |
1 similar comment
Current coverage is 80.66% (diff: 26.19%)@@ 8.0 #9 diff @@
==========================================
Files 7 8 +1
Lines 139 181 +42
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
+ Hits 135 146 +11
- Misses 4 35 +31
Partials 0 0
|
"name": "HR Payslip Compute Sheet", | ||
"version": "8.0.1.0.0", | ||
"summary": "Adds wizard to compute sheet", | ||
"category": "Base", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Human Resources
"data": [ | ||
"wizards/hr_compute_sheet_views.xml", | ||
"views/hr_payslip_views.xml", | ||
"data/ir_actions_server_data.xml" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mikevhe18 ini dinamain ir_values_data.xml aja mike
@@ -0,0 +1,29 @@ | |||
<?xml version="1.0" encoding="utf-8"?> | |||
<!-- Copyright <YEAR(S)> <AUTHOR(S)> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dibetulin
<openerp> | ||
<data> | ||
|
||
<record id="hr_compute_sheet_act_window" model="ir.actions.act_window"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Kan udah ada hr_compute_sheet_action di wizardnya mike. Pake yg udah ada aja, ga usah dibuat 2x
<button | ||
name="%(hr_compute_sheet_action)d" | ||
type="action" | ||
string="Compute Sheet" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
State nya samain dengan tombol aslinya mike
self.ensure_one() | ||
obj_hr_payslip = self.env['hr.payslip'] | ||
context = self._context | ||
record_id = context['active_ids'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
record_ids . jamak
context = self._context | ||
record_id = context['active_ids'] | ||
|
||
for data in record_id: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for payslip in self.browse(active_ids)
record_id = context['active_ids'] | ||
|
||
for data in record_id: | ||
payslip = obj_hr_payslip.browse(data) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Buang
payslip = obj_hr_payslip.browse(data) | ||
if payslip.state == 'draft': | ||
if self.reload_input: | ||
contract_ids = payslip.get_contract( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Baris 34 sampai 51 dijadikan method sendiri
} | ||
payslip.write(vals) | ||
if self.reload_workdays: | ||
contract_ids = payslip.get_contract( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Baris 53 - 71 dijadikan method sendiri
f3a5aff
to
fd1d075
Compare
No description provided.