Issue 179103: Use root user for create/delete default properties in account chart creation

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Issue 179103: Use root user for create/delete default properties in account chart creation (Closed)

Can't Edit
Can't Publish+Mail
Start Review

Created:
16 years, 5 months ago by ced1

Modified:
16 years, 5 months ago

Reviewers:
bch, udono, yangoon1

Visibility:
Public.

Patch Set 1 #

Created: 16 years, 5 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+4 lines, -4 lines)			Patch
	M	account.py	View		2 chunks	+4 lines, -4 lines	0 comments	Download

Messages

Total messages: 6

Expand All Messages | Collapse All Messages

yangoon1

I don't understand why this change is needed. Could you give a little hint please?

16 years, 5 months ago (2009-12-18 14:00:00 UTC) #3

ced1

On 2009/12/18 14:00:00, yangoon wrote: > I don't understand why this change is needed. Could ...

16 years, 5 months ago (2009-12-18 14:49:02 UTC) #4

bch

On 2009/12/18 14:49:02, ced wrote: > On 2009/12/18 14:00:00, yangoon wrote: > > I don't ...

16 years, 5 months ago (2009-12-18 16:54:38 UTC) #5

ced1

16 years, 5 months ago (2009-12-18 17:34:11 UTC) #6

On 2009/12/18 16:54:38, bch wrote:
> On 2009/12/18 14:49:02, ced wrote:
> > On 2009/12/18 14:00:00, yangoon wrote:
> > > I don't understand why this change is needed. Could you give a little hint
> > > please?
> > 
> > If you are not in the Administration group, you can not create the default
> > properties for account_receivable and account_payable
> 
> Are the access restriction needed on properties? because there are already
> access control on the models targeted by the property field.

It is done on set/get method not on create/write/delete.
So the current access control seems correct for me as normally user must not
access to ir.property directly.
By the way, it will be good to add some ModelSingleton to be able to update some
properties without admin right like the default account receivable/payable.

Expand All Messages | Collapse All Messages