[PATCH 5/8] header-zone: refactor hook_config_destroy

[Jonatan Schlag <jonatan.schlag@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 1087 bytes --]

With the new id function this function gets a zone name and a id,
but hook_config_cmd needs also the name of the hook.

So this function now calls zone_config_get_hook_from_id to get the hook
and calls then hook_config_cmd with the correct argument order.

Signed-off-by: Jonatan Schlag <jonatan.schlag(a)ipfire.org>
---
 src/header-zone | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/header-zone b/src/header-zone
index c445d55..fdd6e51 100644
--- a/src/header-zone
+++ b/src/header-zone
@@ -206,7 +206,22 @@ hook_config_new() {
 }
 
 hook_config_destroy() {
-	hook_config_cmd "destroy" "$@"
+	assert [ $# -eq 2 ]
+	local zone=${1}
+	# The id must be the id and not the hid.
+	local id=${2}
+
+	shift 2
+
+	# Check if we get a valid id
+	if ! zone_config_id_is_valid ${zone} ${id}; then
+		log ERROR "ID: ${id} is not a valid id for zone ${zone}"
+	fi
+
+	local hook=$(zone_config_get_hook_from_id ${zone}  ${id})
+	assert isset hook
+
+	hook_config_cmd "destroy" "${zone}" "${hook}" "${hook}.${id}" "$@"
 }
 
 hook_config_edit() {
-- 
2.6.3