Notifications
Clear all

wpForo 1.x.x Extra padding in admin bar area for any user other than admin

22 Posts
5 Users
11 Likes
2,377 Views
Posts: 199
 fawp
Topic starter
(@fawp)
Reputable Member
Joined: 4 years ago
Posted by: @tutrix

sorry, for me this is not a very big problem
i added the 2 codes and everything works fine

Thanks tutrix, and I understand, by adding the code everything works.

 

As for me, I want to establish what the root cause of the problem is. I do not want to start adding code as workaround to fix every potential bug.

 

So I'll keep digging. 🙂

Reply
1 Reply
Tutrix
(@tutrix)
Joined: 4 years ago

Noble Member
Posts: 1121

So I'll keep digging. 🙂

good luck digging and tell us when you find something 😀 

Reply
Posts: 19
(@zmgma)
Eminent Member
Joined: 4 years ago

I found the problem.

The fact is that WPFORO changed their add_action hook from init to admin_bar_init in 38 line of wpf-includes/wpf-hooks.php

To make a long story short,

due to the load order of show_admin_bar(false), the _admin_bar_bump_cb is loaded.

I don't know the reason WPFORO changed the action hook. Is add_action on 'init' bad behavior?

Reply
3 Replies
 fawp
(@fawp)
Joined: 4 years ago

Reputable Member
Posts: 199

thank you @zmgma

 

@robert or @alvina, are you able to comment on @zmgma's post please?

Reply
Alvina
Moderator
(@alvina)
Joined: 4 years ago

Member
Posts: 1870

Hi @fawp, @zmgma, @tutrix, @danniee,

The issues will be fixed in the next version update.

Reply
 fawp
(@fawp)
Joined: 4 years ago

Reputable Member
Posts: 199
Posted by: @alvina

Hi @fawp, @zmgma, @tutrix, @danniee,

The issues will be fixed in the next version update.

Great, thank you for confirming Alvina.

 

Reply
Page 3 / 3