0.6.0 Breaking Changes

[baris]

filter:category.get parameters changed, related commit
https://github.com/NodeBB/NodeBB/commit/db979d7a20acb93ed1961f2c3579e2ed660bf89e

@trevor I know this affects one of the plugins that you use.

[baris]

Two new hooks action:post.purge and action:topic.purge. Relevant commit https://github.com/NodeBB/NodeBB/commit/f9f1cf63cf9f46dfc2c4a0d64139efbe09873638

[Schamper]

So with most (all?) multi-argument hooks being updated to use a single data object, what will become of the single argument hooks? Will they get the same treatment with in mind that more data can be added later?

Sorry if this has been discussed before, must've missed it.

[baris]

Ideally we should change those to be objects as well so we don't have to do this amount of breaking changes in the future. Ie stuff like

plugins.fireHook('action:post.delete', pid);

Should be

plugins.fireHook('action:post.delete', {pid: pid, uid:uid});

There aren't alot of these as far as I know so I will make a list and let you all know about the changes here when they happen.

[julian]

@baris I thought we were leaving the single-argument hooks as-is for now? I don't mind changing it, though... may as well get it all over in one go.

[frissdiegurke]

@julian said:

@baris I thought we were leaving the single-argument hooks as-is for now? I don't mind changing it, though...

may as well get it all over in one go.

[baris]

Recent Replies widget is deprecated please switch over to Recent Posts widget if you were using it. It has the same functionality.

[baris]

Updated with link to config changes. https://community.nodebb.org/topic/3392/config-json-updated-base_url-and-relative_path-no-longer-exist

[agarcia17]

I'm confused, are all hooks being updated to use a single data object? in particular
plugins.fireHook('action:topic.delete', tid); ?

PS. is there a set date you guys are aiming for the 0.6.0 release?

[baris]

Yes, all hooks use only a single param now.

No more plugins.fireHook('someHookName', a, b, c, d, callback);

Not all the params are an object though like the one you posted.

[PitaJ]

@baris make every. Single. Hook. Have an object param. That way we will never ever need to worry about breaking changes again!

Seriously: consistency is the best possible thing you can have with software

[baris]

Yeah agreed will try to get that in for 0.6.0

[agarcia17]

ohh ok thanks for making it clear

[baris]

Updated with header.tpl changes, relevant thread https://community.nodebb.org/topic/3488/header-tpl-changes