This change fixes a compiler warning and changes the way a PPRScheduleTi...

...me computes its description string.

PPRScheduleTime.m - use a DateFormatter etc., rather than a crude stringWithFormat;
PPRTestIIntialiser.m [sic] - fix the warning;
PPRScheduleTimeTests.m - added, for a crude test.
I'll try and do this now.  

d


On 18/09/2014, at 16:48 , davidbernard <notifications@github.com> wrote:

> In ppreminderer/Models/PPRScheduleTime.m:
> 
>>    NSString *description;
>>    switch (self.type) {
>> -        case PPRScheduleTimeTimeOfDay:
>> -            description = [NSString stringWithFormat:@"At %02.2d %02.2d",
>> -                           (int)self.offset.hour, (int)self.offset.minute];
>> +        case PPRScheduleTimeTimeOfDay: {
>> +            
>> +            NSDate * todayButAtOffset = [[NSDate alloc] initWithTimeIntervalSinceNow:0];
>> +            NSCalendar *gregorian = [[NSCalendar alloc] initWithCalendarIdentifier: NSGregorianCalendar];
> 
> I think we should use the current calendar rather than gregorian. I think this is set up in the locale.
> 
> —
> Reply to this email directly or view it on GitHub.
> 

-- 
David A Vincent
Code Legal - Programming Matters
david.vincent@entasker.com
0404 985 493