Rename arguments for clarity and ensure correct usage

nginxinc · Nov 27, 2023 · 3d9f7e8 · 3d9f7e8
1 parent f77cc0b
commit 3d9f7e8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/internal/certification/certificates.go b/internal/certification/certificates.go
@@ -176,7 +176,7 @@ func (c *Certificates) handleDeleteEvent(obj interface{}) {
 	logrus.Debugf("Certificates::handleDeleteEvent: certificates (%d)", len(c.Certificates))
 }
 
-func (c *Certificates) handleUpdateEvent(oldValue interface{}, newValue interface{}) {
+func (c *Certificates) handleUpdateEvent(_ interface{}, newValue interface{}) {
 	logrus.Debug("Certificates::handleUpdateEvent")
 
 	secret, ok := newValue.(*corev1.Secret)

diff --git a/internal/configuration/settings.go b/internal/configuration/settings.go
@@ -266,10 +266,10 @@ func (s *Settings) handleDeleteEvent(obj interface{}) {
 	}
 }
 
-func (s *Settings) handleUpdateEvent(_ interface{}, obj interface{}) {
+func (s *Settings) handleUpdateEvent(_ interface{}, newValue interface{}) {
 	logrus.Debug("Settings::handleUpdateEvent")
 
-	configMap, yes := isOurConfig(obj)
+	configMap, yes := isOurConfig(newValue)
 	if !yes {
 		return
 	}