eBierek

@Rick: Sure :)
It’s working for new members and renewing members but demoted members which have to upgrade from level 0 to 1 seems difficult to solve.

Is there a way I can split these roles up (by hook), just as Cristián suggested for the new users?

This should catch existing users when they renew their active account, and sets EOT to current year+1/12/31

add_action (" ? ", "my_fixed_EOT_time_upd");
function my_fixed_EOT_time_upd ($vars)
{	
$expire_on = strtotime((date('Y') + 1) . '-12-31');
update_user_option ($vars["user_id"], "s2member_auto_eot_time", $expire_on);
}

This should catch demoted users when they decide to pay, and upgrades from level 0 to 1, and sets EOT to current year/12/31

add_action (" ? ", "my_fixed_EOT_time_upg");
function my_fixed_EOT_time_upg ($vars)
{	
$expire_on = strtotime ( date('Y') . '-12-31');
update_user_option ($vars["user_id"], "s2member_auto_eot_time", $expire_on);
}

Hi,

I’m sorry to say this doesn’t work either.

add_action ("ws_plugin__s2member_during_paypal_notify_during_before_subscr_modify", "my_fixed_EOT_time_upd");
add_action ("ws_plugin__s2member_during_paypal_notify_before_subscr_signup_w_update_vars", "my_fixed_EOT_time_upd");

function my_fixed_EOT_time_upd ($vars)
{	
	//If user is demoted to subrsciber
	if (user_is($vars["user_id"], 'subscriber')) { 
		$expire_on = strtotime ( date('Y') . '-12-31');
		update_user_option ($vars["user_id"], "s2member_auto_eot_time", $expire_on);
	//elseif user is renewing active membership
	} else { 
		$expire_on = strtotime((date('Y') + 1) . '-12-31');
		update_user_option ($vars["user_id"], "s2member_auto_eot_time", $expire_on);
	} 
}

It still gives me (date(‘Y’) + 1) even if the user is a ‘subscriber’ (demoted). What can I use here? Many thanks.

Since a demoted user still is considered to be a “user that will updated” I need to work with these, right?:

add_action ("ws_plugin__s2member_during_paypal_notify_during_subscr_signup_w_update_vars", "my_fixed_EOT_time_upd");
add_action ("ws_plugin__s2member_during_paypal_notify_during_subscr_modify", "my_fixed_EOT_time_upd");

That also handles change for other users (level1-3). So I tried following:

function my_fixed_EOT_time_upd ($vars)
{
	if (current_user_is("s2member_level0")){ 
		$expire_on = strtotime ( date('Y') . '-12-31');
		update_user_option ($vars["user_id"], "s2member_auto_eot_time", $expire_on);
	
	} else { 
		$expire_on = strtotime((date('Y') + 1) . '-12-31');
		update_user_option ($vars["user_id"], "s2member_auto_eot_time", $expire_on);
	} 
}

This still gives me “current year + one year” even if the user is demoted (level0). Maybe the hooks cant handle conditions? Do you know of others that have tried this?

Hi and thank you for trying to help :)

I have now two functions as you suggested. One for new members setting EOT to “current year-12-31” and one for members updating their account setting EOT to “(current year+1)-12-31”. This works great!

But we also need to handle demoted users. When they decides to finally pay again, their EOT should be set to the same as new users -> “current year-12-31”. Can we handle them with another hook? Right now they get one year too much. Advice much appreciated.
/Emily

Hello again,

Setting EOT with conditionals works for new members and demoted members (level0) where EOT is empty. I have tested accounts logged in on level 1-3, no EOT update. But when I do following, without the (current_user_is(“s2member_level2”))-conditionals it works fine:

function my_fixed_EOT_time ($vars) {
$expire_on = strtotime((date('Y') + 1) . '-12-31');
update_user_option ($vars["user_id"], "s2member_auto_eot_time", $expire_on);	
}

This is confusing. Is this a bug?

Ok, I have tested alot and this works, it updates EOT:

function my_fixed_EOT_time ($vars) {
$date =  date('Y') . '-12-31'; //get current year and adds -12-31
$expire_on = strtotime ( '+1 year' , strtotime ( $date ) ) ; //add one year to current year
update_user_option ($vars["user_id"], "s2member_auto_eot_time", $expire_on);	
}

But when I add the conditionals for different levels, it doesn’t work. Can’t I use

else if (current_user_is("s2member_level2")) {some code to add EOT}

in s2hacks.php? See first post for reference.

Hi and thank you for answering :)

No need to use a hacks file. You can have a page accessible to Level 1 users (assuming that’s the paid access you sold), so only them would see this page. And in it you’d have a condition that checks the month and shows the renewal pro-form during November and December only.

But the EOT-code for renewal/upgrade must be in the s2hacks file, no?

Sorry if the questions are confusing.

Hi Eduan,

Thank you for advice. I have now tried removing mu-plugins, switching to another theme (2012), deactivating all other plugins but S2Member and the error still remains.

To make sure no custom code is interfering I installed a completely new WP 3.5 and downloaded lastest S2M and only this in the setup causes problems. Unfortunately.

First error after install:

Second error efter just klicking the plugin tab once:

The same errors appear with theme 2011 and 2012. Deactivating S2M also removes the errors. Is this a bug?

Hi again,

Thanks for the advice. This works great. When a new member registers, EOT is set to ‘current-year/12/31’.

Question for next step:
During the period of november 1st – 30th every year existing and logged in members can renew their membership. (I’m thinking they would access a renew-page in member menu.)

Then EOT should be set to ‘current-year+1/12/31’, but what would be the best method to do that? Another form? Paypal button (using paypal express checkout)? Are there hooks for renewals?

(After ‘current-year/12/31’ has passed, non paid members gets demoted, and can only see an upgrade page if they try to log in.)

Thank you, that seems to work. Could you explain the difference please?