Address review comments

Author: knewbury01

javascript/frameworks/ui5/lib/advanced_security/javascript/frameworks/ui5/UI5LogsToHttpQuery.qll

@@ -29,7 +29,7 @@ module UI5LogEntryToHttp implements DataFlow::StateConfigSig {
     UI5LogInjection::isAdditionalFlowStep(start, end) and
     preState = postState
     or
-    stepLogger(start, end) and
+    logArgumentToListener(start, end) and
     preState = "not-logged-not-accessed" and
     postState = "logged-and-accessed"
   }

javascript/frameworks/ui5/lib/advanced_security/javascript/frameworks/ui5/dataflow/FlowSteps.qll

@@ -347,7 +347,7 @@ class ResourceBundleGetTextCallArgToReturnValueStep extends DataFlow::SharedFlow
  * A step from any argument of a SAP logging function to the `onLogEntry`
  * method of a custom log listener in the same application.
  */
-predicate stepLogger(DataFlow::Node start, DataFlow::Node end) {
+predicate logArgumentToListener(DataFlow::Node start, DataFlow::Node end) {
   inSameWebApp(start.getFile(), end.getFile()) and
   start =
     ModelOutput::getATypeNode("SapLogger")
@@ -357,6 +357,12 @@ predicate stepLogger(DataFlow::Node start, DataFlow::Node end) {
   end = ModelOutput::getATypeNode("SapLogEntries").asSource()
 }
 
+/**
+ * A step from any argument of a SAP logging function to the `onLogEntry`
+ * method of a custom log listener in the same application.
+ */
 class LogArgumentToListener extends DataFlow::SharedFlowStep {
-  override predicate step(DataFlow::Node start, DataFlow::Node end) { stepLogger(start, end) }
+  override predicate step(DataFlow::Node start, DataFlow::Node end) {
+    logArgumentToListener(start, end)
+  }
 }