tobias/radicale
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Rename variable recurrence to is_recurrence

Browse Source
This commit is contained in:
Unrud 2017-08-23 13:36:12 +02:00
parent f912642c20
commit 574961417d
1 changed files with 30 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

59
radicale/xmlutils.py
View File

@ -246,12 +246,12 @@ def _time_range_match(vobject_item, filter_, child_name):
matched = False matched = False
def range_fn(range_start, range_end, recurrence): def range_fn(range_start, range_end, is_recurrence):
nonlocal matched nonlocal matched
if start < range_end and range_start < end: if start < range_end and range_start < end:
matched = True matched = True
return True return True
if end < range_start and not recurrence: if end < range_start and not is_recurrence:
return True return True
return False return False
@ -267,7 +267,7 @@ def _visit_time_ranges(vobject_item, child_name, range_fn, infinity_fn):
`vobject_item`` with visitors ``range_fn`` and ``infinity_fn``. `vobject_item`` with visitors ``range_fn`` and ``infinity_fn``.
``range_fn`` gets called for every time_range with ``start`` and ``end`` ``range_fn`` gets called for every time_range with ``start`` and ``end``
datetimes and ``recurrence`` as arguments. If the function returns True, datetimes and ``is_recurrence`` as arguments. If the function returns True,
the operation is cancelled. the operation is cancelled.
``infinity_fn`` gets called when an infiite recurrence rule is detected ``infinity_fn`` gets called when an infiite recurrence rule is detected
@ -316,7 +316,7 @@ def _visit_time_ranges(vobject_item, child_name, range_fn, infinity_fn):
# Comments give the lines in the tables of the specification # Comments give the lines in the tables of the specification
if child_name == "VEVENT": if child_name == "VEVENT":
for child, recurrence, recurrences in get_children( for child, is_recurrence, recurrences in get_children(
vobject_item.vevent_list): vobject_item.vevent_list):
# TODO: check if there's a timezone # TODO: check if there's a timezone
dtstart = child.dtstart.value dtstart = child.dtstart.value
@ -345,30 +345,31 @@ def _visit_time_ranges(vobject_item, child_name, range_fn, infinity_fn):
if dtend is not None: if dtend is not None:
# Line 1 # Line 1
dtend = dtstart + timedelta(seconds=original_duration) dtend = dtstart + timedelta(seconds=original_duration)
if range_fn(dtstart, dtend, recurrence): if range_fn(dtstart, dtend, is_recurrence):
return return
elif duration is not None: elif duration is not None:
if original_duration is None: if original_duration is None:
original_duration = duration.seconds original_duration = duration.seconds
if duration.seconds > 0: if duration.seconds > 0:
# Line 2 # Line 2
if range_fn(dtstart, dtstart + duration, recurrence): if range_fn(dtstart, dtstart + duration,
is_recurrence):
return return
else: else:
# Line 3 # Line 3
if range_fn(dtstart, dtstart + SECOND, recurrence): if range_fn(dtstart, dtstart + SECOND, is_recurrence):
return return
elif dtstart_is_datetime: elif dtstart_is_datetime:
# Line 4 # Line 4
if range_fn(dtstart, dtstart + SECOND, recurrence): if range_fn(dtstart, dtstart + SECOND, is_recurrence):
return return
else: else:
# Line 5 # Line 5
if range_fn(dtstart, dtstart + DAY, recurrence): if range_fn(dtstart, dtstart + DAY, is_recurrence):
return return
elif child_name == "VTODO": elif child_name == "VTODO":
for child, recurrence, recurrences in get_children( for child, is_recurrence, recurrences in get_children(
vobject_item.vtodo_list): vobject_item.vtodo_list):
dtstart = getattr(child, "dtstart", None) dtstart = getattr(child, "dtstart", None)
duration = getattr(child, "duration", None) duration = getattr(child, "duration", None)
@ -407,7 +408,7 @@ def _visit_time_ranges(vobject_item, child_name, range_fn, infinity_fn):
reference_dates = (created,) reference_dates = (created,)
else: else:
# Line 8 # Line 8
if range_fn(DATETIME_MIN, DATETIME_MAX, recurrence): if range_fn(DATETIME_MIN, DATETIME_MAX, is_recurrence):
return return
reference_dates = () reference_dates = ()
@ -418,30 +419,30 @@ def _visit_time_ranges(vobject_item, child_name, range_fn, infinity_fn):
# Line 1 # Line 1
if range_fn(reference_date, if range_fn(reference_date,
reference_date + duration + SECOND, reference_date + duration + SECOND,
recurrence): is_recurrence):
return return
if range_fn(reference_date + duration - SECOND, if range_fn(reference_date + duration - SECOND,
reference_date + duration + SECOND, reference_date + duration + SECOND,
recurrence): is_recurrence):
return return
elif dtstart is not None and due is not None: elif dtstart is not None and due is not None:
# Line 2 # Line 2
due = reference_date + timedelta(seconds=original_duration) due = reference_date + timedelta(seconds=original_duration)
if (range_fn(reference_date, due, recurrence) or if (range_fn(reference_date, due, is_recurrence) or
range_fn(reference_date, range_fn(reference_date,
reference_date + SECOND, recurrence) or reference_date + SECOND, is_recurrence) or
range_fn(due - SECOND, due, recurrence) or range_fn(due - SECOND, due, is_recurrence) or
range_fn(due - SECOND, reference_date + SECOND, range_fn(due - SECOND, reference_date + SECOND,
recurrence)): is_recurrence)):
return return
elif dtstart is not None: elif dtstart is not None:
if range_fn(reference_date, reference_date + SECOND, if range_fn(reference_date, reference_date + SECOND,
recurrence): is_recurrence):
return return
elif due is not None: elif due is not None:
# Line 4 # Line 4
if range_fn(reference_date - SECOND, reference_date, if range_fn(reference_date - SECOND, reference_date,
recurrence): is_recurrence):
return return
elif completed is not None and created is not None: elif completed is not None and created is not None:
# Line 5 # Line 5
@ -449,26 +450,26 @@ def _visit_time_ranges(vobject_item, child_name, range_fn, infinity_fn):
seconds=original_duration) seconds=original_duration)
if (range_fn(reference_date - SECOND, if (range_fn(reference_date - SECOND,
reference_date + SECOND, reference_date + SECOND,
recurrence) or is_recurrence) or
range_fn(completed - SECOND, completed + SECOND, range_fn(completed - SECOND, completed + SECOND,
recurrence) or is_recurrence) or
range_fn(reference_date - SECOND, range_fn(reference_date - SECOND,
reference_date + SECOND, recurrence) or reference_date + SECOND, is_recurrence) or
range_fn(completed - SECOND, completed + SECOND, range_fn(completed - SECOND, completed + SECOND,
recurrence)): is_recurrence)):
return return
elif completed is not None: elif completed is not None:
# Line 6 # Line 6
if range_fn(reference_date - SECOND, if range_fn(reference_date - SECOND,
reference_date + SECOND, recurrence): reference_date + SECOND, is_recurrence):
return return
elif created is not None: elif created is not None:
# Line 7 # Line 7
if range_fn(reference_date, DATETIME_MAX, recurrence): if range_fn(reference_date, DATETIME_MAX, is_recurrence):
return return
elif child_name == "VJOURNAL": elif child_name == "VJOURNAL":
for child, recurrence, recurrences in get_children( for child, is_recurrence, recurrences in get_children(
vobject_item.vjournal_list): vobject_item.vjournal_list):
dtstart = getattr(child, "dtstart", None) dtstart = getattr(child, "dtstart", None)
@ -487,11 +488,11 @@ def _visit_time_ranges(vobject_item, child_name, range_fn, infinity_fn):
if dtstart_is_datetime: if dtstart_is_datetime:
# Line 1 # Line 1
if range_fn(dtstart, dtstart + SECOND, recurrence): if range_fn(dtstart, dtstart + SECOND, is_recurrence):
return return
else: else:
# Line 2 # Line 2
if range_fn(dtstart, dtstart + DAY, recurrence): if range_fn(dtstart, dtstart + DAY, is_recurrence):
return return
elif isinstance(child, date): elif isinstance(child, date):
@ -617,7 +618,7 @@ def find_tag_and_time_range(vobject_item):
return (None, TIMESTAMP_MIN, TIMESTAMP_MAX) return (None, TIMESTAMP_MIN, TIMESTAMP_MAX)
start = end = None start = end = None
def range_fn(range_start, range_end, recurrence): def range_fn(range_start, range_end, is_recurrence):
nonlocal start, end nonlocal start, end
if start is None or range_start < start: if start is None or range_start < start:
start = range_start start = range_start