BBP Style Pack 5.1.0 site login crash WordPress 6.1

Resolved theoverload
(@theoverload)

1 year, 5 months ago

Hello Robin – firstly thank you as ever for your excellent plugin. User and appreciator of what it adds to bbpress

Today WordPress notified the update to BBP Style Pack 5.1.0 citing it to be compatible with WordPress 6.1

However, that update brought on immediate site login crash and lockout with the following errors:

Cannot modify header information – headers already sent by (output started at/homepages/2/dblahblah/htdocs/appblahblahblah/wp-content/plugins/bbp-style-pack/includes/settings_subscriptions_management.php:120) in
/homepages/2/dblahblah/htdocs/appblahblah/wp-login.php
on line
464 etc

Only resolution was to deactivate BBP Style Pack plugin and there was no roll back option.

Obviously seems to to do with log in? This is a simple site. Has no subscriptions etc. Uses Loginpress and Login Limits Reloaded plugins current versions.

All was fine after Wordress 6.1 update until BBP Style Pack update?

How do I either (a) roll back to bbp style pack version that works with 6.1 (b) fix/avoid? Any questions dont hesitate to ask.

Thanks and your mighty efforts always appreciated

Kr D

Viewing 14 replies - 1 through 14 (of 14 total)

scottmotion
(@scottmotion)

1 year, 5 months ago
I’m crashing as well on post.php. Haven’t tried login.

As a temporary solution you can disable the plugin and get the code from bspstyle.css which will fix some of the issues.
- This reply was modified 1 year, 5 months ago by scottmotion.
- This reply was modified 1 year, 5 months ago by scottmotion.
RT_PDX
(@ronald2020)

1 year, 5 months ago

Same here. When trying to manage subscriptions from user panel see Debug below. Also, please see my other post when enqueuing bsp javascript files that use jQuery but do not have the jQuery dependency when loaded

When seeking help with this issue, you may be asked for some of the following information: WordPress version 6.1 Active theme: Twenty Twenty-One Child Current plugin: bbp style pack (version 5.1.0) PHP version 8.1.11 Error Details ============= An error of type E_ERROR was caused in line 584 of the file //public_html/wp-content/plugins/bbp-style-pack/includes/subscriptions_management.php. Error message: Uncaught Error: Call to undefined function bsptoolkit_removehtml_and_cutwords() in public_html/wp-content/plugins/bbp-style-pack/includes/subscriptions_management.php:584 Stack trace: #0 /public_html/wp-includes/class-wp-hook.php(308): bsp_forum_subscriptions_page() #1 /public_html/wp-includes/class-wp-hook.php(332): WP_Hook->apply_filters() #2 /public_html/wp-includes/plugin.php(517): WP_Hook->do_action() #3 /public_html/wp-admin/admin.php(259): do_action() #4 /public_html/wp-admin/edit.php(10): require_once('/home/...') #5 {main} thrown

Plugin Author Robin W
(@robin-w)

1 year, 5 months ago

Apologies, I have reverted to the previous version – now released as 5.1.1.

I’ll take a look at why this was erroring for you guys but not for me

Sorry, I try my best to test all changes

Thread Starter theoverload
(@theoverload)

1 year, 5 months ago

Thank you Robin. That reversion released as 5.1.1 worked.

Should further error detail help, happy to help I already emailed. And thanks again for all you do it’s really appreciated.

Kr D

RT_PDX
(@ronald2020)

1 year, 5 months ago

Thanks Robin. Happy to help testing. Not sure if this is one of the things you intended to fix in 5.1.0 but I get this error since rolling back to 5.1.1 (aka 5.0.9)

Got error ‘PHP message: PHP Warning: Undefined array key “HTTP_REFERER” in /public_html/wp-content/plugins/bbp-style-pack/includes/functions.php on line 2681

Plugin Author Robin W
(@robin-w)

1 year, 5 months ago

thanks, that error has been there for a while, as it is only a warning, I haven’t got around to fixing it – it is on a list but doesn’t affect how the plugin works.

But yes – I’m working on a fix to my previous attempts to incorporate a disused plugin, and would love help in ensuring it works this time – I’ll come back shortly

RT_PDX
(@ronald2020)

1 year, 5 months ago

Fair enough! If it is any help, instead of the (unreliable) HTTP_REFERER $_SERVER variable add this function and call it instead:

// Get the full URL of the current page
function current_page_url(){
$page_url = ‘http’;
if(isset($_SERVER[‘HTTPS’]) && $_SERVER[‘HTTPS’] == ‘on’){
$page_url .= ‘s’;
}
return $page_url.’://’.$_SERVER[‘SERVER_NAME’].$_SERVER[‘REQUEST_URI’];
}

