Therp Addons

Merge lp:~therp-nl/therp-addons/7.0_firstname_display_name_trigger_by_delegation into lp:~therp-nl/therp-addons/7.0

  1. 7.0_firstname_display_name_trigger_by_delegation
  2. Merge into 7.0
Proposed by Holger Brunn (Therp)
Status: Merged
Approved by: Stefan Rijnhart (Opener)
Approved revision: 107
Merged at revision: 108
Proposed branch: lp:~therp-nl/therp-addons/7.0_firstname_display_name_trigger_by_delegation
Merge into: lp:~therp-nl/therp-addons/7.0
Diff against target: 12 lines (+2/-0)
1 file modified
partner_name_dutch/model/res_partner.py (+2/-0)
To merge this branch: bzr merge lp:~therp-nl/therp-addons/7.0_firstname_display_name_trigger_by_delegation
Reviewer Review Type Date Requested Status
Stefan Rijnhart (Opener) (community) Approve
Review via email: mp+227674@code.launchpad.net
To post a comment you must log in.
lp:~therp-nl/therp-addons/7.0_firstname_display_name_trigger_by_delegation updated
107. By Holger Brunn (Therp)

[FIX] return/continue

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'partner_name_dutch/model/res_partner.py'
2--- partner_name_dutch/model/res_partner.py 2014-07-17 08:28:36 +0000
3+++ partner_name_dutch/model/res_partner.py 2014-07-22 07:39:21 +0000
4@@ -45,6 +45,8 @@
5 #if firstname_display_name_trigger is installed, add our keys to
6 #the trigger
7 if hasattr(self, '_display_name_store_triggers'):
8+ if self._name not in self._display_name_store_triggers:
9+ return
10 self._display_name_store_triggers[self._name][1].extend(
11 ['infix', 'initials'])
12 for trigger in self.pool._store_function[self._name]:

Subscribers

People subscribed via source and target branches

to all changes: