c2c-oerpscenario team mailing list archive

["Albert Cervera i Areny - http://www.NaN-tic.com" <albert@xxxxxxxxxxx>]

A Dilluns, 2 de maig de 2011 07:42:35, Amit Parik (OpenERP) va escriure:
> Hello Albert,
> 
> I have checked your suggested patch again but there is no need to add
> this type of domain because we have  already define a record rule for
> multi-company.
> 
> So no need to add this type of domain in account.account object.

Well, rules do not apply to admin user. Anyway, fixing this here would mean 
fixing it to many other places so it may have no sense to fix only one place.

-- 
Albert Cervera i Areny
http://www.NaN-tic.com
OpenERP Partners
Tel: +34 93 553 18 03
skype: nan-oficina

http://twitter.com/albertnan 
http://www.nan-tic.com/blog

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

Title:
  default accounts in journal allows selecting invalid company

Status in OpenERP Modules (addons):
  Invalid

Bug description:
  If user has multi-company rights she can select accounts from any
  company as default credit/debit accounts.

  The attached patch changes the domain of default-account fields so
  only accounts from current company can be selected. However, the patch
  does not include a check constraint that IMHO it should be added to
  ensure account and journal companies match because the patch does not
  prevent the user from selecting another company once accounts have
  been filled in.

  Regarding the patch, note that I've removed the domain from the view
  and left only the domain in the .py file.