Skip to content

Commit dd0b093

Browse files
committed
[pfsense_interface] Support diff mode
1 parent eba1c3a commit dd0b093

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
minor_changes:
2+
- pfsense_interface - support ``--diff`` (https://github.com/pfsensible/core/pull/228).

plugins/module_utils/interface.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -217,15 +217,13 @@ def _validate_params(self):
217217
def _copy_and_add_target(self):
218218
""" create the XML target_elt """
219219
self.pfsense.copy_dict_to_element(self.obj, self.target_elt)
220+
self.diff['after'] = self.obj
220221
self.setup_interface_cmds += "interface_configure('{0}', true);\n".format(self.target_elt.tag)
221222
self.result['ifname'] = self.target_elt.tag
222223

223224
def _copy_and_update_target(self):
224225
""" update the XML target_elt """
225-
before = self.pfsense.element_to_dict(self.target_elt)
226-
changed = self.pfsense.copy_dict_to_element(self.obj, self.target_elt)
227-
if self._remove_deleted_params():
228-
changed = True
226+
(before, changed) = super(PFSenseInterfaceModule, self)._copy_and_update_target()
229227

230228
if changed:
231229
if self.params['enable']:
@@ -326,6 +324,7 @@ def _get_params_to_remove():
326324

327325
def _pre_remove_target_elt(self):
328326
""" processing before removing elt """
327+
super(PFSenseInterfaceModule, self)._pre_remove_target_elt()
329328
self.obj['if'] = self.target_elt.find('if').text
330329

331330
ifname = self.target_elt.tag

0 commit comments

Comments
 (0)