Probably better to make those years actually be ints instead of what I
assume is strings at some point


On 4 June 2014 21:38, Andy Durant <notifications@github.com> wrote:

> In src/Classes/ServiceAPI/MyRadio_User.php:
>
> > @@ -1565,8 +1565,8 @@ public function isActiveMemberForYear($year=null)
> >              $year = CoreUtils::getAcademicYear();
> >          }
> >          // If the current year exists in payments (even with a value of £0, the member is active)
> > -        foreach ($this->payment as $v) {
> > -            if ($v['year'] === $year) {
> > +        foreach ($this->getAllPayments() as $payment) {
>
> I basically looked at how it was done elsewhere (L351) and copied that =P
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/UniversityRadioYork/MyRadio/pull/75/files#r13409209>.
>