Rename internal vars

mablae · mablae · commit 8e2e96f41b39 · 2018-05-01T02:18:28.000+02:00
diff --git a/src/Symfony/Component/Ldap/Adapter/ExtLdap/EntryManager.php b/src/Symfony/Component/Ldap/Adapter/ExtLdap/EntryManager.php
@@ -124,19 +124,19 @@ private function getConnectionResource()
     }
 
     /**
-     * @param string                     $dn            Distinguished name to batch modify
-     * @param iterable|UpdateOperation[] $modifications An array or iterable of Modification instances
+     * @param string                     $dn         Distinguished name to apply operations on
+     * @param iterable|UpdateOperation[] $operations An array or iterable of Modification instances
      *
      * @throws UpdateOperationException in case of an error
      */
-    public function applyOperations(string $dn, iterable $modifications): void
+    public function applyOperations(string $dn, iterable $operations): void
     {
-        $modificationsMapped = array();
-        foreach ($modifications as $modification) {
-            $modificationsMapped[] = $modification->toArray();
+        $operationsMapped = array();
+        foreach ($operations as $modification) {
+            $operationsMapped[] = $modification->toArray();
         }
 
-        if (!@ldap_modify_batch($this->getConnectionResource(), $dn, $modificationsMapped)) {
+        if (!@ldap_modify_batch($this->getConnectionResource(), $dn, $operationsMapped)) {
             throw new UpdateOperationException(sprintf('Error executing batch modification on "%s": "%s".', $dn, ldap_error($this->getConnectionResource())));
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -124,19 +124,19 @@ private function getConnectionResource()`
`124`	`124`	`}`
`125`	`125`
`126`	`126`	`/**`
`127`		`- * @param string $dn Distinguished name to batch modify`
`128`		`- * @param iterable\|UpdateOperation[] $modifications An array or iterable of Modification instances`
	`127`	`+ * @param string $dn Distinguished name to apply operations on`
	`128`	`+ * @param iterable\|UpdateOperation[] $operations An array or iterable of Modification instances`
`129`	`129`	`*`
`130`	`130`	`* @throws UpdateOperationException in case of an error`
`131`	`131`	`*/`
`132`		`- public function applyOperations(string $dn, iterable $modifications): void`
	`132`	`+ public function applyOperations(string $dn, iterable $operations): void`
`133`	`133`	`{`
`134`		`- $modificationsMapped = array();`
`135`		`- foreach ($modifications as $modification) {`
`136`		`- $modificationsMapped[] = $modification->toArray();`
	`134`	`+ $operationsMapped = array();`
	`135`	`+ foreach ($operations as $modification) {`
	`136`	`+ $operationsMapped[] = $modification->toArray();`
`137`	`137`	`}`
`138`	`138`
`139`		`- if (!@ldap_modify_batch($this->getConnectionResource(), $dn, $modificationsMapped)) {`
	`139`	`+ if (!@ldap_modify_batch($this->getConnectionResource(), $dn, $operationsMapped)) {`
`140`	`140`	`throw new UpdateOperationException(sprintf('Error executing batch modification on "%s": "%s".', $dn, ldap_error($this->getConnectionResource())));`
`141`	`141`	`}`
`142`	`142`	`}`