vladmos / photoblog

Photoblog engine with picasa as a photo storage.
0 stars 0 forks source link

Отдельно обработать случай одного дня #5

Closed alien43 closed 12 years ago

alien43 commented 12 years ago

Если начальная и конечная даты совпадают, писать её один раз, а не дважды через тире —.

vladmos commented 12 years ago

Обрабатывается же, разве нет? Как ты это узнал без учетки?

alien43 commented 12 years ago

Я не узнал, я на всякий случай.

def event_date_human(self):
    if self.event_beginning == self.event_end:
        return self.event_beginning.strftime('%d.%m.%Y')
    return u'%s — %s' % (
        self.event_beginning.strftime('%d.%m.%Y'),
        self.event_end.strftime('%d.%m.%Y'),

Фи, как некрасиво. Надо так:

def event_date_human(self):
    def date_to_string(d):
        return d.strftime('%d.%m.%Y')
    if self.event_beginning == self.event_end:
        return date_to_string(self.event_beginning)
    return u'%s — %s' % (
        date_to_string(self.event_beginning),
        date_to_string(self.event_end))
vladmos commented 12 years ago

В общем, оттуда следует, что работает.

Вообще, это черновой вариант, в беловом даты не должны показываться, только гд и месяц(ы). Твой вариант не сильно лучше - менее гибкий. Даты в разных случаях надо будет показывать по-разному: январь 2012, январь -февраль 2012, декабрь 2011 - январь 2012.

alien43 commented 12 years ago

Нет.

vladmos commented 12 years ago

Нет.

alien43 commented 12 years ago

Что нет-то? Нетто?