fix: ensure information about blocked services pause schedule exists …

…in state before attempting to use it (#86)
gmichels · Dec 19, 2023 · 5c5684a · 5c5684a
1 parent 48a140d
commit 5c5684a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/adguard/client_common.go b/adguard/client_common.go
@@ -90,7 +90,7 @@ func (o *clientCommonModel) Read(ctx context.Context, adg adguard.ADG, currState
 	// need special handling for timezone in resource due to inconsistent API response for `Local`
 	if rtype == "resource" {
 		// last updated will exist on create operation, null on import operation
-		if !currState.LastUpdated.IsNull() {
+		if !currState.LastUpdated.IsNull() && !currState.BlockedServicesPauseSchedule.IsNull() {
 			// unpack current state
 			var currStateBlockedServicesPauseScheduleClient scheduleModel
 			*diags = currState.BlockedServicesPauseSchedule.As(ctx, &currStateBlockedServicesPauseScheduleClient, basetypes.ObjectAsOptions{})

diff --git a/adguard/config_common.go b/adguard/config_common.go
@@ -558,7 +558,7 @@ func (o *configCommonModel) Read(ctx context.Context, adg adguard.ADG, currState
 	stateBlockedServicesPauseScheduleConfig := mapAdgScheduleToBlockedServicesPauseSchedule(ctx, &blockedServicesPauseSchedule.Schedule)
 
 	// need special handling for timezone in resource due to inconsistent API response for `Local`
-	if rtype == "resource" {
+	if rtype == "resource" && !currState.BlockedServicesPauseSchedule.IsNull() {
 		// last updated will exist on create operation, null on import operation
 		if !currState.LastUpdated.IsNull() {
 			// unpack current state