installation module hotel: Error ! You can not create recursive Menu.

Bug #745560 reported by Alessio Gerace
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Low
Amit Parik

Bug Description

delete the character "/" Inside the name attribute of each of these menuitem tag xml:
hotel_view.xml
hotel_wizard.xm Hotel in module
after which it will present a new error in the construction of the folio of the hotel reservation form:

hotel_reservation.py line 127

need to change it like this:

str = 'insert into hotel_folio_reservation_rel (order_id, invoice_id) values ​​(% d,% d)'% (reservation.id, folio)
cr.execute (str)

Changed in openobject-addons:
assignee: nobody → OpenERP Committers (openerp-commiter)
Revision history for this message
OpenBMS JSC (openbmsjsc) wrote :

these have been fixed in https://code.launchpad.net/~phong-nguyen-thanh/openobject-addons/addons-extra-trunk-hotel, but still wait for merging into the official repository.

Can someone in the Extra committer team take a look and feedback?

Revision history for this message
Martin Collins (mkc-steadfast) wrote :

I just had a look. Seems fine as far as it goes but I'm not a committer.

The same type of fixes are also required in board_reservation, board_restaurant, and board_frontdesk.

Why is this stuff split into seven modules? It should be two at most: hotel and hotel_restaurant.

Language quibble: Unless "folio" is a special hotelling word it should be changed to "invoice" or "sales order" or whatever it is supposed to mean.
Also "confirm" should probably be "confirmed" and "done" should be "completed".

Changed in openobject-addons:
assignee: OpenERP Committers (openerp-commiter) → nobody
Changed in openobject-addons:
assignee: nobody → Amit Parik (OpenERP) (amp-openerp)
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Phong,

I have tested your patch and it is working fine.
So I have applied it to lp:openobject-addons/extra-6.0 with revision 5434 <email address hidden> for solving the above bug.

Thanks for the contribution.

Changed in openobject-addons:
status: Confirmed → Fix Released
Revision history for this message
OpenBMS JSC (openbmsjsc) wrote :

Hi Amit,

Could you please look at this https://code.launchpad.net/~phong-nguyen-thanh/openobject-addons/addons-extra-trunk-hotel, and change the status for me? It's been pending for too long.

Thanks and best regards,

Phong.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.