@frissdiegurke said:
seems similar to the bug https://github.com/NodeBB/NodeBB/issues/3489 so you may just need to update your dependencies (rm -rf node_modules && npm cache clean && npm i).
That worked, thanks!
@pichalite
Yer I think your right here, I've done a hack for now but knowing quite well that will get over written on the next upgrade.
And even tho this would normally be a minor concern for normal links, links to a another app on the site really need to be in a seperate tab.
So I've had to hack that as well, so I need to get my head around writting some plug-ins and how they all work, I want to fork this one [nodebb-plugin-outside] and see what I can do with it as I think this would serve my purpose but I don't think it's working at the moment.
Do you know if anyone has had any luck with getting any of the debuggers working?
I learn faster by seeing what's going on under the hood.
@rod It's nothing special, it's just a habit that I have trained myself into.
The reason I check for the existence of app.user is because you cannot reference an object property (that is, the uid property of the user object in app) if the object itself does not exist. You'll get a javascript error (ReferenceError: user is not defined).
If there's even a chance the parent object is not defined, I make a check for it first (though app is always defined in NodeBB, so I skip that).
The proper way to write it would be: if (app.hasOwnProperty('user')), but even sometimes that can be tripped up if app.user is a property, but undefined
