directus 9.23.1 → 9.23.3

package/dist/services/specifications.js

@@ -6,8 +6,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.SpecificationService = void 0;
 const specs_1 = __importDefault(require("@directus/specs"));
 const lodash_1 = require("lodash");
-// @ts-ignore
 const package_json_1 = require("../../package.json");
+const constants_1 = require("../constants");
 const database_1 = __importDefault(require("../database"));
 const env_1 = __importDefault(require("../env"));
 const get_relation_type_1 = require("../utils/get-relation-type");
@@ -15,10 +15,17 @@ const collections_1 = require("./collections");
 const fields_1 = require("./fields");
 const graphql_1 = require("./graphql");
 const relations_1 = require("./relations");
-const constants_1 = require("../constants");
 // @ts-ignore
 const format_title_1 = __importDefault(require("@directus/format-title"));
 class SpecificationService {
+    accountability;
+    knex;
+    schema;
+    fieldsService;
+    collectionsService;
+    relationsService;
+    oas;
+    graphql;
     constructor(options) {
         this.accountability = options.accountability || null;
         this.knex = options.knex || (0, database_1.default)();
@@ -36,93 +43,13 @@ class SpecificationService {
 }
 exports.SpecificationService = SpecificationService;
 class OASSpecsService {
+    accountability;
+    knex;
+    schema;
+    fieldsService;
+    collectionsService;
+    relationsService;
     constructor(options, { fieldsService, collectionsService, relationsService, }) {
-        this.fieldTypes = {
-            alias: {
-                type: 'string',
-            },
-            bigInteger: {
-                type: 'integer',
-                format: 'int64',
-            },
-            binary: {
-                type: 'string',
-                format: 'binary',
-            },
-            boolean: {
-                type: 'boolean',
-            },
-            csv: {
-                type: 'array',
-                items: {
-                    type: 'string',
-                },
-            },
-            date: {
-                type: 'string',
-                format: 'date',
-            },
-            dateTime: {
-                type: 'string',
-                format: 'date-time',
-            },
-            decimal: {
-                type: 'number',
-            },
-            float: {
-                type: 'number',
-                format: 'float',
-            },
-            hash: {
-                type: 'string',
-            },
-            integer: {
-                type: 'integer',
-            },
-            json: {},
-            string: {
-                type: 'string',
-            },
-            text: {
-                type: 'string',
-            },
-            time: {
-                type: 'string',
-                format: 'time',
-            },
-            timestamp: {
-                type: 'string',
-                format: 'timestamp',
-            },
-            unknown: {
-                type: undefined,
-            },
-            uuid: {
-                type: 'string',
-                format: 'uuid',
-            },
-            geometry: {
-                type: 'object',
-            },
-            'geometry.Point': {
-                type: 'object',
-            },
-            'geometry.LineString': {
-                type: 'object',
-            },
-            'geometry.Polygon': {
-                type: 'object',
-            },
-            'geometry.MultiPoint': {
-                type: 'object',
-            },
-            'geometry.MultiLineString': {
-                type: 'object',
-            },
-            'geometry.MultiPolygon': {
-                type: 'object',
-            },
-        };
         this.accountability = options.accountability || null;
         this.knex = options.knex || (0, database_1.default)();
         this.schema = options.schema;
@@ -131,11 +58,10 @@ class OASSpecsService {
         this.relationsService = relationsService;
     }
     async generate() {
-        var _a, _b;
         const collections = await this.collectionsService.readByQuery();
         const fields = await this.fieldsService.readAll();
         const relations = (await this.relationsService.readAll());
-        const permissions = (_b = (_a = this.accountability) === null || _a === void 0 ? void 0 : _a.permissions) !== null && _b !== void 0 ? _b : [];
+        const permissions = this.accountability?.permissions ?? [];
         const tags = await this.generateTags(collections);
         const paths = await this.generatePaths(permissions, tags);
         const components = await this.generateComponents(collections, fields, relations, tags);
@@ -152,14 +78,15 @@ class OASSpecsService {
                     description: 'Your current Directus instance.',
                 },
             ],
-            tags,
             paths,
-            components,
         };
+        if (tags)
+            spec.tags = tags;
+        if (components)
+            spec.components = components;
         return spec;
     }
     async generateTags(collections) {
-        var _a;
         const systemTags = (0, lodash_1.cloneDeep)(specs_1.default.tags);
         const tags = [];
         // System tags that don't have an associated collection are always readable to the user
@@ -180,18 +107,20 @@ class OASSpecsService {
                 }
             }
             else {
-                tags.push({
+                const tag = {
                     name: 'Items' + (0, format_title_1.default)(collection.collection).replace(/ /g, ''),
-                    description: ((_a = collection.meta) === null || _a === void 0 ? void 0 : _a.note) || undefined,
                     'x-collection': collection.collection,
-                });
+                };
+                if (collection.meta?.note) {
+                    tag.description = collection.meta.note;
+                }
+                tags.push(tag);
             }
         }
         // Filter out the generic Items information
         return tags.filter((tag) => tag.name !== 'Items');
     }
     async generatePaths(permissions, tags) {
-        var _a, _b, _c, _d, _e;
         const paths = {};
         if (!tags)
             return paths;
@@ -200,11 +129,11 @@ class OASSpecsService {
             if (isSystem) {
                 for (const [path, pathItem] of Object.entries(specs_1.default.paths)) {
                     for (const [method, operation] of Object.entries(pathItem)) {
-                        if ((_a = operation.tags) === null || _a === void 0 ? void 0 : _a.includes(tag.name)) {
+                        if (operation.tags?.includes(tag.name)) {
                             if (!paths[path]) {
                                 paths[path] = {};
                             }
-                            const hasPermission = ((_b = this.accountability) === null || _b === void 0 ? void 0 : _b.admin) === true ||
+                            const hasPermission = this.accountability?.admin === true ||
                                 'x-collection' in tag === false ||
                                 !!permissions.find((permission) => permission.collection === tag['x-collection'] &&
                                     permission.action === this.getActionForMethod(method));
@@ -212,7 +141,7 @@ class OASSpecsService {
                                 if ('parameters' in pathItem) {
                                     paths[path][method] = {
                                         ...operation,
-                                        parameters: [...((_c = pathItem.parameters) !== null && _c !== void 0 ? _c : []), ...((_d = operation === null || operation === void 0 ? void 0 : operation.parameters) !== null && _d !== void 0 ? _d : [])],
+                                        parameters: [...(pathItem.parameters ?? []), ...(operation?.parameters ?? [])],
                                     };
                                 }
                                 else {
@@ -228,7 +157,7 @@ class OASSpecsService {
                 const detailBase = (0, lodash_1.cloneDeep)(specs_1.default.paths['/items/{collection}/{id}']);
                 const collection = tag['x-collection'];
                 for (const method of ['post', 'get', 'patch', 'delete']) {
-                    const hasPermission = ((_e = this.accountability) === null || _e === void 0 ? void 0 : _e.admin) === true ||
+                    const hasPermission = this.accountability?.admin === true ||
                         !!permissions.find((permission) => permission.collection === collection && permission.action === this.getActionForMethod(method));
                     if (hasPermission) {
                         if (!paths[`/items/${collection}`])
@@ -332,13 +261,12 @@ class OASSpecsService {
         return paths;
     }
     async generateComponents(collections, fields, relations, tags) {
-        var _a;
         let components = (0, lodash_1.cloneDeep)(specs_1.default.components);
         if (!components)
             components = {};
         components.schemas = {};
         // Always includes the schemas with these names
-        if (((_a = specs_1.default.components) === null || _a === void 0 ? void 0 : _a.schemas) !== null) {
+        if (specs_1.default.components?.schemas !== null) {
             for (const schemaName of constants_1.OAS_REQUIRED_SCHEMAS) {
                 if (specs_1.default.components.schemas[schemaName] !== null) {
                     components.schemas[schemaName] = (0, lodash_1.cloneDeep)(specs_1.default.components.schemas[schemaName]);
@@ -377,7 +305,7 @@ class OASSpecsService {
         return components;
     }
     filterCollectionFromParams(parameters) {
-        return parameters.filter((param) => (param === null || param === void 0 ? void 0 : param.$ref) !== '#/components/parameters/Collection');
+        return parameters.filter((param) => param?.$ref !== '#/components/parameters/Collection');
     }
     getActionForMethod(method) {
         switch (method) {
@@ -393,16 +321,15 @@ class OASSpecsService {
         }
     }
     generateField(field, relations, tags, fields) {
-        var _a, _b;
-        let propertyObject = {
-            nullable: (_a = field.schema) === null || _a === void 0 ? void 0 : _a.is_nullable,
-            description: ((_b = field.meta) === null || _b === void 0 ? void 0 : _b.note) || undefined,
-        };
-        const relation = relations.find((relation) => {
-            var _a;
-            return (relation.collection === field.collection && relation.field === field.field) ||
-                (relation.related_collection === field.collection && ((_a = relation.meta) === null || _a === void 0 ? void 0 : _a.one_field) === field.field);
-        });
+        let propertyObject = {};
+        if (field.schema && 'is_nullable' in field.schema) {
+            propertyObject.nullable = field.schema.is_nullable;
+        }
+        if (field.meta?.note) {
+            propertyObject.description = field.meta.note;
+        }
+        const relation = relations.find((relation) => (relation.collection === field.collection && relation.field === field.field) ||
+            (relation.related_collection === field.collection && relation.meta?.one_field === field.field));
         if (!relation) {
             propertyObject = {
                 ...propertyObject,
@@ -417,7 +344,7 @@ class OASSpecsService {
             });
             if (relationType === 'm2o') {
                 const relatedTag = tags.find((tag) => tag['x-collection'] === relation.related_collection);
-                const relatedPrimaryKeyField = fields.find((field) => { var _a; return field.collection === relation.related_collection && ((_a = field.schema) === null || _a === void 0 ? void 0 : _a.is_primary_key); });
+                const relatedPrimaryKeyField = fields.find((field) => field.collection === relation.related_collection && field.schema?.is_primary_key);
                 if (!relatedTag || !relatedPrimaryKeyField)
                     return propertyObject;
                 propertyObject.oneOf = [
@@ -431,7 +358,7 @@ class OASSpecsService {
             }
             else if (relationType === 'o2m') {
                 const relatedTag = tags.find((tag) => tag['x-collection'] === relation.collection);
-                const relatedPrimaryKeyField = fields.find((field) => { var _a; return field.collection === relation.collection && ((_a = field.schema) === null || _a === void 0 ? void 0 : _a.is_primary_key); });
+                const relatedPrimaryKeyField = fields.find((field) => field.collection === relation.collection && field.schema?.is_primary_key);
                 if (!relatedTag || !relatedPrimaryKeyField)
                     return propertyObject;
                 propertyObject.type = 'array';
@@ -463,8 +390,97 @@ class OASSpecsService {
         }
         return propertyObject;
     }
+    fieldTypes = {
+        alias: {
+            type: 'string',
+        },
+        bigInteger: {
+            type: 'integer',
+            format: 'int64',
+        },
+        binary: {
+            type: 'string',
+            format: 'binary',
+        },
+        boolean: {
+            type: 'boolean',
+        },
+        csv: {
+            type: 'array',
+            items: {
+                type: 'string',
+            },
+        },
+        date: {
+            type: 'string',
+            format: 'date',
+        },
+        dateTime: {
+            type: 'string',
+            format: 'date-time',
+        },
+        decimal: {
+            type: 'number',
+        },
+        float: {
+            type: 'number',
+            format: 'float',
+        },
+        hash: {
+            type: 'string',
+        },
+        integer: {
+            type: 'integer',
+        },
+        json: {},
+        string: {
+            type: 'string',
+        },
+        text: {
+            type: 'string',
+        },
+        time: {
+            type: 'string',
+            format: 'time',
+        },
+        timestamp: {
+            type: 'string',
+            format: 'timestamp',
+        },
+        unknown: {},
+        uuid: {
+            type: 'string',
+            format: 'uuid',
+        },
+        geometry: {
+            type: 'object',
+        },
+        'geometry.Point': {
+            type: 'object',
+        },
+        'geometry.LineString': {
+            type: 'object',
+        },
+        'geometry.Polygon': {
+            type: 'object',
+        },
+        'geometry.MultiPoint': {
+            type: 'object',
+        },
+        'geometry.MultiLineString': {
+            type: 'object',
+        },
+        'geometry.MultiPolygon': {
+            type: 'object',
+        },
+    };
 }
 class GraphQLSpecsService {
+    accountability;
+    knex;
+    schema;
+    items;
+    system;
     constructor(options) {
         this.accountability = options.accountability || null;
         this.knex = options.knex || (0, database_1.default)();

package/dist/services/tfa.d.ts

@@ -1,6 +1,6 @@
-import { Knex } from 'knex';
+import type { Knex } from 'knex';
+import type { AbstractServiceOptions, PrimaryKey } from '../types';
 import { ItemsService } from './items';
-import { AbstractServiceOptions, PrimaryKey } from '../types';
 export declare class TFAService {
     knex: Knex;
     itemsService: ItemsService;

package/dist/services/tfa.js

@@ -9,6 +9,8 @@ const database_1 = __importDefault(require("../database"));
 const exceptions_1 = require("../exceptions");
 const items_1 = require("./items");
 class TFAService {
+    knex;
+    itemsService;
     constructor(options) {
         this.knex = options.knex || (0, database_1.default)();
         this.itemsService = new items_1.ItemsService('directus_users', options);
@@ -18,29 +20,29 @@ class TFAService {
             return otplib_1.authenticator.check(otp, secret);
         }
         const user = await this.knex.select('tfa_secret').from('directus_users').where({ id: key }).first();
-        if (!(user === null || user === void 0 ? void 0 : user.tfa_secret)) {
+        if (!user?.tfa_secret) {
             throw new exceptions_1.InvalidPayloadException(`User "${key}" doesn't have TFA enabled.`);
         }
         return otplib_1.authenticator.check(otp, user.tfa_secret);
     }
     async generateTFA(key) {
         const user = await this.knex.select('email', 'tfa_secret').from('directus_users').where({ id: key }).first();
-        if ((user === null || user === void 0 ? void 0 : user.tfa_secret) !== null) {
+        if (user?.tfa_secret !== null) {
             throw new exceptions_1.InvalidPayloadException('TFA Secret is already set for this user');
         }
-        if (!(user === null || user === void 0 ? void 0 : user.email)) {
+        if (!user?.email) {
             throw new exceptions_1.InvalidPayloadException('User must have a valid email to enable TFA');
         }
         const secret = otplib_1.authenticator.generateSecret();
         const project = await this.knex.select('project_name').from('directus_settings').limit(1).first();
         return {
             secret,
-            url: otplib_1.authenticator.keyuri(user.email, (project === null || project === void 0 ? void 0 : project.project_name) || 'Directus', secret),
+            url: otplib_1.authenticator.keyuri(user.email, project?.project_name || 'Directus', secret),
         };
     }
     async enableTFA(key, otp, secret) {
         const user = await this.knex.select('tfa_secret').from('directus_users').where({ id: key }).first();
-        if ((user === null || user === void 0 ? void 0 : user.tfa_secret) !== null) {
+        if (user?.tfa_secret !== null) {
             throw new exceptions_1.InvalidPayloadException('TFA Secret is already set for this user');
         }
         if (!otplib_1.authenticator.check(otp, secret)) {

package/dist/services/users.d.ts

@@ -1,6 +1,6 @@
-import { Knex } from 'knex';
-import { AbstractServiceOptions, Item, PrimaryKey, MutationOptions } from '../types';
-import { Query, SchemaOverview, Accountability } from '@directus/shared/types';
+import type { Accountability, Query, SchemaOverview } from '@directus/shared/types';
+import type { Knex } from 'knex';
+import type { AbstractServiceOptions, Item, MutationOptions, PrimaryKey } from '../types';
 import { ItemsService } from './items';
 export declare class UsersService extends ItemsService {
     knex: Knex;

package/dist/services/users.js

@@ -4,23 +4,25 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.UsersService = void 0;
+const exceptions_1 = require("@directus/shared/exceptions");
+const utils_1 = require("@directus/shared/utils");
 const jsonwebtoken_1 = __importDefault(require("jsonwebtoken"));
 const lodash_1 = require("lodash");
+const perf_hooks_1 = require("perf_hooks");
 const database_1 = __importDefault(require("../database"));
 const env_1 = __importDefault(require("../env"));
-const exceptions_1 = require("@directus/shared/exceptions");
 const exceptions_2 = require("../exceptions");
 const record_not_unique_1 = require("../exceptions/database/record-not-unique");
 const is_url_allowed_1 = __importDefault(require("../utils/is-url-allowed"));
-const utils_1 = require("@directus/shared/utils");
+const stall_1 = require("../utils/stall");
 const url_1 = require("../utils/url");
 const items_1 = require("./items");
 const mail_1 = require("./mail");
 const settings_1 = require("./settings");
-const stall_1 = require("../utils/stall");
-const perf_hooks_1 = require("perf_hooks");
-const utils_2 = require("@directus/shared/utils");
 class UsersService extends items_1.ItemsService {
+    knex;
+    accountability;
+    schema;
     constructor(options) {
         super('directus_users', options);
         this.knex = options.knex || (0, database_1.default)();
@@ -96,7 +98,7 @@ class UsersService extends items_1.ItemsService {
             .andWhere({ 'directus_roles.admin_access': true })
             .leftJoin('directus_roles', 'directus_users.role', 'directus_roles.id')
             .first();
-        const otherAdminUsersCount = +((otherAdminUsers === null || otherAdminUsers === void 0 ? void 0 : otherAdminUsers.count) || 0);
+        const otherAdminUsersCount = +(otherAdminUsers?.count || 0);
         if (otherAdminUsersCount === 0) {
             throw new exceptions_2.UnprocessableEntityException(`You can't remove the last admin user from the role.`);
         }
@@ -113,7 +115,7 @@ class UsersService extends items_1.ItemsService {
             .andWhere({ 'directus_users.status': 'active' })
             .leftJoin('directus_roles', 'directus_users.role', 'directus_roles.id')
             .first();
-        const otherAdminUsersCount = +((otherAdminUsers === null || otherAdminUsers === void 0 ? void 0 : otherAdminUsers.count) || 0);
+        const otherAdminUsersCount = +(otherAdminUsers?.count || 0);
         if (otherAdminUsersCount === 0) {
             throw new exceptions_2.UnprocessableEntityException(`You can't change the active status of the last admin user.`);
         }
@@ -179,13 +181,12 @@ class UsersService extends items_1.ItemsService {
      * Update many users by primary key
      */
     async updateMany(keys, data, opts) {
-        var _a, _b;
         try {
             if (data.role) {
                 // data.role will be an object with id with GraphQL mutations
-                const roleId = (_b = (_a = data.role) === null || _a === void 0 ? void 0 : _a.id) !== null && _b !== void 0 ? _b : data.role;
+                const roleId = data.role?.id ?? data.role;
                 const newRole = await this.knex.select('admin_access').from('directus_roles').where('id', roleId).first();
-                if (!(newRole === null || newRole === void 0 ? void 0 : newRole.admin_access)) {
+                if (!newRole?.admin_access) {
                     await this.checkRemainingAdminExistence(keys);
                 }
             }
@@ -280,7 +281,7 @@ class UsersService extends items_1.ItemsService {
         for (const email of emails) {
             const payload = { email, scope: 'invite' };
             const token = jsonwebtoken_1.default.sign(payload, env_1.default.SECRET, { expiresIn: '7d', issuer: 'directus' });
-            const subjectLine = subject !== null && subject !== void 0 ? subject : "You've been invited";
+            const subjectLine = subject ?? "You've been invited";
             const inviteURL = url ? new url_1.Url(url) : new url_1.Url(env_1.default.PUBLIC_URL).addPath('admin', 'accept-invite');
             inviteURL.setQuery('token', token);
             // Create user first to verify uniqueness
@@ -303,7 +304,7 @@ class UsersService extends items_1.ItemsService {
         if (scope !== 'invite')
             throw new exceptions_2.ForbiddenException();
         const user = await this.knex.select('id', 'status').from('directus_users').where({ email }).first();
-        if ((user === null || user === void 0 ? void 0 : user.status) !== 'invited') {
+        if (user?.status !== 'invited') {
             throw new exceptions_2.InvalidPayloadException(`Email address ${email} hasn't been invited.`);
         }
         // Allow unauthenticated update
@@ -321,7 +322,7 @@ class UsersService extends items_1.ItemsService {
             .from('directus_users')
             .whereRaw('LOWER(??) = ?', ['email', email.toLowerCase()])
             .first();
-        if ((user === null || user === void 0 ? void 0 : user.status) !== 'active') {
+        if (user?.status !== 'active') {
             await (0, stall_1.stall)(STALL_TIME, timeStart);
             throw new exceptions_2.ForbiddenException();
         }
@@ -333,7 +334,7 @@ class UsersService extends items_1.ItemsService {
             knex: this.knex,
             accountability: this.accountability,
         });
-        const payload = { email, scope: 'password-reset', hash: (0, utils_2.getSimpleHash)('' + user.password) };
+        const payload = { email, scope: 'password-reset', hash: (0, utils_1.getSimpleHash)('' + user.password) };
         const token = jsonwebtoken_1.default.sign(payload, env_1.default.SECRET, { expiresIn: '1d', issuer: 'directus' });
         const acceptURL = url
             ? new url_1.Url(url).setQuery('token', token).toString()
@@ -353,7 +354,6 @@ class UsersService extends items_1.ItemsService {
         await (0, stall_1.stall)(STALL_TIME, timeStart);
     }
     async resetPassword(token, password) {
-        var _a;
         const { email, scope, hash } = jsonwebtoken_1.default.verify(token, env_1.default.SECRET, { issuer: 'directus' });
         if (scope !== 'password-reset' || !hash)
             throw new exceptions_2.ForbiddenException();
@@ -365,7 +365,7 @@ class UsersService extends items_1.ItemsService {
             opts.preMutationException = err;
         }
         const user = await this.knex.select('id', 'status', 'password').from('directus_users').where({ email }).first();
-        if ((user === null || user === void 0 ? void 0 : user.status) !== 'active' || hash !== (0, utils_2.getSimpleHash)('' + user.password)) {
+        if (user?.status !== 'active' || hash !== (0, utils_1.getSimpleHash)('' + user.password)) {
             throw new exceptions_2.ForbiddenException();
         }
         // Allow unauthenticated update
@@ -373,7 +373,7 @@ class UsersService extends items_1.ItemsService {
             knex: this.knex,
             schema: this.schema,
             accountability: {
-                ...((_a = this.accountability) !== null && _a !== void 0 ? _a : { role: null }),
+                ...(this.accountability ?? { role: null }),
                 admin: true, // We need to skip permissions checks for the update call below
             },
         });

package/dist/services/utils.d.ts

@@ -1,6 +1,6 @@
-import { Knex } from 'knex';
-import { AbstractServiceOptions, PrimaryKey } from '../types';
-import { Accountability, SchemaOverview } from '@directus/shared/types';
+import type { Accountability, SchemaOverview } from '@directus/shared/types';
+import type { Knex } from 'knex';
+import type { AbstractServiceOptions, PrimaryKey } from '../types';
 export declare class UtilsService {
     knex: Knex;
     accountability: Accountability | null;

package/dist/services/utils.js

@@ -6,30 +6,32 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.UtilsService = void 0;
 const database_1 = __importDefault(require("../database"));
 const collections_1 = require("../database/system-data/collections");
-const exceptions_1 = require("../exceptions");
 const emitter_1 = __importDefault(require("../emitter"));
+const exceptions_1 = require("../exceptions");
 class UtilsService {
+    knex;
+    accountability;
+    schema;
     constructor(options) {
         this.knex = options.knex || (0, database_1.default)();
         this.accountability = options.accountability || null;
         this.schema = options.schema;
     }
     async sort(collection, { item, to }) {
-        var _a, _b, _c, _d;
         const sortFieldResponse = (await this.knex.select('sort_field').from('directus_collections').where({ collection }).first()) ||
             collections_1.systemCollectionRows;
-        const sortField = sortFieldResponse === null || sortFieldResponse === void 0 ? void 0 : sortFieldResponse.sort_field;
+        const sortField = sortFieldResponse?.sort_field;
         if (!sortField) {
             throw new exceptions_1.InvalidPayloadException(`Collection "${collection}" doesn't have a sort field.`);
         }
-        if (((_a = this.accountability) === null || _a === void 0 ? void 0 : _a.admin) !== true) {
-            const permissions = (_c = (_b = this.accountability) === null || _b === void 0 ? void 0 : _b.permissions) === null || _c === void 0 ? void 0 : _c.find((permission) => {
+        if (this.accountability?.admin !== true) {
+            const permissions = this.accountability?.permissions?.find((permission) => {
                 return permission.collection === collection && permission.action === 'update';
             });
             if (!permissions) {
                 throw new exceptions_1.ForbiddenException();
             }
-            const allowedFields = (_d = permissions.fields) !== null && _d !== void 0 ? _d : [];
+            const allowedFields = permissions.fields ?? [];
             if (allowedFields[0] !== '*' && allowedFields.includes(sortField) === false) {
                 throw new exceptions_1.ForbiddenException();
             }
@@ -37,7 +39,7 @@ class UtilsService {
         const primaryKeyField = this.schema.collections[collection].primary;
         // Make sure all rows have a sort value
         const countResponse = await this.knex.count('* as count').from(collection).whereNull(sortField).first();
-        if ((countResponse === null || countResponse === void 0 ? void 0 : countResponse.count) && +countResponse.count !== 0) {
+        if (countResponse?.count && +countResponse.count !== 0) {
             const lastSortValueResponse = await this.knex.max(sortField).from(collection).first();
             const rowsWithoutSortValue = await this.knex
                 .select(primaryKeyField, sortField)
@@ -59,7 +61,7 @@ class UtilsService {
             .groupBy(sortField)
             .from(collection)
             .havingRaw('count(??) > 1', [sortField]);
-        if ((duplicates === null || duplicates === void 0 ? void 0 : duplicates.length) > 0) {
+        if (duplicates?.length > 0) {
             const ids = await this.knex.select(primaryKeyField).from(collection).orderBy(sortField);
             // This might not scale that well, but I don't really know how to accurately set all rows
             // to a sequential value that works cross-DB vendor otherwise

package/dist/services/webhooks.d.ts

@@ -1,6 +1,6 @@
-import { AbstractServiceOptions, Item, PrimaryKey, Webhook, MutationOptions } from '../types';
-import { ItemsService } from './items';
 import { Messenger } from '../messenger';
+import type { AbstractServiceOptions, Item, MutationOptions, PrimaryKey, Webhook } from '../types';
+import { ItemsService } from './items';
 export declare class WebhooksService extends ItemsService<Webhook> {
     messenger: Messenger;
     constructor(options: AbstractServiceOptions);

package/dist/services/webhooks.js

@@ -1,9 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.WebhooksService = void 0;
-const items_1 = require("./items");
 const messenger_1 = require("../messenger");
+const items_1 = require("./items");
 class WebhooksService extends items_1.ItemsService {
+    messenger;
     constructor(options) {
         super('directus_webhooks', options);
         this.messenger = (0, messenger_1.getMessenger)();