fix(controls): skip controls for modify and del

Author: Mathieu Bour

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## [Version 0.1.3 - Fix Optional Controls](https://github.com/csquare-ai/ldapjs-client/releases/tag/0.1.3)
+
+- **fix(controls)**: skip controls for `modify` and `del`
+
 ## [Version 0.1.2 - Optional Controls](https://github.com/csquare-ai/ldapjs-client/releases/tag/0.1.2)
 
 - **feat**: controls arguments are now optional

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@csquare/ldapjs-client",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Promisified wrapper around the ldapjs client.",
   "keywords": [
     "ldap",

package.json

@@ -78,7 +78,7 @@ export class Client extends EventEmitter {
     return new Promise<void>((resolve, reject) => {
       const cb: ErrorCallback = (error) => (!error ? resolve() : reject(error));
 
-      if (controls) {
+      if (!controls) {
         this.ldapjs.del(name, cb);
       } else {
         this.ldapjs.del(name, controls, cb);
@@ -90,7 +90,7 @@ export class Client extends EventEmitter {
     return new Promise<void>((resolve, reject) => {
       const cb: ErrorCallback = (error) => (!error ? resolve() : reject(error));
 
-      if (controls) {
+      if (!controls) {
         this.ldapjs.modify(name, change, cb);
       } else {
         this.ldapjs.modify(name, change, controls, cb);