Opened 9 years ago

Closed 9 years ago

#24 closed defect (fixed)

Rooms view broken after change of room name

Reported by: gregoa Owned by:
Priority: serious Milestone: confclerk 0.5.2
Component: GUI Version: confclerk 0.5.1
Keywords: Cc:

Description

The debconf talks team just changed the names of the rooms in penta.
After an update the new names don't appear in the Roms tab but the old ones are duplicated ...

Change History (5)

comment:1 Changed 9 years ago by gregoa

Priority: important → serious

comment:2 Changed 9 years ago by gregoa

Milestone: confclerk 0.6.0 → confclerk 0.5.2

comment:3 Changed 9 years ago by gregoa

hopefully fixed in r1364

comment:4 Changed 9 years ago by philipp

I quickly reviewed your change. Nice :-)

But why do you first check whether the event is already there and delete it just if it is there? The SQL statement DELETE FROM EVENT_ROOM WHERE xid_conference=:conference_id AND xid_event=:event_id" does nothing without error if the conference_id/event_id combination does not exist.

Maybe it would be even possible to perform a DELETE FROM EVENT_ROOM WHERE xid_conference=:conference_id before assigning the new rooms?

comment:5 Changed 9 years ago by gregoa

Resolution: → fixed
Status: new → closed

thanks philipp, updated in r1365

Note: See TracTickets for help on using tickets.