Commit 25abc2ac authored by Per Cederqvist's avatar Per Cederqvist
Browse files

(mark_as_read): The list of read texts is part of the person, so use

	mark_person_as_changed, not mark_conference_as_changed.  (Bug 877).
(mark_as_unread): FIXME comment removed.
(set_read_ranges): Ditto.
parent eb885995
/*
* $Id: membership.c,v 0.83 2002/11/27 23:36:50 ceder Exp $
* $Id: membership.c,v 0.84 2003/01/10 01:09:39 ceder Exp $
* Copyright (C) 1991-2002 Lysator Academic Computer Association.
*
* This file is part of the LysKOM server.
......@@ -1561,11 +1561,7 @@ mark_as_read (Conf_no conference,
}
adjust_read(m, conf_c);
/* FIXME (bug 877): shouldn't this be mark_person_as_changed? */
mark_conference_as_changed(conference);
if (active_connection->cwc == conference)
mark_person_as_changed(ACTPERS);
mark_person_as_changed(ACTPERS);
}
#ifdef DEBUG_MARK_AS_READ
......@@ -1623,8 +1619,6 @@ mark_as_unread(Conf_no conference,
{
remove_loc_no(lno, m);
adjust_read(m, conf_c);
/* FIXME (bug 877): mark_as_read uses mark_conference_as_changed
instead. If that is proper, then this line is wrong. */
mark_person_as_changed(ACTPERS);
}
......@@ -1713,8 +1707,6 @@ set_read_ranges(Conf_no conference,
sfree(m->read_ranges);
m->read_ranges = NULL;
m->no_of_read_ranges = 0;
/* FIXME (bug 877): mark_as_read uses mark_conference_as_changed
instead. If that is proper, then this line is wrong. */
mark_person_as_changed(ACTPERS);
}
}
......@@ -1737,8 +1729,6 @@ set_read_ranges(Conf_no conference,
memcpy(m->read_ranges, read_ranges->ranges,
m->no_of_read_ranges * sizeof(m->read_ranges[0]));
adjust_read(m, conf_c);
/* FIXME (bug 877): mark_as_read uses mark_conference_as_changed
instead. If that is proper, then this line is wrong. */
mark_person_as_changed(ACTPERS);
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment