Foundry/taf
1
0
Fork 0
Code Issues 5 Pull requests Releases 1 Wiki Activity Actions

Remove compatability code that isn't required

Browse source
This commit is contained in:
Oliver 2026-01-11 13:25:11 -07:00
parent 47e5eab951
commit afb909a8a2
2 changed files with 2 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

7
module/sockets/query/notify.mjs
View file

@ -2,7 +2,7 @@ import { localizer } from "../../utils/localizer.mjs";
import { respondedToQueries } from "../../utils/QueryManager.mjs";
export function queryNotify(payload, sender) {
const { id, userID, content, includeGM, includeRequestor } = payload;
const { id, userID, content, includeRequestor } = payload;
if (userID !== game.user.id) { return };
@ -11,11 +11,6 @@ export function queryNotify(payload, sender) {
let whisper = [game.user.id];
// TODO: remove this code with #19
if (includeGM) {
whisper = game.users.filter(u => u.isGM).map(u => u.id);
};
if (includeRequestor) {
whisper = [sender.id];
};

11
module/utils/QueryManager.mjs
View file

@ -229,25 +229,16 @@ async function maybeResolve(requestID) {
};
};
export async function notify(requestID, userID, content, { includeGM = false, includeRequestor } = {}) {
export async function notify(requestID, userID, content, { includeRequestor } = {}) {
// Prevent sending notifications for not-your queries
if (!queries.has(requestID)) { return };
// TODO: remove this code with #19
if (includeGM) {
foundry.utils.logCompatibilityWarning(
`The "includeGM" option has been deprecated in favour of "includeRequestor"`,
{ since: `v2.4.0`, until: `v3.0.0` },
);
};
game.socket.emit(`system.taf`, {
event: `query.notify`,
payload: {
id: requestID,
userID,
content,
includeGM, // TODO: remove this code with #19
includeRequestor,
},
});