CiviCRM Community Forums (archive)

Just reproduced this on CiviCRM 4.3 Demo site:

Memberships -> New Membership -> (borrowed) Allan Cooper
California Culture Fund - Select: Student - Fixed (a 1y fixed period Membership)
Member Since: 11/14/2011 (say he joined 2y ago - but we're only getting around to entering him now)
Start Date: 11/14/2013 (today - renewal chq in hand for the next year)
End Date: leave empty
Save

Check Allan's Membership:

Active: Student (which is a rolling one - that worked as expected for these data entered)
But: Inactive (expired): Student - Fixed:

Membership        Start Date                   End Date                      Status   
Student - Fixed        November 14th, 2013   December 31st, 2011      Expired

I was expecting an End Date = December 31, 2013

Looks like for Fixed period Membership Types CiviCRM calculates End date using Member Since instead of Start Date. I don't understand when you would (ever?) want to use the MemberSince (also called Join) date to calculate an End date - shouldn't it always be based on Start Date?

I came across something similar during an import this morning: http://forum.civicrm.org/index.php/topic,30668.0.html - it's easy to ignore MemberSince during an import - but it now looks like the actual calculation doesn't work for Fixed period Memberships.

i think you might want to consider doing this in 2 steps
create his membership from 2 years ago
renew that membership based on the current payment you have.

Is it following the logic explained here?

http://wiki.civicrm.org/confluence/display/CRMDOC40/Membership+Status+Rules

I remember a page where there were a whole bunch of test cases

I don't think this is it but I can't find a more recent one and it is similar to what i was looking for. dgg is best guy to get a response from i think

http://wiki.civicrm.org/confluence/display/CRM/Test+Plan+-+Membership+Renewal

If you are importing, can't you just import the End Date (expiration date)

I think we need two edits (this is a 4.3.7):

1. If there is a start date - use it to calculate the $fixedStartDate. If there is no start date - then use $joinDate (="today").

Old:
$toDay = explode('-', $joinDate);
// get year from join date
$year = $toDay[0];
$month = $toDay[1];

New:
           
// get year from join date
// if !$startDate then use $joinDate
$toDay = explode('-', (empty($startDate) ? $joinDate : $startDate));
$year = $toDay[0];
$month = $toDay[1];


2.
Old:
       
//make sure membership should not start in future.
if ($joinDate < $actualStartDate) {
   $actualStartDate = date('Y-m-d', mktime(0, 0, 0, $startMonth, $startDay, $year - 1));
}

New:
       
//make sure membership should not start in future.
// if ($joinDate < $actualStartDate) {
if ($startDate < $actualStartDate) {
   $actualStartDate = date('Y-m-d', mktime(0, 0, 0, $startMonth, $startDay, $year - 1));
}

Will write this up in JIRA - for review.

The example I created in #1 has been wiped. I set up a new one on 4.4 demo site:

Student: rolling 1y Membership Type (comes standard w/ CiviCRM)
Student - Fixed: fixed 1y Membership Type (just created)

Both are with the same Organization: MrAdvocate

New Contact: Robyn A

1. Add Membership:
MrAdvocate: Student
Member Since: 01/01/2010
Start Date: 01/01/2013
End Date: leave empty
don't record payment
Save

Result: as expected: End Date is calculated correctly: Dec. 31, 2013

2. Add Membership:
MrAdvocate: Student - Fixed
Member Since: 01/01/2010
Start Date: 01/01/2013
End Date: leave empty
don't record payment
Save

Result: is wrong: End Date is calculated incorrectly: Dec. 31, 2010

View of both Memberships:
http://drupal.demo.civicrm.org/civicrm/contact/view?reset=1&cid=206

in JIRA:
http://issues.civicrm.org/jira/browse/CRM-13786

Patch made it into 4.2.2 and 4.2.14 (last night). It will also be in 4.3.9 once that comes out.

Should read: 4.4.2 instead of 4.2.2 !