fix(projects.pipelineschedule): Remove redundant mapping in notSaved(…

…) and notDeleted()

Signed-off-by: Vitalijs <[email protected]>

pkg/controller/projects/pipelineschedules/controller.go

@@ -368,32 +368,17 @@ func isVariablesUpToDate(crv []v1alpha1.PipelineVariable, inv []*gitlab.Pipeline
 }
 
 func notDeleted(inv *gitlab.PipelineVariable, crvArr []v1alpha1.PipelineVariable) bool {
-	victim := v1alpha1.PipelineVariable{
-		Key:          inv.Key,
-		Value:        inv.Value,
-		VariableType: &inv.Value,
-	}
-
 	for _, v := range crvArr {
-		if victim.Key == v.Key {
+		if inv.Key == v.Key {
 			return false
 		}
 	}
 	return true
 }
 
 func notSaved(crv v1alpha1.PipelineVariable, invArr []*gitlab.PipelineVariable) bool {
-	victim := gitlab.PipelineVariable{
-		Key:   crv.Key,
-		Value: crv.Value,
-	}
-
-	if crv.VariableType != nil {
-		victim.VariableType = *crv.VariableType
-	}
-
 	for _, v := range invArr {
-		if victim.Key == v.Key {
+		if crv.Key == v.Key {
 			return false
 		}
 	}