From: Matthias Kruk <m@m10k.eu>
Date: Sun, 8 Aug 2021 02:32:16 +0000 (+0900)
Subject: include/msg/merge: Remove log property
X-Git-Url: https://git.corax.cc/?a=commitdiff_plain;h=87d6330b4f3cd17e51ff4e4c46b5b14289d7c0f6;p=foundry

include/msg/merge: Remove log property

The log property was removed from the message definition of merge
messages, but the foundry/msg/merge module provides means to set and
get the value of this property.
This commit removes the log argument from foundry_msg_merge_new() and
the foundry_msg_merge_get_log() method, so that the log property
cannot be accessed anymore.
---

diff --git a/include/msg/merge.sh b/include/msg/merge.sh
index 39b0d8d..3b64d61 100644
--- a/include/msg/merge.sh
+++ b/include/msg/merge.sh
@@ -16,7 +16,6 @@ foundry_msg_merge_new() {
 	local srcbranch="$3"
 	local dstbranch="$4"
 	local status="$5"
-	local log="$6"
 
 	local json
 	local msg
@@ -25,8 +24,7 @@ foundry_msg_merge_new() {
 				"repository" "$repository" \
 				"srcbranch"  "$srcbranch"  \
 				"dstbranch"  "$dstbranch"  \
-				"status"     "$status"     \
-				"log"        "$log"); then
+				"status"     "$status"); then
 		return 1
 	fi
 
@@ -102,16 +100,3 @@ foundry_msg_merge_get_status() {
 	echo "$status"
 	return 0
 }
-
-foundry_msg_merge_get_log() {
-	local msg="$1"
-
-	local log
-
-	if ! log=$(foundry_msg_get_data_field "$msg" "log"); then
-		return 1
-	fi
-
-	echo "$log"
-	return 0
-}