payment : error into menu + modification of payment allowed even if confimed

Bug #318712 reported by Luc Maurer @ Camptocamp
2
Affects Status Importance Assigned to Milestone
OpenERP
Fix Released
Undecided
Jay Vora (Serpent Consulting Services)

Bug Description

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

 affects openerp

there is a mistake into the menu structure of the payment module => I
think that it should be like invoices =>

first level = all payment order
second level = new payment order / draft / to validate

- -------

it is also possible to modify (add or delete or modify lines) even if
the payment order is confirmed => if the payment order is confimre
(statut = done) then it should not be possible to change it

Luc Maurer @ camptocamp

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFJdF9jhgG7fdis3UURAqlCAKC43TJey4L98aiXyPKlQBYpfll4MgCguIRl
iyuKDmMKwyxvWGdkbDZjfLY=
=+tTj
-----END PGP SIGNATURE-----

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

That was not a bug in actual. Only non-important fields were allowed for changes.
Thanks for the menu.
Fixed in rev 2101(Openobject-Addons)

Changed in openerp:
assignee: nobody → jvo-openerp
status: New → Fix Released
Revision history for this message
Luc Maurer @ Camptocamp (lmaurer-c2c) wrote :

Is it a joke :-)

Do you think that be able to add a new line is "non-important" ???

=> if you press the button "select invoice to pay", you ca add new lines => for my point of view, it is very important !
=> you can change the date, the prefered date, and ALL field..... => in fact, you can change all fields of the payment ordre object which should not be possible.

Changed in openerp:
status: Fix Released → Confirmed
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

No Luc, i didn't mean by a joke.
Please,Don't take it either way.
Fixed in rev 2103.
Thanks again.

Changed in openerp:
status: Confirmed → Fix Released
Revision history for this message
Luc Maurer @ Camptocamp (lmaurer-c2c) wrote :

Hello, no problem :-) many thanks for the patch

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.