Temporary conference are getting bookmarked

Description

Steps to reproduce:
1.Log into Spark
2.Create a temporary conference room from the option available under Actions->Join Conference room->Create or Join conference room named "Conf1"
3.Close the conference chat room window
4.Now try to bookmark the temporary conference room created "Conf1"

Expected result: The temporary conference rooms should not allow to bookmark

Actual result: We are able to bookmark the temporary conference room

Environment

None

Attachments

1

Activity

Show:

wroot March 27, 2014 at 8:33 PM

Looks like it now works correctly.

wroot August 14, 2011 at 12:13 PM

I still get the error when trying to bookmark a persistent room.

Mircea Carasel August 12, 2011 at 4:02 PM

commited:

r12571 = d4a5ecc2160d44654f3d31ad4a2c5a2c3a76b16b

Description:

-removed the filter added on a previous commit that was preventing for a temporary room to be added on the room browser list
(that filtering cannot be consistent - all queried rooms should show on the room browser, otherwise its confusing)
-when room configuration is changed, and marked as non persistent, the bookmark associated will automatically be removed (if any)
changes will take effect at next spark restart

Mircea Carasel August 12, 2011 at 11:18 AM

What is noticed is that in Spark group chat room configuration dialog,
when I create a persistent room - if then I uncheck: List Room in Directory checkbox (the Persistent checkbox remains checked)
then in spark that room is reported as NOT persistent

The code: final RoomInfo roomInfo = MultiUserChat.getRoomInfo(SparkManager.getConnection(), roomJID);
persistent = roomInfo.isPersistent();

returns persistent = false.

If then I mark as checked the List Room in Directory checkbox, the code from above returns persistent true (Persistent checkbox always marked as checked - so the room is technically persistent - I went to Openfire Admin Console and that room was marked always as persistent)

so looks like a bug in SMACK...I think.

As a side note: I will remove my filtration that when a room is created temporary not to add it in the browser list - that is wrong: the: List Room in Directory option offers such behavior

wroot August 11, 2011 at 7:35 AM

This maybe related to some other bug. All my rooms show N/A in the Occupants column. This is due to some bug in the embedded database. It wasn't showing any rooms in the Conference browser. Then Wolf made a workaround to show N/A if there are errors in the database. Maybe that's why i can't bookmark persistent rooms now? The only thing to fix errors is to start with a fresh database which is not an option for most users.

I have updated my source code via SVN, builded it with Netbeans and launched on XP, then i tried to create a persistent room via Spark and Admin Console. Then i refreshed the room browser and tried to bookmark one of these rooms and got the error.

Fixed

Details

Priority

Assignee

Reporter

Affects versions

Fix versions

Components

Created August 8, 2011 at 5:50 AM
Updated March 27, 2014 at 8:33 PM
Resolved March 27, 2014 at 8:33 PM