Skip to content

Commit

Permalink
remove old firewall zone value
Browse files Browse the repository at this point in the history
  • Loading branch information
jreidinger committed Sep 3, 2019
1 parent 174d81c commit 68f45e9
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
2 changes: 0 additions & 2 deletions src/lib/y2network/interface_config_builder.rb
Original file line number Diff line number Diff line change
Expand Up @@ -121,8 +121,6 @@ def save
# create new instance as name can change
firewall_interface = Y2Firewall::Firewalld::Interface.new(name)
if Y2Firewall::Firewalld.instance.installed?
# TODO: New backend?
Yast::LanItems.firewall_zone = firewall_zone
# TODO: should change only if different, but maybe firewall_interface responsibility?
firewall_interface.zone = firewall_zone if !firewall_interface.zone || firewall_zone != firewall_interface.zone.name
end
Expand Down
3 changes: 0 additions & 3 deletions src/modules/LanItems.rb
Original file line number Diff line number Diff line change
Expand Up @@ -49,8 +49,6 @@ module Yast

# FIXME: well this class really is not nice
class LanItemsClass < Module
attr_accessor :firewall_zone

include Logger
include Wicked

Expand Down Expand Up @@ -121,7 +119,6 @@ def main
@wl_key = []
@wl_default_key = 0
@wl_nick = ""
@firewall_zone = nil

# FIXME: We should unify bridge_ports and bond_slaves variables

Expand Down

0 comments on commit 68f45e9

Please sign in to comment.