Forum OpenACS Q&A: Re: Room Reservation in contrib on CVS

Collapse
Posted by Bruno Mattarollo on

Hello Caroline,

Thanks for this package (also to Deds and Scott). I have installed it on my test instance (on my laptop) and I have a small patch already. When you make a reservation for a room, there is a missing argument in the approval for the room when this room has been created to not need approval. This is the patch:

[Bruno-Mattarollos-Computer:room-reservation/sql/postgresql] bruno% cvs diff -c reservations-package-create.sql
Index: reservations-package-create.sql
===================================================================
RCS file: /cvsroot/openacs-4/contrib/packages/room-reservation/sql/postgresql/reservations-package-create.sql,v
retrieving revision 1.1
diff -c -r1.1 reservations-package-create.sql
*** reservations-package-create.sql     15 Mar 2004 03:39:30 -0000      1.1
--- reservations-package-create.sql     15 Mar 2004 09:54:59 -0000
***************
*** 64,70 ****
      where room_id = p_room_id;
  
      if (v_approval_needed_p = ''f'') then
!       perform rr_reservations__approve (p_reservation_id, ''Open Policy Room'',p_creation_user);
      end if;
  
      perform acs_object__update_last_modified(p_reservation_id, p_creation_user, p_creation_ip);
--- 64,70 ----
      where room_id = p_room_id;
  
      if (v_approval_needed_p = ''f'') then
!       perform rr_reservations__approve (p_reservation_id, ''Open Policy Room'',p_creation_user, p_creation_ip);
      end if;
  
      perform acs_object__update_last_modified(p_reservation_id, p_creation_user, p_creation_ip);

Thanks again

Collapse
Posted by Caroline Meeks on
Thanks Bruno,

I'm sure there will be more. This package is not in production anywhere in its current state.  Please put bugs and patches into bugtracker.

Thanks
Caroline