Adding Module base_copy_access

[mikevhe18]

No description provided.

[oca-clabot]

Hey @mikevhe18, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/website.cla
Here is a list of the users:

• mike (no github login found)

Appreciation of efforts,
OCA CLAbot

[andhit-r]

I think WizardBaseCopyUserAccess should be models.TransientModel, not models.Model

[mikevhe18]

@andhit-r Yes, you're right. I'd missed it. Thank You

[mikevhe18]

Dear Community,
Can someone tell me how to pass the coverage/coveralls checks?
I'm a newbie here.
Thank You

[sebalix]

Hi,

What is the purpose of the module? The description is not very clear to me (I encourage you to follow the CONTRIBUTING guide, and the template module (README.rst, etc).
I read the code, and it seems that the wizard copies the groups of one user on the current user. Excepted the admin user account, no user should be able to do this by default. If a user is able to copy any user rights, this is a security hole. Maybe I do not understand well the purpose of this, that's why I am asking :)

[mikevhe18]

Hi,

The main purpose of this module is to help out admin to work easier. Sometimes users want to delegate their access right to another user for some reason. Not all user can use this feature , only admin can be.

[sebalix]

Okay, I was misreading the code (we are using https://github.com/osiell/base_user_role for this kind of use case). Anyway, you should update your module to follow the CONTRIBUTING guidelines (new module in 8.0 should be in written in new API) and add a README.rst file, explaining how to use this module (select users in the list, then launch the action, etc).

[dreispt]

Please rebase on latest server-tools 8.0: I can see commits regarding unrelated modules, such as audit_log.

[andhit-r]

Remove empty key on https://github.com/OCA/server-tools/pull/323/files#diff-378482bb041861896faabba9932bb193R12

[andhit-r]

Use models.TransientModel on https://github.com/OCA/server-tools/pull/323/files#diff-032362b48ae883d61fa810d7f9f83c57R10

[andhit-r]

Add self.ensure_one()

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.