[#SAK-5945] MsgCntr "unread only" topic view renders blank rows for read messages

Sakai

MsgCntr "unread only" topic view renders blank rows for read messages

Created: 16-Aug-2006 12:19 Updated: 23-Oct-2008 11:08

Component/s:

Message Center (Deprecated as of 2.4) - Don't use

Affects Version/s:

2.1.2, 2.1.x

Fix Version/s:

2.1.x, 2.3.0

Time Tracking:

Not Specified

File Attachments:	1. sakai-BugFix-SAK-5945.patch (19 kB)
Environment:	« Hide sakai-2-1-x branch msgcntr-2-1-2 branch Show » sakai-2-1-x branch msgcntr-2-1-2 branch

2.6.x Status:	None
2.5.x Status:	None
2.4.x Status:	None

Description

« Hide

Select a forum and topic that has both read and unread messages. From withing the topic view, selected View = Unread Only. This renders a message list showing only the unread messages but it also renders blank lines for all read messages. Please see attached screen shots.

For more information please see:
This issue is also tracked in our UCD-JIRA instance: https://jira.ucdavis.edu:8444/jira/browse/SAK-128

All

Comments

Work Log

Change History

Subversion Commits

git Commits

Sort Order:

[ Permlink | « Hide ]

Thomas Amsler added a comment - 28-Sep-2006 08:21

BugFix has been attached to this issue.

[ Permlink | « Hide ]

Kristol Hancock added a comment - 05-Oct-2006 15:02

Michelle, can you take a look at the fix that Thomas provided and check it in if all checks out ok? Thanks! Kris

[ Permlink | « Hide ]

Michelle Wagner added a comment - 06-Oct-2006 10:21

r15744

[ Permlink | « Hide ]

Nicolas Pachis added a comment - 13-Oct-2006 07:31

Issue is still present on both:
http://qa1-uk.sakaiproject.org
http://qa1-us.sakaiproject.org:8580/portal

There are still blank spaces left in the display of topics when you select "View Unread Only"

[ Permlink | « Hide ]

Megan May added a comment - 13-Oct-2006 07:36

I mismarked and this will be in 2.4

[ Permlink | « Hide ]

Brian Jorgensen added a comment - 17-Nov-2006 16:14

Fixed in 2.1.x in r18226.

[ Permlink | « Hide ]

Megan May added a comment - 08-Feb-2007 09:04

original fix merged in prior to 2.3 branch so this would be included in that release

[ Permlink | « Hide ]

Megan May added a comment - 08-Feb-2007 09:05

Due to inadaquete resources, this was not tested during the 2.3. QA cycle