c2c-oerpscenario team mailing list archive

["Amit Parik \(OpenERP\)" <amp@xxxxxxxxxxx>]

Hello DR Ferdinand,

I have checked your issue, I am not agree with you the condition is correct, but you have give a suggestion on it.
You have specified that "update_posted should be checked" this field is in account.journal but it is visible only when we have to install account_cancel module. This boolean field is used when we have to cancel the journal entires .

The  _update_check method is called when we have updated a account move line (journal entires).
When the invoice is in draft state we allow to update it without install account_cancel module or without checked the update_posted.
This is not a bug but it's your suggestion as well as opinion so before implementing it we need to more discussion on it.

That's why currently I am setting this as an "Opinion" .

@Experts : would you please share your views on this.

Thanks and more suggestions are welcomed!


** Changed in: openobject-addons
       Status: New => Opinion

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to OpenERP Project Group.
https://bugs.launchpad.net/bugs/883161

Title:
  account_move_line _update_check wrong condition

Status in OpenERP Addons (modules):
  Opinion

Bug description:
  since 6.0 the if statmente includes
  ....and (not line.journal_id.entry_posted)
   
      def _update_check(self, cr, uid, ids, context=None):
          done = {}
          for line in self.browse(cr, uid, ids, context=context):
              err_msg = _('Move name (id): ')  + line.move_id.name +' ('+ str(line.move_id.id) +')'
              if line.move_id.state <> 'draft' and (not line.journal_id.entry_posted):

  IMHO update_posted should be checked.

  entry_posted is used to control if moves will be directly posted.

  may be I miss something

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-addons/+bug/883161/+subscriptions