Skip to content

Commit 06c95ba

Browse files
committed
Ruby: QLDoc
1 parent 75e1497 commit 06c95ba

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

ruby/ql/lib/codeql/ruby/frameworks/ActiveResource.qll

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ module ActiveResource {
3838
this.getSuperclassExpr() = model.getAValueReachableFromSource().asExpr().getExpr()
3939
}
4040

41+
/** Gets the API node for this model */
4142
API::Node getModelApiNode() { result = model }
4243

4344
SiteAssignCall getASiteAssignment() { result.getModelClass() = this }
@@ -66,6 +67,7 @@ module ActiveResource {
6667
this = classMethodCall(model, _)
6768
}
6869

70+
/** Gets the model class for this call. */
6971
ModelClass getModelClass() { result.getModelApiNode() = model }
7072
}
7173

@@ -79,13 +81,15 @@ module ActiveResource {
7981
SiteAssignCall() { model = modelApiNode() and this = classMethodCall(model, "site=") }
8082

8183
/**
82-
* A node that contributes to the URLs used for HTTP requests by the parent
84+
* Gets a node that contributes to the URLs used for HTTP requests by the parent
8385
* class.
8486
*/
8587
DataFlow::Node getAUrlPart() { result = this.getArgument(0) }
8688

89+
/** Gets the model class for this call. */
8790
ModelClass getModelClass() { result.getModelApiNode() = model }
8891

92+
/** Holds if this site value specifies HTTP rather than HTTPS. */
8993
predicate disablesCertificateValidation() {
9094
this.getAUrlPart().asExpr().getConstantValue().getString().regexpMatch("^http(^s)")
9195
}
@@ -154,6 +158,7 @@ module ActiveResource {
154158
)
155159
}
156160

161+
/** Gets the model class for this instance. */
157162
ModelClass getModelClass() { result = cls }
158163
}
159164

@@ -165,9 +170,11 @@ module ActiveResource {
165170

166171
ModelInstanceMethodCall() { this.getReceiver() = i }
167172

173+
/** Gets the model instance for this call. */
168174
ModelInstance getInstance() { result = i }
169175

170-
ModelClass getModelClass() { result = this.getReceiver().(ModelInstance).getModelClass() }
176+
/** Gets the model class for this call. */
177+
ModelClass getModelClass() { result = i.getModelClass() }
171178
}
172179

173180
/**
@@ -185,6 +192,7 @@ module ActiveResource {
185192
)
186193
}
187194

195+
/** Gets the model class for this collection. */
188196
ModelClass getModelClass() { result = classMethodCall.getModelClass() }
189197
}
190198

@@ -194,6 +202,7 @@ module ActiveResource {
194202
class CollectionCall extends DataFlow::CallNode {
195203
CollectionCall() { this.getReceiver() instanceof Collection }
196204

205+
/** Gets the collection for this call. */
197206
Collection getCollection() { result = this.getReceiver() }
198207
}
199208

0 commit comments

Comments
 (0)