mirror of
https://github.com/frappe/erpnext.git
synced 2026-10-31 23:07:51 +00:00
fix(setup): allow fleet managers to maintain drivers
Unlike Vehicle, Driver only gave Fleet Manager read access, blocking routine fleet maintenance. Align its create, write and delete permissions with Vehicle.
Issue: frappe/erpnext-bug-reviews#381 (bug 5)
(cherry picked from commit ad8e44acc2)
This commit is contained in:
@@ -126,13 +126,16 @@
|
||||
"owner": "Administrator",
|
||||
"permissions": [
|
||||
{
|
||||
"create": 1,
|
||||
"delete": 1,
|
||||
"email": 1,
|
||||
"export": 1,
|
||||
"print": 1,
|
||||
"read": 1,
|
||||
"report": 1,
|
||||
"role": "Fleet Manager",
|
||||
"share": 1
|
||||
"share": 1,
|
||||
"write": 1
|
||||
},
|
||||
{
|
||||
"create": 1,
|
||||
|
||||
@@ -19,3 +19,11 @@ class TestDriver(unittest.TestCase):
|
||||
)
|
||||
with self.assertRaises(frappe.ValidationError):
|
||||
driver.insert()
|
||||
|
||||
def test_fleet_manager_can_maintain_drivers(self):
|
||||
user = f"fleet-{frappe.generate_hash(length=10)}@example.com"
|
||||
frappe.get_doc(
|
||||
{"doctype": "User", "email": user, "first_name": "Fleet", "send_welcome_email": 0}
|
||||
).insert().add_roles("Fleet Manager")
|
||||
for permission in ("create", "write", "delete"):
|
||||
self.assertTrue(frappe.has_permission("Driver", permission, user=user))
|
||||
|
||||
Reference in New Issue
Block a user