Crm 15475

The reason for using Pseudoconstant vs the buildoptions function was to avoid a bit of overhead,  buildoptions would return all the statuses and checks a number of php conditions which are not needed in this case.  CRM_Member_PseudoConstant::membershipStatus() is been called multiple times before my code calls it and since the status gets cached, it directly retrieves value from cache.  

Since my aim is to just return only the status 'Cancelled' even if it is inactive / disabled i used pseudoconstant with the filter(as i need Id of only one option), Since i feel both the function returns with same results as what i need.

NB. If the plan is to avoid using pseudoconstant to retrieve statuses, then we should implement buildOptions in other places in CiviCRM and get rid of the definition totally.