Skip to content

Commit 10da31a

Browse files
authored
Merge pull request #7758 from ProcessMaker/bugfix/FOUR-20443
FOUR-20443: The permission "View All Cases" and "View My Request" are not showing
2 parents 2e53820 + 998c1aa commit 10da31a

3 files changed

Lines changed: 74 additions & 27 deletions

File tree

‎database/seeders/RenameRequestsToCasesPermissionSeeder.php‎

Lines changed: 0 additions & 25 deletions
This file was deleted.

‎resources/jscomposition/cases/casesMain/utils/counters.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export const formatCounters = (data) => {
3030
},
3131
];
3232

33-
if (data.totalAllCases) {
33+
if (data.totalAllCases !== null) {
3434
counters.push({
3535
header: t("All cases"),
3636
body: data.totalAllCases.toString(),
@@ -41,7 +41,7 @@ export const formatCounters = (data) => {
4141
});
4242
}
4343

44-
if (data.totalMyRequest) {
44+
if (data.totalMyRequest !== null) {
4545
counters.push({
4646
header: t("My requests"),
4747
body: data.totalMyRequest.toString(),
Lines changed: 72 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,72 @@
1+
<?php
2+
3+
use ProcessMaker\Models\Permission;
4+
use ProcessMaker\Upgrades\UpgradeMigration as Upgrade;
5+
6+
class CreatePermissionViewAllCasesAllRequests extends Upgrade
7+
{
8+
private const NEW_NAME_GROUP = 'Cases and Requests';
9+
10+
private const PERMISSIONS = [
11+
[
12+
'name' => 'view-all_cases',
13+
'title' => 'View All Cases',
14+
],
15+
[
16+
'name' => 'view-my_requests',
17+
'title' => 'View My Requests',
18+
],
19+
];
20+
21+
/**
22+
* Run the migrations.
23+
*/
24+
public function up(): void
25+
{
26+
// Update the group Requests to Cases and Requests
27+
DB::table('permissions')
28+
->where('group', 'Requests')
29+
->update(['group' => self::NEW_NAME_GROUP]);
30+
// Update the group Cases to Cases and Requests
31+
DB::table('permissions')
32+
->where('group', 'Cases')
33+
->update(['group' => self::NEW_NAME_GROUP]);
34+
// Update with the correct label "View All Request"
35+
// Summer 2024: PO requested change to "View All Cases"
36+
// Fall 2024: PO requested revert to "View All Request"
37+
$permission = Permission::where('name', 'view-all_requests')->first();
38+
if ($permission && $permission->title == 'View All Cases') {
39+
$permission->title = 'View All Request';
40+
$permission->save();
41+
}
42+
// Create new permissions [view-all_cases, view-all_requests]
43+
$this->createPermissions();
44+
}
45+
46+
/**
47+
* Create new permissions
48+
*
49+
* @return void
50+
*/
51+
private function createPermissions(): void
52+
{
53+
foreach (self::PERMISSIONS as $permission) {
54+
Permission::updateOrCreate([
55+
'name' => $permission['name'],
56+
], [
57+
'title' => $permission['title'],
58+
'name' => $permission['name'],
59+
'group' => self::NEW_NAME_GROUP,
60+
]);
61+
}
62+
}
63+
64+
/**
65+
* Reverse the migrations.
66+
*/
67+
public function down(): void
68+
{
69+
Permission::where('name', 'view-all_cases')->delete();
70+
Permission::where('name', 'view-my_requests')->delete();
71+
}
72+
}

0 commit comments

Comments
 (0)