Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: return null if field isn't in hash

Browse Source
This commit is contained in:
Barış Soner Uşaklı
2025-02-11 13:37:21 -05:00
parent e85662a536
commit 70a9f6d337
5 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/categories/data.js
View File

@@ -73,7 +73,7 @@ module.exports = function (Categories) {
Categories.getCategoryField = async function (cid, field) { Categories.getCategoryField = async function (cid, field) {
const category = await Categories.getCategoryFields(cid, [field]); const category = await Categories.getCategoryFields(cid, [field]);
return category ? category[field] : null; return category && category.hasOwnProperty(field) ? category[field] : null;
}; };
Categories.getCategoryFields = async function (cid, fields) { Categories.getCategoryFields = async function (cid, fields) {

2
src/groups/data.js
View File

@@ -51,7 +51,7 @@ module.exports = function (Groups) {
Groups.getGroupField = async function (groupName, field) { Groups.getGroupField = async function (groupName, field) {
const groupData = await Groups.getGroupFields(groupName, [field]); const groupData = await Groups.getGroupFields(groupName, [field]);
return groupData ? groupData[field] : null; return groupData && groupData.hasOwnProperty(field) ? groupData[field] : null;
}; };
Groups.getGroupFields = async function (groupName, fields) { Groups.getGroupFields = async function (groupName, fields) {

2
src/messaging/data.js
View File

@@ -29,7 +29,7 @@ module.exports = function (Messaging) {
Messaging.getMessageField = async (mid, field) => { Messaging.getMessageField = async (mid, field) => {
const fields = await Messaging.getMessageFields(mid, [field]); const fields = await Messaging.getMessageFields(mid, [field]);
return fields ? fields[field] : null; return fields && fields.hasOwnProperty(field) ? fields[field] : null;
}; };
Messaging.getMessageFields = async (mid, fields) => { Messaging.getMessageFields = async (mid, fields) => {

2
src/posts/data.js
View File

@@ -37,7 +37,7 @@ module.exports = function (Posts) {
Posts.getPostField = async function (pid, field) { Posts.getPostField = async function (pid, field) {
const post = await Posts.getPostFields(pid, [field]); const post = await Posts.getPostFields(pid, [field]);
return post ? post[field] : null; return post && post.hasOwnProperty(field) ? post[field] : null;
}; };
Posts.getPostFields = async function (pid, fields) { Posts.getPostFields = async function (pid, fields) {

2
src/topics/data.js
View File

@@ -40,7 +40,7 @@ module.exports = function (Topics) {
Topics.getTopicField = async function (tid, field) { Topics.getTopicField = async function (tid, field) {
const topic = await Topics.getTopicFields(tid, [field]); const topic = await Topics.getTopicFields(tid, [field]);
return topic ? topic[field] : null; return topic && topic.hasOwnProperty(field) ? topic[field] : null;
}; };
Topics.getTopicFields = async function (tid, fields) { Topics.getTopicFields = async function (tid, fields) {