Plugin Author Robin W
(@robin-w)

1 year, 5 months ago

I think I have found the issues.

However rather than add the amended code back and find any further issues, I’ve put it into an additional plugin, prior to incorporating it.

If any of you fancy testing for me, that would be great.

so go to

http://www.rewweb.co.uk/download/rew-subscriptions/

and download, then add as a plugin by uploading it

Then in forums, look at subscriptions and maybe add and delete one
and in topics do the same
and finally in users

Let me know if if it looks ok.

As I say it will be added back into the style pack plugin in the next week or so, unless anyone finds any issues.

Plugin Author Robin W
(@robin-w)

1 year, 5 months ago

@ronald2020

HTTP_REFERER changed as per above, and jquery amended in 5.1.2 soon to be released – thanks for your help 🙂

RT_PDX
(@ronald2020)

1 year, 5 months ago

Hello Robin,

Thanks for the quick turn around. I installed and tested it. A few thoughts:

– The ability the add default forums to new users … GREAT! I had this as some custom code on my site already so i can remove that in the future
– I received one error after testing all features. Only details I can provide: 500 GET /wp-admin/options-general.php?page=rew-sub but i could not reproduce it. It was after i enabled subscriptions and added all forums to new user creation. Then i saved. When i went back there I got the 500. My guess is it has to do with pulling up the new user defaults.
– The subscription panel (e.g. forums) shows the Name of the user 3 times. Looks like it is the display name. E.g. I see: Manage subscriptions for user: C10 – C10 – C10. I think just the user login name would be just fine.
– In the admin panel, under forums, there is a tab called ‘subscriptions’. Personally i think it is clutter. It does not exist for topic or users and does not add really more than what is already described in the settings panel.
– In the subscriptions panel for a single user would it be possible to combine the add and remove forums options?
– In the settings tab the sentence: Subscribe new users automatically to the below ticked forums: is underlined. But it is not a link.

Thanks much, let me know if I need to elaborate on any of the above

Plugin Author Robin W
(@robin-w)

1 year, 5 months ago

Hey, thanks for the quick testing – I think all your points make sense, I’ll work through them now.

This was code that was in the bbp toolkit plugin which is no longer supported and was starting to error under php 8 and wp 6.1 – I forked most of it straight across into my plugin – I obviously missed some stuff in testing, hence the style pack problem.
Plugin Author Robin W
(@robin-w)

1 year, 5 months ago
@ronald2020

Many thanks again for your testing.

I could not get the 500 problem, so I’ll be pragmatic and ignore!!
I’ve changed the subscription panel to show login and display name if different.
single user – amended to allow both subscribe and unsubscribe
I’ve taken out the underline in ‘Subscribe new users’
on ‘In the admin panel, under forums, there is a tab called ‘subscriptions’.’ Agree, this is due to the way the plugin was written (by someone else). I only have so long to play with this, and can’t quite work out how he is doing the permissions thing for the subscriptions pages which seems to require this tab. The code is in lines 154-163 of /includes/subscriptions_managament.php if you fancy looking.

so I’ve done a fresh version – if you fancy taking a look, I’d be very happy, but of course equally happy for all your help so far if you don’t 🙂 🙂

http://www.rewweb.co.uk/download/rew-subscriptions/
- This reply was modified 1 year, 5 months ago by Robin W.
- This reply was modified 1 year, 5 months ago by Robin W.
RT_PDX
(@ronald2020)

1 year, 5 months ago

Hi Robin!

Single user forum update looks great! Cosmetics: I did notice that the page_sections (the < page > underneath Forum and Topics) are populated and clickable to next/prev pages even when all forums / topics (on my site) a user has assigned fit easily on the on a single page. It doesn’t throw errors so not a big deal. Same shows when managing Forums / Topics so it may be something e can tackle later for all three areas.

To remove the submenu without breaking the edit capability replace line 161 with:

$confHook = add_submenu_page(null, ‘Subscriptions’, ‘Subscriptions’, ‘edit_forums’, ‘forum_subscriptions’, ‘rew_forum_subscriptions_page’);

Hope this helps !

Best, Ronald

Plugin Author Robin W
(@robin-w)

1 year, 5 months ago

just released version 5.1.3 with subscriptions management.

@ronald2020 – you’ll need to deactivate the rew-subscriptions version, you were the only one who downloaded this, so I haven’t added a check into the style pack code 🙂

and yes major thanks for your code help here – really appreciate your wisdom and efforts

Viewing 14 replies - 1 through 14 (of 14 total)

The topic ‘BBP Style Pack 5.1.0 site login crash WordPress 6.1’ is closed to new replies.