[Openerp-dev-web] [Bug 671608] Re: customer payment doesn't work in web version
** Project changed: openobject-addons = openobject-client-web ** Changed in: openobject-client-web Assignee: (unassigned) = OpenERP SA's Web Client RD (openerp-dev-web) -- customer payment doesn't work in web version https://bugs.launchpad.net/bugs/671608 You received this bug notification because you are a member of OpenERP SA's Web Client RD, which is a bug assignee. Status in OpenObject Web Client: New Bug description: The customer payment screen doesn't work with the web version. When I enter the customer name, nothing gets filled into the screen. However, if I use the GTK version, when I enter the customer name, the outstanding invoices are listed. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision into lp:openobject-addons has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision/+merge/37594 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision/+merge/37594 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision into lp:openobject-addons has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision/+merge/37594 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision/+merge/37594 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-dev-addons_decimal_precision. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/jam-dev-addons2 into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/jam-dev-addons2 into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/jam-dev-addons2/+merge/44027 -- https://code.launchpad.net/~openerp-dev/openobject-addons/jam-dev-addons2/+merge/44027 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/jam-dev-addons2. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/xmo-reformat-cleanup-partial-pickings into lp:openobject-addons
as discussed with xmo, i'm rejcting this merge prop becazuse it was already done by the dev2 team. -- https://code.launchpad.net/~openerp-dev/openobject-addons/xmo-reformat-cleanup-partial-pickings/+merge/46147 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/xmo-reformat-cleanup-partial-pickings. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/xmo-reformat-cleanup-partial-pickings into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/xmo-reformat-cleanup-partial-pickings into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/xmo-reformat-cleanup-partial-pickings/+merge/46147 -- https://code.launchpad.net/~openerp-dev/openobject-addons/xmo-reformat-cleanup-partial-pickings/+merge/46147 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/xmo-reformat-cleanup-partial-pickings. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~vmt-openerp/openobject-addons/vmt-traceability-action into lp:openobject-addons
qdp (OpenERP) has proposed merging lp:~vmt-openerp/openobject-addons/vmt-traceability-action into lp:openobject-addons. Requested reviews: OpenERP RD Team (openerp-dev) Related bugs: #695024 Upstream / Downstream traceability does not work https://bugs.launchpad.net/bugs/695024 For more details, see: https://code.launchpad.net/~vmt-openerp/openobject-addons/vmt-traceability-action/+merge/46524 -- https://code.launchpad.net/~vmt-openerp/openobject-addons/vmt-traceability-action/+merge/46524 Your team OpenERP RD Team is requested to review the proposed merge of lp:~vmt-openerp/openobject-addons/vmt-traceability-action into lp:openobject-addons. === modified file 'stock/stock_view.xml' --- stock/stock_view.xml 2011-01-17 16:59:44 + +++ stock/stock_view.xml 2011-01-17 19:49:45 + @@ -466,6 +466,40 @@ /field /record +record id=action3 model=ir.actions.act_window +field name=nameDownstream traceability/field +field name=typeir.actions.act_window/field +field name=res_modelstock.move/field +field name=domain[('id','in',active_ids)]/field +field name=view_typetree/field +field eval=stock_move_tree2 name=view_id/ +/record + +record id=ir_move_traceability_upstream model=ir.values +field name=key2tree_but_action/field +field name=modelstock.move/field +field name=nameDownstream traceability/field +field eval='ir.actions.act_window,'+str(action3) name=value/ +field eval=True name=object/ +/record + +record id=action5 model=ir.actions.act_window +field name=nameUpstream traceability/field +field name=typeir.actions.act_window/field +field name=res_modelstock.move/field +field name=domain[('id','in',active_ids)]/field +field name=view_typetree/field +field eval=stock_move_tree name=view_id/ +/record + +record id=ir_move_traceability_downstream model=ir.values +field name=key2tree_but_action/field +field name=modelstock.move/field +field name=nameUpstream traceability/field +field eval='ir.actions.act_window,'+str(action5) name=value/ +field eval=True name=object/ +/record + record id=view_location_form model=ir.ui.view field name=namestock.location.form/field field name=modelstock.location/field ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-720629-ara into lp:openobject-addons
i improved: 1) _('Supplier Invoices') for translability 2) -'view_mode': 'form', +'view_mode': 'form,tree', to allwo to switch in tree view after processing that invoice -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-720629-ara/+merge/50335 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-720629-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-728943-mra into lp:openobject-addons
Review: Disapprove for me, the bug isn't a bug but a whishlist, and currently this fix isn't good because it will create bigger problems while hole in the numbering cause each time we will cancel a voucher we will lsot the number. So i'm rejecting that merge prop -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728943-mra/+merge/52653 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-728943-mra. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-728943-mra into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-728943-mra into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728943-mra/+merge/52653 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728943-mra/+merge/52653 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-728943-mra. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-724073-ron into lp:openobject-addons
Review: Disapprove in accounting, we should keep in mind that a user can see the objects belonging to *its company *a child of its company but never to a company parent than its company. So you should remove that part of the domain: ('company_id.child_ids','child_of',[user.company_id.id]) -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724073-ron/+merge/51257 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-724073-ron. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-724059-rha into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-724059-rha into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724059-rha/+merge/51493 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724059-rha/+merge/51493 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-724059-rha. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-724059-rha into lp:openobject-addons
Review: Disapprove rejected: the rules shouldn't allow the users to see objects belonging to parent company -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724059-rha/+merge/51493 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-724059-rha. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-718722-rha into lp:openobject-addons
Review: Needs Information i don't see the point with the linked bug. Could someone explain? -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-718722-rha/+merge/52161 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-718722-rha. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-725908-ksa into lp:openobject-addons
Review: Abstain need to double-check -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-725908-ksa/+merge/52519 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-725908-ksa. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-730313-aag into lp:openobject-addons
Review: Disapprove Should have been set as wishlist... i do not share the view of Ferdinand because i've a doubt on the utility of such improvement. Moreover, if i merge the branch as it, in less than 2 weeks we will have another bug report for 'subtotal in form view of SO line is only updated when saving but it should be done on the fly'. So for now, let's set this bug as won't fix, and also i'm rejecting this merge proposal -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-730313-aag/+merge/52373 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-730313-aag. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-730313-aag into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-730313-aag into lp:openobject-addons has been updated. Status: Approved = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-730313-aag/+merge/52373 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-730313-aag/+merge/52373 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-730313-aag. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-728382-aag into lp:openobject-addons
already fixed in revno: 4493 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728382-aag/+merge/52514 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-728382-aag. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-728382-aag into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-728382-aag into lp:openobject-addons has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728382-aag/+merge/52514 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728382-aag/+merge/52514 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-728382-aag. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-723713-ara into lp:openobject-addons
fix already there as the bug was a duplicated one of https://bugs.launchpad.net/openobject-addons/+bug/713177. I'm closing this merge prop for the sake of clarity -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-723713-ara/+merge/51515 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-723713-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-723713-ara into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-723713-ara into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-723713-ara/+merge/51515 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-723713-ara/+merge/51515 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-723713-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-715470-ara into lp:openobject-addons
patch was working, but by luck only. Actually, the way to use the store = {} feature wasn't good but the orm was bugged and the priorities of functions wasn't working at all. So i just commited the right way to fix the issue in the addons, a patch is under testing server side and if it succeeds, i will merge without delay that branch. -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-715470-ara/+merge/51288 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-715470-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-724201-ara into lp:openobject-addons
Review: Disapprove rejected, for me it is not really a bug and adding those fields in the list view is against the principle of usability/ease of use: users that will encode their timesheets don't have a clue of what are those fields! they should be invisible but pre-configured with default data. What you will do is: raise a warning on the filling of employee in the timesheet.sheet form if those values aren't configured for him. thanks, -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724201-ara/+merge/52406 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-724201-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-724201-ara into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-724201-ara into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724201-ara/+merge/52406 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724201-ara/+merge/52406 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-724201-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-728858-ara into lp:openobject-addons
Review: Disapprove rejected, for the reason depicted on the bug report -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728858-ara/+merge/52806 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-728858-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-728858-ara into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-728858-ara into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728858-ara/+merge/52806 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728858-ara/+merge/52806 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-728858-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-728417-ara into lp:openobject-addons
Review: Needs Fixing seems really a weird fix... you should try by doing res = {} at the beginning of the function and doing -return res +return res at the end (decrease the indentation of last line of this function) -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728417-ara/+merge/52660 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-728417-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-729602-ara into lp:openobject-addons
i agree with Bogdan Stanciu. Please update your merge proposal accordingly. Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-729602-ara/+merge/52684 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-729602-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-728417-ara into lp:openobject-addons
Review: Needs Fixing it's not working. I guess my suggestion wasn't good. Did you test at your side? -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728417-ara/+merge/52660 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-728417-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-728417-ara into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-728417-ara into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728417-ara/+merge/52660 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-728417-ara/+merge/52660 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-728417-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-730562-xrg into lp:openobject-addons
as discussed with xrg, i'm rejecting this merge proposal that have been done only for testing purpose. -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-730562-xrg/+merge/52419 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-730562-xrg. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-730562-xrg into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-730562-xrg into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-730562-xrg/+merge/52419 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-730562-xrg/+merge/52419 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-730562-xrg. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-configuration-rework into lp:openobject-addons
i'm rejecting for the sake of clarity in my merge proposal list although i didn't found any mistake (yet :p), but i have several other improvements to discuss with you, Rucha. In particular, our specifications weren't good when saying: on the module list, one click = one installed module. Actually, what is wanted is: one click = one module scheduled to install (as before) + run the wiazrd 'apply scheduled installation' (just simulate a click on this wizard). So, it will allow us to keep a validation step (the old one) before installing a module (latest improvements were quiet violent ^^) Also, before merging into trunk, i'd like to have the list of wizards ready (specified on the pad). thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-configuration-rework/+merge/53208 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-configuration-rework into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-configuration-rework into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-configuration-rework/+merge/53208 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-configuration-rework/+merge/53208 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-740353-ara into lp:openobject-addons
i commited in that branch how to properly use the set type. Please have a look Ashvin. By the way, why don't i see Mustufa as reviewer? Anyway, i merged this branch with my fix -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-740353-ara/+merge/54669 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-740353-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/qdp-l10n_us into lp:~openerp-dev/openobject-addons/dsh-l10n_us
qdp (OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/qdp-l10n_us into lp:~openerp-dev/openobject-addons/dsh-l10n_us. Requested reviews: dsh (Open ERP) (dsh-openerp) For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/qdp-l10n_us/+merge/55141 merge some of the improvements of David Mitchell from NovaPoint + redesign form view of check Remaining things to do: - set defaut and demo data in order to have a check sequence given on the check journal - (to be discussed) set check sequence field as mandatory on account.journal if allow_check is True - fill the check number of voucher when posting/validating the check - check rml improvements made by NovaPoint and merge the interesting things Thanks, Quentin -- https://code.launchpad.net/~openerp-dev/openobject-addons/qdp-l10n_us/+merge/55141 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/dsh-l10n_us. === modified file 'account_check_writing/__openerp__.py' --- account_check_writing/__openerp__.py 2011-01-18 01:33:32 + +++ account_check_writing/__openerp__.py 2011-03-28 13:09:33 + @@ -21,7 +21,7 @@ { name : Check writing, version : 1.1, -author : OpenERP SA, +author : OpenERP SA, NovaPoint Group, category: Generic Modules/Accounting, description: Module for the Check writing and check printing @@ -30,8 +30,10 @@ 'init_xml': [], depends : [ account_voucher, +account_accountant, ], 'update_xml': [ +'check_sequence.xml', 'account_check_writing_report.xml', 'account_view.xml', 'account_voucher_view.xml', === modified file 'account_check_writing/account.py' --- account_check_writing/account.py 2010-12-23 18:25:45 + +++ account_check_writing/account.py 2011-03-28 13:09:33 + @@ -26,6 +26,8 @@ _columns = { 'allow_check_writing': fields.boolean('Allow Check writing', help='Fill this if the journal is to be used for writing checks.'), +'use_preprint_check': fields.boolean('Use Preprinted Check'), +'check_sequence': fields.many2one('ir.sequence', 'Check Sequence', help=This field contains the information related to the numbering of the check number.), } account_journal() === modified file 'account_check_writing/account_view.xml' --- account_check_writing/account_view.xml 2011-01-18 01:33:32 + +++ account_check_writing/account_view.xml 2011-03-28 13:09:33 + @@ -10,11 +10,16 @@ field name=nameaccount.journal.form/field field name=modelaccount.journal/field field name=typeform/field - field name=inherit_id ref=account.view_account_journal_form / +field name=inherit_id ref=account.view_account_journal_form / field name=arch type=xml -field name=type on_change=onchange_type(type, currency)/ +field name=type on_change=onchange_type(type, currency) / field name=entry_posted position=after -field name=allow_check_writing attrs={'readonly':[('type','!=','bank')]}/ +field name=allow_check_writing attrs={'readonly':[('type','!=','bank')]} / +field name=use_preprint_check attrs={'readonly':[('type','!=','bank')]} / +/field +field name=sequence_id required=0 position=after +field name=check_sequence +attrs={'readonly':[('allow_check_writing','=',False)]} / /field /field /record === modified file 'account_check_writing/account_voucher.py' --- account_check_writing/account_voucher.py 2011-02-04 01:19:58 + +++ account_check_writing/account_voucher.py 2011-03-28 13:09:33 + @@ -38,6 +38,7 @@ _columns = { 'amount_in_word' : fields.char(Amount in word , size=128, readonly=True, states={'draft':[('readonly',False)]}), 'allow_check' : fields.boolean('Allow Check Writing'), # attrs does not support '.' format and fields.relates get the value when v save the record +'chk_seq' : fields.char(Check Number , size=64, readonly=True), } @@ -82,7 +83,6 @@ if journal_id: allow_check_writing = self.pool.get('account.journal').browse(cr, uid, journal_id).allow_check_writing default['value'].update({'allow_check':allow_check_writing}) - return default def print_check(self, cr, uid, ids, context=None): === modified file 'account_check_writing/account_voucher_view.xml' --- account_check_writing/account_voucher_view.xml 2011-02-24 17:23:49 + +++ account_check_writing/account_voucher_view.xml 2011-03-28 13:09:33 + @@ -2,7 +2,7 @@ openerp data -record model=ir.ui.view id=view_vendor_payment_check_form +!--record model=ir.ui.view id=view_vendor_payment_check_form
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-724820-ksa into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-724820-ksa into lp:openobject-addons has been updated. Status: Merged = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724820-ksa/+merge/54695 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724820-ksa/+merge/54695 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-724820-ksa. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-724820-ksa into lp:openobject-addons
bugfix was invalid -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724820-ksa/+merge/54695 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-724820-ksa. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-724820-ksa into lp:openobject-addons
Review: Disapprove -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-724820-ksa/+merge/54695 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-724820-ksa. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-739181-rpa into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-739181-rpa into lp:openobject-addons has been updated. Status: Merged = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-739181-rpa/+merge/54812 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-739181-rpa/+merge/54812 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-739181-rpa. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-739181-rpa into lp:openobject-addons
Review: Disapprove actually, the bug was invalid. -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-739181-rpa/+merge/54812 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-739181-rpa. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-700928-ksa into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-700928-ksa into lp:openobject-addons has been updated. Status: Merged = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-700928-ksa/+merge/54691 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-700928-ksa/+merge/54691 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-700928-ksa. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-700928-ksa into lp:openobject-addons
Review: Disapprove fix reverted, ass the bug was invalid. -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-700928-ksa/+merge/54691 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-700928-ksa. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-payroll-demo-data-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll
Review: Approve after a fast review it seems correct -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-demo-data-mtr/+merge/57161 Your team OpenERP RD Team is requested to review the proposed merge of lp:~openerp-dev/openobject-addons/trunk-payroll-demo-data-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp into lp:openobject-addons
qdp (OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp into lp:openobject-addons. Requested reviews: Olivier Dony (OpenERP) (odo) Related bugs: Bug #744789 in OpenERP Addons: [V6] Third party ledger : Initial Balance https://bugs.launchpad.net/openobject-addons/+bug/744789 For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-744789-qdp/+merge/58334 in the query_get() of account.move.line, if we don't give any other criteria in the context than the fiscalyear when computing an initial balance then it simply means that the entries returned will be exactly the same as without initial balance, which lead to count the entries twice... this hack is a little nasty (not so say a lot nasty), but i can't think to something else. -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-744789-qdp/+merge/58334 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp. === modified file 'account/account_move_line.py' --- account/account_move_line.py 2011-03-25 15:38:54 + +++ account/account_move_line.py 2011-04-19 15:46:29 + @@ -93,6 +93,10 @@ else: query = obj+.state 'draft' AND +obj+.period_id IN (SELECT id FROM account_period WHERE fiscalyear_id IN (%s)) %s %s % (fiscalyear_clause, where_move_state, where_move_lines_by_date) +if initial_bal and not context.get('periods', False) and not where_move_lines_by_date: +#we didn't pass any filter in the context, and the initial balance can't be computed using only the fiscalyear otherwise entries will be summed twice +#so we have to invalidate this query +query += AND 1=2 if context.get('journal_ids', False): query += ' AND '+obj+'.journal_id IN (%s)' % ','.join(map(str, context['journal_ids'])) ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-746422-pso into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-746422-pso into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-746422-pso/+merge/55896 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-746422-pso/+merge/55896 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-746422-pso. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-739562-skh into lp:openobject-addons
Review: Approve don't forget to include the context in the browse next time ;-) -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-739562-skh/+merge/56343 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-739562-skh. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh into lp:openobject-addons
Review: Disapprove it's not the proper way to fix. The bug purpose is about: we shouldn't see computed fields when going in list view of leave types in configuration. So we should define a new list view without the fields: field name=max_leaves/ field name=leaves_taken/ field name=remaining_leaves/ with low priority and assign this view to the menuitem in configuration. thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-751299-skh/+merge/56516 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-751299-skh/+merge/56516 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-751299-skh/+merge/56516 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-755609-pso into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-755609-pso into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-755609-pso/+merge/57140 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-755609-pso/+merge/57140 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-755609-pso. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-716300-ara into lp:openobject-addons
Review: Needs Information what's the link with the bug report? -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-716300-ara/+merge/57670 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-716300-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-746544-ara into lp:openobject-addons
Review: Needs Fixing not a good idea, set ondelete=cascade on the field move_id of aaccount.analytic.line please -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-746544-ara/+merge/55924 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-746544-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-rework-21Apr-server-aag into lp:~openerp-dev/openobject-server/trunk-configuration-rework
what is that??? -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-rework-21Apr-server-aag/+merge/59032 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-rework-21Apr-addons-aag into lp:~openerp-dev/openobject-addons/trunk-configuration-rework
wtf? can someone exaplin me what this have to do with config-rework? -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-rework-21Apr-addons-aag/+merge/59019 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp into lp:openobject-addons has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-744789-qdp/+merge/58334 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-744789-qdp/+merge/58334 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-749958-xrg into lp:openobject-addons
Review: Needs Information i'm waiting the validation by the bug reporter that your patch is doing the trick, Panos. :-) -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-749958-xrg/+merge/58487 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-749958-xrg. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-722832-pso into lp:openobject-addons
Review: Needs Information having just a fast look, i'm wondering about those lines: 6 def sum_cr(self): 7 if self.res_bl['type'] == _('Net Loss'): 8 - self.result_sum_cr += self.res_bl['balance'] 9 + self.result_sum_dr += self.res_bl['balance'] we are in sum_cr method and we use result_sum_dr ? are those lines correcting the bug or it's a hunk that wouldn't have landed on the branch? -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-722832-pso/+merge/56115 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-722832-pso. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-719063-pso into lp:openobject-addons
Review: Needs Fixing patch for payroll is wrong: HR / Officer should be able to see all the payslips of him or of employess for which he's the manager. But this will be addressed in the RD project related to hr_payroll, so you can simply remove that hunk. second path for timesheet is correct, but i'm not sure that there isn't any other patch needed. More investigation is needed, in the meanwhile you can already remove the 1st patch Priyesh. Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-719063-pso/+merge/55337 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-719063-pso. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-configuration-rework into lp:openobject-server
The proposal to merge lp:~openerp-dev/openobject-server/trunk-configuration-rework into lp:openobject-server has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-server/trunk-configuration-rework/+merge/53209 -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-configuration-rework/+merge/53209 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-configuration-rework into lp:openobject-server
it's not yet ready and even not fully tested, so i reject it for the sake of clarity ^^ -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-configuration-rework/+merge/53209 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-commiter/openobject-server/trunk-fix-uninstallation-sus into lp:~openerp-dev/openobject-server/trunk-configuration-rework
Review: Disapprove i think this branch isn't stacked on the good one (should be the main project branch, not the trunk-addons) and thus it's giving here a wrong diff. Please Sudhir resubmit a correct branch so that we can see the diff directly. Thanks -- https://code.launchpad.net/~openerp-commiter/openobject-server/trunk-fix-uninstallation-sus/+merge/52526 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-commiter/openobject-server/trunk-fix-uninstallation-sus into lp:~openerp-dev/openobject-server/trunk-configuration-rework
The proposal to merge lp:~openerp-commiter/openobject-server/trunk-fix-uninstallation-sus into lp:~openerp-dev/openobject-server/trunk-configuration-rework has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-commiter/openobject-server/trunk-fix-uninstallation-sus/+merge/52526 -- https://code.launchpad.net/~openerp-commiter/openobject-server/trunk-fix-uninstallation-sus/+merge/52526 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-rework-21Apr-server-aag into lp:~openerp-dev/openobject-server/trunk-configuration-rework
Review: Needs Fixing ha yes, i didn't though to you could do it with simply a col=2. Great! but not exatly what they want as usability :-s, as they want something i could summary like this: *The list of application is col=4 and each application name has a colspan of 2. *under each installed application, a separator of colspan=4 and 2 columns of the available modules for this application (each application with a colspan of 2 also) so i just sent you a drawing of what they want. Thanks -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-rework-21Apr-server-aag/+merge/59032 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-rework-21Apr-addons-aag into lp:~openerp-dev/openobject-addons/trunk-configuration-rework
Review: Needs Fixing cfr my comment in same proposal for server side -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-rework-21Apr-addons-aag/+merge/59165 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp into lp:openobject-addons
qdp (OpenERP) has proposed merging lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp into lp:openobject-addons. Requested reviews: Purnendu Singh (OpenERP) (psi-tinyerp) Olivier Dony (OpenERP) (odo) Related bugs: Bug #744789 in OpenERP Addons: [V6] Third party ledger : Initial Balance https://bugs.launchpad.net/openobject-addons/+bug/744789 For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-744789-qdp/+merge/58334 in the query_get() of account.move.line, if we don't give any other criteria in the context than the fiscalyear when computing an initial balance then it simply means that the entries returned will be exactly the same as without initial balance, which lead to count the entries twice... this hack is a little nasty (not so say a lot nasty), but i can't think to something else. -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-744789-qdp/+merge/58334 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp. === modified file 'account/account_move_line.py' --- account/account_move_line.py 2011-04-25 06:11:14 + +++ account/account_move_line.py 2011-04-28 07:25:56 + @@ -82,17 +82,25 @@ period_ids = fiscalperiod_obj.search(cr, uid, [('id', 'in', context['periods'])], order='date_start', limit=1) if period_ids and period_ids[0]: first_period = fiscalperiod_obj.browse(cr, uid, period_ids[0], context=context) +where_move_lines_by_date = AND +obj+.move_id IN (SELECT id FROM account_move WHERE date ' +first_period.date_start+') # Find the old periods where date start of those periods less then Start period periods = fiscalperiod_obj.search(cr, uid, [('date_start', '', first_period.date_start)]) periods = ','.join([str(x) for x in periods]) if periods: query = obj+.state 'draft' AND +obj+.period_id IN (SELECT id FROM account_period WHERE fiscalyear_id IN (%s) AND id IN (%s)) %s %s % (fiscalyear_clause, periods, where_move_state, where_move_lines_by_date) +else: +query = obj+.state 'draft' AND +obj+.period_id IN (SELECT id FROM account_period WHERE fiscalyear_id IN (%s)) %s %s % (fiscalyear_clause, where_move_state, where_move_lines_by_date) else: ids = ','.join([str(x) for x in context['periods']]) query = obj+.state 'draft' AND +obj+.period_id IN (SELECT id FROM account_period WHERE fiscalyear_id IN (%s) AND id IN (%s)) %s %s % (fiscalyear_clause, ids, where_move_state, where_move_lines_by_date) else: query = obj+.state 'draft' AND +obj+.period_id IN (SELECT id FROM account_period WHERE fiscalyear_id IN (%s)) %s %s % (fiscalyear_clause, where_move_state, where_move_lines_by_date) +if initial_bal and not context.get('periods', False) and not where_move_lines_by_date: +#we didn't pass any filter in the context, and the initial balance can't be computed using only the fiscalyear otherwise entries will be summed twice +#so we have to invalidate this query +raise osv.except_osv(_('Warning !'),_(You haven't supplied enough argument to compute the initial balance)) + if context.get('journal_ids', False): query += ' AND '+obj+'.journal_id IN (%s)' % ','.join(map(str, context['journal_ids'])) === modified file 'account/report/account_partner_ledger.py' --- account/report/account_partner_ledger.py 2011-01-14 00:11:01 + +++ account/report/account_partner_ledger.py 2011-04-28 07:25:56 + @@ -58,10 +58,11 @@ obj_partner = self.pool.get('res.partner') self.query = obj_move._query_get(self.cr, self.uid, obj='l', context=data['form'].get('used_context', {})) ctx2 = data['form'].get('used_context',{}).copy() -ctx2.update({'initial_bal': True}) +self.initial_balance = data['form'].get('initial_balance', True) +if self.initial_balance: +ctx2.update({'initial_bal': True}) self.init_query = obj_move._query_get(self.cr, self.uid, obj='l', context=ctx2) self.reconcil = data['form'].get('reconcil', True) -self.initial_balance = data['form'].get('initial_balance', True) self.result_selection = data['form'].get('result_selection', 'customer') self.amount_currency = data['form'].get('amount_currency', False) self.target_move = data['form'].get('target_move', 'all') === modified file 'account/wizard/account_report_partner_ledger.py' --- account/wizard/account_report_partner_ledger.py 2011-04-15 10:09:01 + +++ account/wizard/account_report_partner_ledger.py 2011-04-28 07:25:56 + @@ -39,9 +39,22 @@ } _defaults = { 'reconcil': True, - 'initial_balance': True
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-payroll-tooltips-psi into lp:~openerp-dev/openobject-addons/trunk-payroll
Review: Needs Fixing it's not really meaningfull i think. i'd prefer something like: The code of salary rules can be used as reference in computation of other rules. In that case, it is case sensitive. -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-tooltips-psi/+merge/59048 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-payroll-wizard_payslip_employee_category-wise-psi into lp:~openerp-dev/openobject-addons/trunk-payroll
Review: Needs Fixing few points to correct: 1) [2011-04-28 15:14:35,238][payroll_avr_28] WARNING:init:object hr.payslip.groups (hr.payslip.groups) has no access rules! 2) when selecting a category, i should search for employees in that category but also for employees in a category child of this category 3) wwhen computing the payslip of employees, you must call the onchange_employee_id method otherwise the input lines for him won't be created and when openerp is trying to compute them it's raising error like invalid quantity for meal voucher rule (because there isn't an input line with code = WORK100) 4) add the m2o on the payslip form view too, in the tab other information thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-wizard_payslip_employee_category-wise-psi/+merge/58953 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-temporal-db-rpa into lp:~openerp-dev/openobject-server/trunk-temporal-db
Review: Approve FYI: append() is twice as fast as list concatenation :-) Well done! -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-temporal-db-rpa/+merge/59450 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-temporal-db. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-718609-ron into lp:openobject-addons
similar patch already landed in trunk. SO i reject for the sake of clarity. thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-718609-ron/+merge/59325 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-718609-ron. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-718609-ron into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-718609-ron into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-718609-ron/+merge/59325 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-718609-ron/+merge/59325 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-718609-ron. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-768430-ron into lp:openobject-addons
Review: Needs Information i cannot reproduce this bug. Am i missing something? -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-768430-ron/+merge/59170 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-768430-ron. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-751222-ron into lp:openobject-addons
Review: Needs Information bug has been passed back into confirmed status. Please check it -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-751222-ron/+merge/57112 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-751222-ron. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh into lp:openobject-addons
i fixed this: priority of new view set to 35 in order to low its priority (should only be seen when in configuration\holidays\leave type it's merged now -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-751299-skh/+merge/58634 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh into lp:openobject-addons has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-751299-skh/+merge/58634 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-751299-skh/+merge/58634 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-751299-skh. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-760412-pso into lp:openobject-addons
Review: Needs Fixing you cannot use voucher.account_id as this account is already used for the counter part (its meaningless to use the same account for both the debit the credit of the same operation). Instead, use the default accounts defined on the journal accordingly if the move will be a credit or a debit (default_debit_account or default_credit_account). Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-760412-pso/+merge/58228 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-760412-pso. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-759444-skh into lp:openobject-addons
next time: don't forget to encapsulate your strings with _(my string) in order to enable the translation for it. I fixed and merged it. Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-759444-skh/+merge/59458 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-759444-skh. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-717198-pso into lp:openobject-addons
Review: Needs Fixing for this bug, i'm not sure this is the god way to solve the problem. I would have prefer to have a new fields.float to save the percentage used t compute the amount in company currency, then in the _get_amount function we can just call super() and apply this percentage to the result to get the final result. It seems more interesting because: 1)it reuses the code of super and do not compute again by its own the amount in currency (which is error prone, btw there is an error in this method: you should have passed a date in the context in compute() in order to use the rate at the good date... so maybe there are other errors) 2)it stores a new information that seems very valuable for any future needs Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-717198-pso/+merge/57623 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-717198-pso. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-753835-ara into lp:openobject-addons
don't you find it's really confusing to use a variable named stock_dest_id for the source or stock move?? i made my own fix for that (and it's already merged so this branch is rejected although it contains a working fix), which is similar but with better variable names. Think about that next time. Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-753835-ara/+merge/56884 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-753835-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-753835-ara into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-753835-ara into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-753835-ara/+merge/56884 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-753835-ara/+merge/56884 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-753835-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp into lp:openobject-addons
Review: Needs Fixing i checked and for me it's solving all the known problems of the partner ledger. The only problem with this merge proposal is that it is creating error in yaml test because of the new constraint we've added. [2011-05-04 14:46:25,524][trunk_mai_03] INFO:init:module account: loading test/account_report.yml File /home/qdp/tinydev/addons/trunk/account/account_move_line.py, line 102, in _query_get raise osv.except_osv(_('Warning !'),_(You haven't supplied enough argument to compute the initial balance)) Please check it, thanks, Quentin -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-744789-qdp/+merge/58334 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp into lp:openobject-addons
while testing i found out that there was another bug: if i select the periods from 01/2011 - 05/2011 it was displaying also the opening entries though it should have been added in the initial balance. So i used this branch to fix that too. ;-) It will be merged into the trunk as soon as the errors in yaml will be fixed. Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-744789-qdp/+merge/58334 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp into lp:openobject-addons
a last thought: we should replace the boolean 'include reconciled entries' by a fourth option in the filter selection box Unreconciled Entries. So, the 4 options would be: filter by: No filter filter by: Date filter by: Periods filter by: Unreconciled Entries so that we will be sure that the initial balance is always complete. Can you do that do before i merge in the trunk? Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-744789-qdp/+merge/58334 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-744789-qdp. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-715470-ara into lp:openobject-addons
Review: Approve i aproove and set Olivier as reviewer (in order to clean my merge proposal list) and as we are waiting for his patch server side. -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-715470-ara/+merge/51288 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-715470-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-748178-pso into lp:openobject-addons
why did you removed average_price from view? -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-748178-pso/+merge/57449 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-748178-pso. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-748178-pso into lp:openobject-addons
Review: Approve i remember the problem, :-) it's cool like this -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-748178-pso/+merge/57449 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-748178-pso. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-payroll-wizard_payslip_employee_category-wise-psi into lp:~openerp-dev/openobject-addons/trunk-payroll
Review: Needs Fixing in hr_payroll/wizard/hr_payroll_payslips_by_employees.py, abuot lines 335-346: the behavior seems really weird to me: you call the onchange with [] as ids, then create the payslip, then create the input lines... you can do everything in one operation in you pass the good value for that field in the create. Check the example bellow (to create some order lines at the time of SO creation): vals.update({'order_line': [(0, 0, {'product_id': 1, 'product_uom_qty': 3, 'product_uom': 1, 'name': 'test', 'price_unit': 100, 'type': 'make_to_stock'}), (0, 0, {'product_id': 1, 'product_uom_qty': 3, 'product_uom': 1, 'name': 'test', 'price_unit': 100, 'type': 'make_to_stock'})]}) res = super(sale_order, self).create(cr, uid, vals, context=context) Also: when opening a payslip from the payslip.run object, i don't know why but the note and structure_id fields are readonly although they aren't if i open the same payslip from the payslip menu. Something new: we need also to keep track of the state of payslip.run: [(draft, Draft), (close, Close)] and *the button generate payslip is only visible if the run is draft *on a payslip, we can only select a payslip.run in draft Thanks, Quentin -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-wizard_payslip_employee_category-wise-psi/+merge/58953 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-payroll-payslip-on-payslip-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll
Review: Needs Fixing no, it needs fixing: we need to take into account also if the payslip is a credite note or not: in that case it should subtract pl.total so i didn't merged it, and thus you will be very nice to include in your next try the change i proposed above. ;-) Thanks, Quentin -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-payslip-on-payslip-mtr/+merge/60055 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-payroll-rename_salary_head-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll
Review: Needs Fixing i just realized: we should take care to rename it into salary rule category instead of salary category :-s sorry for the double-work -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-rename_salary_head-mtr/+merge/60498 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-payroll-rename_salary_head-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-payroll-rename_salary_head-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll has been updated. Status: Merged = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-rename_salary_head-mtr/+merge/60498 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-rename_salary_head-mtr/+merge/60498 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-temporal-db-read-search-unlink-ksa into lp:~openerp-dev/openobject-server/trunk-temporal-db
Review: Needs Fixing few comments, just by reading the diff: line 21-22 of the diff are useless as context isn't used later in the code of the function. Can be removed for better readability. line 24: why this ids.reverse()? is it to treat the case where we have records A,B,C respectively in timeline, then change the temporal_date_from of C and set something between A and B (to get A,C,B)? in that case, i would have seen a search with order by temporal_date_from.. I need information here. line 29: why this break? in get_next_id_in_timeline line35: unused line37: the params of the search seem wrong. I would have seen something like: reference_field = record.temporal_parent_id and record.temporal_parent_id.id or record.id ['|',('temporal_parent_id', '=', reference_field), ('id', '=', reference_field),('temporal_date_from', '=', record.temporal_date_to)] line 38: you return always the last value of search_id, it means that if you pass a list of ids like [1,2,3,4] or [4] it will have exactly the same behavior. code of write() can be improved line 77-78: can be removed i think if context.get('temporal_mode', True) and not vals.get('temporal_date_from'): line 69: i don't see the point to do ids.pop(), it will simply not work doing that. Just a continue would be enough code of browse: didn't checked yet, i had set this point as not needed in the pad because in theory it should already work currently as the browse is only using search and read... But keep a patch for that in case of... ;-) (but remove it from this merge prop please thanks, Quentin -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-temporal-db-read-search-unlink-ksa/+merge/60367 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-temporal-db. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-configuration-server-ksa into lp:~openerp-dev/openobject-server/trunk-configuration-rework
Review: Needs Information i don't know the tempfile python module. + tempfileid, tempfilename= tempfile.mkstemp('.rml', 'openerp_') will it work with any Operating System? -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-configuration-server-ksa/+merge/59346 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-payroll-account-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll
i merged another branch that i made but which is based on this one. so i'm rejecting this merge proposal to clear my pending merge to do list Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-account-mtr/+merge/60745 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-payroll-account-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-payroll-account-mtr into lp:~openerp-dev/openobject-addons/trunk-payroll has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-account-mtr/+merge/60745 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-payroll-account-mtr/+merge/60745 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-payroll. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-configuration-rework-company-imp-rpa into lp:~openerp-dev/openobject-server/trunk-configuration-rework
Review: Needs Fixing use multi='address' in every new functional fields that could be computed in the same time. -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-configuration-rework-company-imp-rpa/+merge/60784 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-configuration-rework-company-imp-rpa into lp:~openerp-dev/openobject-server/trunk-configuration-rework
Review: Needs Fixing another enhancement: use the first _default_ address instead of the first address. For that, use the address_get function defined on res.partner object. Thanks -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-configuration-rework-company-imp-rpa/+merge/60784 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-configuration-rework-company-view-imp-rpa into lp:~openerp-dev/openobject-addons/trunk-configuration-rework
it's merged, the res_model had to be changed as well thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-configuration-rework-company-view-imp-rpa/+merge/60783 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-configuration-rework. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-server/trunk-temporal-db-read-search-unlink-ksa into lp:~openerp-dev/openobject-server/trunk-temporal-db
Review: Needs Fixing 67 + # avoid creating history records when writing on a reocrd that is already an history record 68 + if record.temporal_parent_id: 69 + return False replace the return False statement with continue -- https://code.launchpad.net/~openerp-dev/openobject-server/trunk-temporal-db-read-search-unlink-ksa/+merge/60367 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-server/trunk-temporal-db. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-732419-ara into lp:openobject-addons
Review: Needs Fixing i do not agree: when you holiday request is refused, you want to know who (which manager) refused it. So we need to save the user that press the refuse button. Moreover, previously it was saving the refusing manager always in the same field whatever if it was already approved once. This need to be fixed as well. Finally, if the leave type needs only 1 step to be approved, we should display only one manager field, not both. Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-732419-ara/+merge/54314 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-732419-ara. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
Re: [Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-730885-skh into lp:openobject-addons
Review: Needs Information i should already have fixed that 2 weeks ago, so i'm rejecting this merge prop for the sake of clarity in LP. Please resubmit if the bug is confirmed again. Thanks -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-730885-skh/+merge/55318 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-730885-skh. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp
[Openerp-dev-web] [Merge] lp:~openerp-dev/openobject-addons/trunk-bug-730885-skh into lp:openobject-addons
The proposal to merge lp:~openerp-dev/openobject-addons/trunk-bug-730885-skh into lp:openobject-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-730885-skh/+merge/55318 -- https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-730885-skh/+merge/55318 Your team OpenERP RD Team is subscribed to branch lp:~openerp-dev/openobject-addons/trunk-bug-730885-skh. ___ Mailing list: https://launchpad.net/~openerp-dev-web Post to : openerp-dev-web@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-web More help : https://help.launchpad.net/ListHelp