Skip to content

Commit 17827a9

Browse files
committed
!410 - Changed external type dependency to composer.
1 parent bc8a39c commit 17827a9

File tree

3 files changed

+30
-21
lines changed

3 files changed

+30
-21
lines changed

apps/Core/Packages/Devtools/Modules/DevtoolsModules.php

Lines changed: 18 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -594,13 +594,23 @@ public function getDefaultDependencies($type = null)
594594
{
595595
$defaultDependencies =
596596
[
597-
'core' => [],
598-
'apptype' => [],
599-
'components' => [],
600-
'packages' => [],
601-
'middlewares' => [],
602-
'views' => [],
603-
'external' => []
597+
'core' => [],
598+
'apptype' => [],
599+
'components' => [],
600+
'packages' => [],
601+
'middlewares' => [],
602+
'views' => [],
603+
'external' => [
604+
'composer' => [
605+
'require' => []
606+
],
607+
'config' => [
608+
'allow-plugins' => []
609+
],
610+
'extra' => [
611+
'patches' => []
612+
]
613+
]
604614
];
605615

606616
if ($type && $type === 'views') {
@@ -1586,6 +1596,7 @@ protected function initApi($data)
15861596
}
15871597

15881598
$this->apiClient = $this->basepackages->apiClientServices->useApi($data['api_id'], true);
1599+
15891600
$this->apiClientConfig = $this->apiClient->getApiConfig();
15901601

15911602
if ($this->apiClientConfig['auth_type'] === 'auth' &&
@@ -2425,8 +2436,6 @@ public function commitBundleJson($data)
24252436

24262437
return;
24272438
}
2428-
2429-
throw $e;
24302439
}
24312440

24322441
$jsonContent = [];

apps/Core/Views/Default/html/devtools/modules/module.html

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -994,11 +994,11 @@
994994
dataCollectionSectionForm['vars']['bundleModules'][label]['repo'] = moduleData['repo'];
995995

996996
} else if (label === 'external') {
997-
if (dataCollectionSectionForm['vars']['bundleModules']['external'].length === 0) {
998-
dataCollectionSectionForm['vars']['bundleModules']['external'] = { };
997+
if (dataCollectionSectionForm['vars']['bundleModules']['external']['composer']['require'].length === 0) {
998+
dataCollectionSectionForm['vars']['bundleModules']['external']['composer']['require'] = { };
999999
}
10001000

1001-
dataCollectionSectionForm['vars']['bundleModules'][label][$('#{{componentId}}-{{sectionId}}-modules-external').val()]
1001+
dataCollectionSectionForm['vars']['bundleModules'][label]['composer']['require'][$('#{{componentId}}-{{sectionId}}-modules-external').val()]
10021002
= $('#{{componentId}}-{{sectionId}}-modules-external-version').val();
10031003

10041004
reset();
@@ -1041,7 +1041,7 @@
10411041
}
10421042
dataCollectionSectionForm['vars']['bundleModules'][label] = { };
10431043
} else if (label === 'external') {
1044-
delete dataCollectionSectionForm['vars']['bundleModules'][label][$('#{{componentId}}-{{sectionId}}-modules-external').val()];
1044+
delete dataCollectionSectionForm['vars']['bundleModules'][label]['composer']['require'][$('#{{componentId}}-{{sectionId}}-modules-external').val()];
10451045
} else {
10461046
for (var modules in dataCollectionSectionForm['vars']['bundleModules']) {
10471047
if (label === modules) {
@@ -3056,10 +3056,10 @@
30563056
dataCollectionSectionForm['vars']['moduleDependencies']['composer']['require'][$('#{{componentId}}-{{sectionId}}-dependencies-external').val()]
30573057
= $('#{{componentId}}-{{sectionId}}-dependencies-external-version').val();
30583058
} else {
3059-
if (dataCollectionSectionForm['vars']['moduleDependencies']['external'].length === 0) {
3060-
dataCollectionSectionForm['vars']['moduleDependencies']['external'] = { };
3059+
if (dataCollectionSectionForm['vars']['moduleDependencies'][label]['composer']['require'].length === 0) {
3060+
dataCollectionSectionForm['vars']['moduleDependencies'][label]['composer']['require'] = { };
30613061
}
3062-
dataCollectionSectionForm['vars']['moduleDependencies'][label][$('#{{componentId}}-{{sectionId}}-dependencies-external').val()]
3062+
dataCollectionSectionForm['vars']['moduleDependencies'][label]['composer']['require'][$('#{{componentId}}-{{sectionId}}-dependencies-external').val()]
30633063
= $('#{{componentId}}-{{sectionId}}-dependencies-external-version').val();
30643064
}
30653065
} else {
@@ -3120,8 +3120,8 @@
31203120
delete dataCollectionSectionForm['vars']['moduleDependencies']['composer']['require'][$('#{{componentId}}-{{sectionId}}-dependencies-external').val()];
31213121
}
31223122
} else {
3123-
if (dataCollectionSectionForm['vars']['moduleDependencies'][label][$('#{{componentId}}-{{sectionId}}-dependencies-external').val()]) {
3124-
delete dataCollectionSectionForm['vars']['moduleDependencies'][label][$('#{{componentId}}-{{sectionId}}-dependencies-external').val()];
3123+
if (dataCollectionSectionForm['vars']['moduleDependencies'][label]['composer']['require'][$('#{{componentId}}-{{sectionId}}-dependencies-external').val()]) {
3124+
delete dataCollectionSectionForm['vars']['moduleDependencies'][label]['composer']['require'][$('#{{componentId}}-{{sectionId}}-dependencies-external').val()];
31253125
}
31263126
}
31273127
} else {

apps/Core/Views/Default/html/devtools/modules/module/dependencies.html

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -195,15 +195,15 @@
195195
'size' : 'xs',
196196
'disabled' : true,
197197
'type' : 'danger',
198-
'icon' : 'trash'
198+
'icon' : 'trash',
199+
'position' : 'right'
199200
],
200201
'dependencies-add' : [
201202
'title' : 'Add',
202203
'size' : 'xs',
203204
'disabled' : true,
204205
'type' : 'primary',
205-
'icon' : 'plus',
206-
'position' : 'right'
206+
'icon' : 'plus'
207207
]
208208
]
209209
]

0 commit comments

Comments
 (0)