fix #894 by actually updating with the modified settings

This commit is contained in:
Daniel Nägele 2015-09-08 15:28:05 +02:00
parent a55dbb717c
commit 39eb3807f3

View file

@ -76,7 +76,7 @@ options:
notes: notes:
- Not tested on any Debian based system. - Not tested on any Debian based system.
requirements: [ 'firewalld >= 0.2.11' ] requirements: [ 'firewalld >= 0.2.11' ]
author: "Adam Miller (@maxamillion)" author: "Adam Miller (@maxamillion)"
''' '''
EXAMPLES = ''' EXAMPLES = '''
@ -138,7 +138,7 @@ def set_port_disabled_permanent(zone, port, protocol):
#################### ####################
# source handling # source handling
# #
def get_source(zone, source): def get_source(zone, source):
fw_zone = fw.config().getZoneByName(zone) fw_zone = fw.config().getZoneByName(zone)
fw_settings = fw_zone.getSettings() fw_settings = fw_zone.getSettings()
@ -151,11 +151,13 @@ def add_source(zone, source):
fw_zone = fw.config().getZoneByName(zone) fw_zone = fw.config().getZoneByName(zone)
fw_settings = fw_zone.getSettings() fw_settings = fw_zone.getSettings()
fw_settings.addSource(source) fw_settings.addSource(source)
fw_zone.update(fw_settings)
def remove_source(zone, source): def remove_source(zone, source):
fw_zone = fw.config().getZoneByName(zone) fw_zone = fw.config().getZoneByName(zone)
fw_settings = fw_zone.getSettings() fw_settings = fw_zone.getSettings()
fw_settings.removeSource(source) fw_settings.removeSource(source)
fw_zone.update(fw_settings)
#################### ####################
# service handling # service handling
@ -191,7 +193,7 @@ def set_service_disabled_permanent(zone, service):
fw_settings = fw_zone.getSettings() fw_settings = fw_zone.getSettings()
fw_settings.removeService(service) fw_settings.removeService(service)
fw_zone.update(fw_settings) fw_zone.update(fw_settings)
#################### ####################
# rich rule handling # rich rule handling