-
Notifications
You must be signed in to change notification settings - Fork 7
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
8.0.5.1.2 l10n id taxform bukti potong common #138
Conversation
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 ada review yah
l10n_id_taxform_bukti_potong_pph_common/models/account_move_line.py
Outdated
Show resolved
Hide resolved
|
||
@api.multi | ||
def _check_taxform(self, move_line): | ||
self._cr.execute( |
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.
tambahkan lg self.ensure_one() jika menggunakan api.multi
"""SELECT 1 FROM rel_bukpot_line_2_income_move | ||
WHERE account_move_id=%s | ||
""", | ||
(move_line.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.
self.id jika menggunakan api.multi
@api.multi | ||
def unlink(self): | ||
for move_line in self: | ||
taxform = self._check_taxform(move_line) |
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.
self._check_taxform() jika _check_taxform menggunakan api.multi
8c654f1
to
d62773f
Compare
Codecov Report
@@ Coverage Diff @@
## 8.0 #138 +/- ##
==========================================
- Coverage 60.17% 60.05% -0.13%
==========================================
Files 115 116 +1
Lines 2968 2984 +16
==========================================
+ Hits 1786 1792 +6
- Misses 1182 1192 +10
Continue to review full report at Codecov.
|
@api.multi | ||
def unlink(self): | ||
for move_line in self: | ||
taxform = self._check_taxform() |
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 berarti move_line._check_taxform()
d62773f
to
39b967f
Compare
No description provided.