payment_order - sort by id desc to display newest on top of the list

Bug #772509 reported by Ferdinand
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Low
OpenERP R&D Addons Team 3

Bug Description

see patch

Related branches

Revision history for this message
Ferdinand (office-chricar) wrote :
Revision history for this message
Amit Parik (amit-parik) wrote :

Hello DR Ferdinand,

I have checked your issue But I am not get your point exactly where is faced the problem.

I have checked it there is no 'desc' set in to bank statement and cash register form So why you need this in account payment object

And company_id is related field in account payment object with payment mode so no need to add this field in to _default it is automatically set through the payment mode.

And you also don't describe why you add company_id field in get_wizard.

So would you please elaborate with more on this.

Thanks and waiting for your reply!

Changed in openobject-addons:
status: New → Incomplete
Revision history for this message
Ferdinand (office-chricar) wrote :

1) order_by: id desc
users want to see the newest payment orders on top (currently at the end of the list) to avoid scrollig.

2) company_id
in mulitcompany setup users have beein seeing payment orders from other companies in the same database - not only from other companies defined in res_user etc.

I strongly believe that the rule filter does not select payment orders which belong to a company defined in payment mode.

nevertheless it could be sufficient to create a related field. I didn't try

Revision history for this message
Amit Parik (amit-parik) wrote :

Hello,

This is not a bug it's improvement So better to team will decided on it.

Thanks.

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
status: Incomplete → Triaged
importance: Undecided → Low
Changed in openobject-addons:
status: Triaged → Confirmed
status: Confirmed → In Progress
Revision history for this message
Ashvin Rathod (OpenERP) (ara-tinyerp) wrote :

Hello Ferdinand,

I have tested as your description and get the same problem.
And your attached patch is working fine for this but no need to add company_id because this field is already there.
It will be merged soon in trunk addons.

Revision ID: <email address hidden>
Revision No: 4685

Thanks for participation !

Changed in openobject-addons:
status: In Progress → Fix Committed
Changed in openobject-addons:
status: Fix Committed → Fix Released
milestone: none → 6.1
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.