Skip to content

Commit 92a9267

Browse files
committed
Revert "OHRM5X-2117: Exclude claim module from v5.4 (orangehrm#1661)"
This reverts commit 7b486ef.
1 parent de4c404 commit 92a9267

File tree

8 files changed

+27
-8
lines changed

8 files changed

+27
-8
lines changed

.github/workflows/build.yml

-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ jobs:
2020

2121
- name: Install dependencies
2222
run: |
23-
rm -r src/plugins/orangehrmClaimPlugin
2423
cd src
2524
php7.4 -f /usr/bin/composer install --no-dev
2625
php7.4 -f /usr/bin/composer dump-autoload --optimize --no-dev --classmap-authoritative

build/build.xml

-3
Original file line numberDiff line numberDiff line change
@@ -68,9 +68,6 @@
6868
<exclude name="src/test/**"/>
6969
<exclude name="src/plugins/**/test/**"/>
7070

71-
<exclude name="installer/Migration/V5_5_0/**"/>
72-
<exclude name="src/plugins/orangehrmClaimPlugin/**"/>
73-
<exclude name="src/client/src/orangehrmClaimPlugin/**"/>
7471
<exclude name="src/plugins/orangehrmAuthenticationPlugin/lib/**"/>
7572
<exclude name="src/plugins/orangehrmCoreOAuthPlugin/lib/**"/>
7673
<exclude name="src/plugins/orangehrmOpenidAuthenticationPlugin/**"/>

devTools/core/src/Command/GenerateOpenApiDocCommand.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
5454
$paths = [Config::get(Config::PLUGINS_DIR) . '/orangehrmCorePlugin/Controller/Rest/V2'];
5555
foreach (Config::get(Config::PLUGIN_PATHS) as $pluginAbsPath) {
5656
$pathToApiDir = realpath($pluginAbsPath . '/Api');
57-
if ($pathToApiDir !== false && !str_ends_with($pathToApiDir, 'src/plugins/orangehrmClaimPlugin/Api')) {
57+
if ($pathToApiDir !== false) {
5858
$paths[] = $pathToApiDir;
5959
}
6060
}

installer/Util/AppSetupUtility.php

+4-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,10 @@ class AppSetupUtility
6666
'5.1' => \OrangeHRM\Installer\Migration\V5_1_0\Migration::class,
6767
'5.2' => \OrangeHRM\Installer\Migration\V5_2_0\Migration::class,
6868
'5.3' => \OrangeHRM\Installer\Migration\V5_3_0\Migration::class,
69-
'5.4' => \OrangeHRM\Installer\Migration\V5_4_0\Migration::class,
69+
'5.4' => [
70+
\OrangeHRM\Installer\Migration\V5_4_0\Migration::class,
71+
\OrangeHRM\Installer\Migration\V5_5_0\Migration::class,
72+
],
7073
];
7174

7275
public const INSTALLATION_DB_TYPE_NEW = 'new';

phpunit.xml

+3
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,9 @@
7272
<testsuite name="Buzz">
7373
<directory>src/plugins/orangehrmBuzzPlugin/test</directory>
7474
</testsuite>
75+
<testsuite name="Claim">
76+
<directory>src/plugins/orangehrmClaimPlugin/test</directory>
77+
</testsuite>
7578
<testsuite name="Help">
7679
<directory>src/plugins/orangehrmHelpPlugin/test</directory>
7780
</testsuite>

src/client/src/pages.ts

+2
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ import languagePages from '@/orangehrmAdminPlugin';
3333
import dashboardPages from '@/orangehrmDashboardPlugin';
3434
import buzzPages from '@/orangehrmBuzzPlugin';
3535
import systemCheckPages from '@/orangehrmSystemCheckPlugin';
36+
import claimPages from '@/orangehrmClaimPlugin';
3637

3738
export default {
3839
...AdminPages,
@@ -52,4 +53,5 @@ export default {
5253
...dashboardPages,
5354
...buzzPages,
5455
...systemCheckPages,
56+
...claimPages,
5557
};

src/composer.json

+5-2
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@
7979
"OrangeHRM\\LDAP\\": "./plugins/orangehrmLDAPAuthenticationPlugin",
8080
"OrangeHRM\\SystemCheck\\": "./plugins/orangehrmSystemCheckPlugin",
8181
"OrangeHRM\\Mobile\\": "./plugins/orangehrmMobilePlugin",
82+
"OrangeHRM\\Claim\\": "./plugins/orangehrmClaimPlugin",
8283
"OrangeHRM\\Installer\\": "../installer",
8384
"OrangeHRM\\Entity\\": [
8485
"./plugins/orangehrmAdminPlugin/entity",
@@ -94,7 +95,8 @@
9495
"./plugins/orangehrmRecruitmentPlugin/entity",
9596
"./plugins/orangehrmCorporateBrandingPlugin/entity",
9697
"./plugins/orangehrmLDAPAuthenticationPlugin/entity",
97-
"./plugins/orangehrmBuzzPlugin/entity"
98+
"./plugins/orangehrmBuzzPlugin/entity",
99+
"./plugins/orangehrmClaimPlugin/entity"
98100
]
99101
}
100102
},
@@ -119,7 +121,8 @@
119121
"OrangeHRM\\Tests\\LDAP\\": "./plugins/orangehrmLDAPAuthenticationPlugin/test",
120122
"OrangeHRM\\Tests\\Buzz\\": "./plugins/orangehrmBuzzPlugin/test",
121123
"OrangeHRM\\Tests\\Help\\": "./plugins/orangehrmHelpPlugin/test",
122-
"OrangeHRM\\Tests\\Mobile\\": "./plugins/orangehrmMobilePlugin/test"
124+
"OrangeHRM\\Tests\\Mobile\\": "./plugins/orangehrmMobilePlugin/test",
125+
"OrangeHRM\\Tests\\Claim\\": "./plugins/orangehrmClaimPlugin/test"
123126
}
124127
},
125128
"scripts": {

src/plugins/orangehrmCorePlugin/test/Service/MenuServiceTest.php

+12
Original file line numberDiff line numberDiff line change
@@ -174,6 +174,12 @@ public function testGetMenuItemsForAdmin(): void
174174
'url' => '/maintenance/viewMaintenanceModule',
175175
'icon' => 'maintenance',
176176
],
177+
[
178+
'id' => 105,
179+
'name' => 'Claim',
180+
'url' => '/claim/viewClaimModule',
181+
'icon' => 'claim',
182+
],
177183
[
178184
'id' => 101,
179185
'name' => 'Buzz',
@@ -433,6 +439,12 @@ public function testGetMenuItemsForSupervisor(): void
433439
'url' => '/directory/viewDirectory',
434440
'icon' => 'directory',
435441
],
442+
[
443+
'id' => 105,
444+
'name' => 'Claim',
445+
'url' => '/claim/viewClaimModule',
446+
'icon' => 'claim'
447+
],
436448
[
437449
'id' => 101,
438450
'name' => 'Buzz',

0 commit comments

Comments
 (0)