[Plugin: Subscribe2] Multisite Issues

More debugging fodder for you… It looks like you’re actually trying to address this with line 17 of class-s2-core.php, but the check on line 13 is coming up false (at least for me) on all my installs.

Is there a reason you aren’t just using the conditional tag “is_multisite()” to determine if it’s part of a network? It correctly identifies for me, at least.

However, if I just replace your multisite check with “if(is_multisite())”, then I white screen when it tries to call is_blog_user, with the error “call to undefined function wp_get_current_user()” which must happen somewhere inside of is_blog_user

Matty,

Tried it just to confirm, but nope – no change. If you read my follow up comments, you’ll see that this code never actually gets run.

Line 13 in there is always returning false, skipping this whole block. When I manually enter that ‘if’ by using either if (true) or even if (is_multisite), then line 15 causes a whitescreen error somewhere down in the code by saying that wp_get_current_user is not defined.

For debugging, I did the following:

line 1646..
if ( function_exists(‘is_multisite’) && is_multisite() ) {
die(‘its multisite’);

and sure enough – it’s multisite at that point of the check.

However, back up at the code in question, I tried var_dump()’ing $this and at that point, s2_mu is in fact false.

I did a little MORE debugging, putting die(…) statements both in the area of line 1646 (s2init) and in the area of load_strings, and it turns out that load_strings is actually getting called BEFORE s2init, therefore it hasn’t had time to even see if it’s multisite or not yet

More general “trying to be helpful” debuggery:

I tried modifying subscribe2.php to be this at the very bottom:

function ok_really_init() {
	if ( is_admin() ) {
		require_once(S2PATH . 'classes/class-s2-admin.php');
		global $mysubscribe2;
		$mysubscribe2 = new s2_admin;
		$mysubscribe2->s2init();
	} else {
		require_once(S2PATH . 'classes/class-s2-frontend.php');
		global $mysubscribe2;
		$mysubscribe2 = new s2_frontend;
		$mysubscribe2->s2init();
	}
}
add_action('plugins_loaded', 'ok_really_init');

which gets around my undefined function problem, and thus modifies the widget’s subscribe text and link (MOAR PROGRESS!), but it still isn’t right. It generates this link for the text subscribe:
‘www.example.com/sub-site-slug/wp-admin/?s2mu_subscribe={blog-id}’, which looks better, but following the link still gives the error screen of:
You attempted to access the “Sub Site” dashboard, but you do not currently have priveleges on this site…

Line 1659ish says:
add_action(‘init’, array(&$this, ‘wpmu_subscribe’));

However, the function wpmu_subscribe is actually part of the class s2class_multisite, not “$this” – so this function is not being called properly.

Update:
Changing that line to this:
add_action(‘init’, array($s2class_multisite, ‘wpmu_subscribe’));

is working so far.. moving on to more elaborate testing!

You may have missed my ‘Update’ in the previous post, but yup – when I use that modified line, it’s working great! 😀