Forcing UID value

Simos

Is not possible force the uid value on user.create ? i'm doing this but it look don't work :

        if (!_uid) {
            user.create({
                uid: userObj.uid,
                username: userObj.display,
                email: userObj.email
            }, callback);

<baris>

It's not possible right now, user.create generates the next available uid and uses it. Any particular reason you want to overwrite it?

Simos

i've made a fork of nodebb-plugin-login-mysite ( https://github.com/SimosNap/nodebb-plugin-login-anope ) to login to the forum using IRC anope account. On IRC users can change both username and email so UID is the only things i can refer for account updates

Simos

@baris do you have any suggestions for my purpose ?

julian

I'd recommend against updating the uid, but letting NodeBB handle uids on its own. If you want to save the id of the user, then you should save it as a different value into the user hash instead...

Simos

@julian it's only some days i'm using nodebb and i'm new to redis too, do you mean create a new field in user table ? If so how do ?

julian

In the fork of your login plugin, you should be able to save the remoteId into the user hash on the user.create call.

Simos

@julian Pheraps i miss something , i don't see an user hash field in user table :

[ { uid: 4,
username: 'antani',
userslug: 'antani',
picture: '',
status: 'online',
postcount: 0,
reputation: 0,
'email:confirmed': 0,
lastonline: 1542925209728,
flags: null,
banned: 0,
'banned:expire': 0,
joindate: 1542925209728,
uploadedpicture: undefined,
'icon:text': 'A',
'icon:bgColor': '#2196f3',
joindateISO: '2018-11-22T22:20:09.728Z',
lastonlineISO: '2018-11-22T22:20:09.728Z',
banned_until: 0,
banned_until_readable: 'Not Banned',
administrator: false } ]

PitaJ

@Simos what you just posted is what Julian referred to by "user hash". He was saying "hash table" otherwise called an object or a dictionary.

Simos

@PitaJ i've try to add the remoteid on user.create :

	console.log(_uid);
        if (!_uid) {
            user.create({
                remoteid: userObj.uid,
                username: userObj.display,
                email: userObj.email
            }, callback);
        } else {
            callback(null, _uid)
        }

I suppose it don't add the remoteid value because remoteid field doesn't exist right ?

<baris>

You will need a plugin to save the remoteid into the user hash, so in your plugin listen for 'filter:user.create' here is how it should look.

myPlugin.filterUserCreate = function (hookData, callback) {
    hookData.user.remoteid = hookData.data.remoteid;
    callback(null, hookData);
};

Simos

Thanks ! Why i can get it with user.getUserFields but not with user.getUser ?

Simos

Is there any existent function to get uid having a field value ? (in my case external uid)

julian

.getUserField() should work, though you will need to implement the whitelist as @baris has mentioned.

Simos

@julian it look getUserField() need nodebb uid i've to get userdata having the remoteid (the customfield i've added)

plugin.filterUserCreate = function (data, callback) {
data.user.auid = data.data.auid;
callback(null, data);
};

plugin.whitelistFields = function(data, callback) {
data.whitelist.push('auid');
callback(null, data);
};

PitaJ

Yeah you'll have to save a map of auids to uids in the database.

Simos

@PitaJ is there any example of code for this ?

julian

@Simos what have you tried so far?

Simos

@julian I've try only user.getUserDataByField function than i've discovered is limited.

julian

Yes, you can use that method. You just need to create a plugin hook listener to the whitelist hook @baris mentioned, so you can add the remoteId to the whitelisted fields.