Skip to content

Commit e9a3678

Browse files
Merge branch 'MAGETWO-55193' of https://github.com/magento-falcons/magento2ce into MAGETWO-56240
2 parents 1ae119f + ce5171f commit e9a3678

File tree

24 files changed

+370
-281
lines changed

24 files changed

+370
-281
lines changed

app/design/adminhtml/Magento/backend/web/app/updater/styles/less/components/_data-grid.less

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,26 @@
112112
}
113113
}
114114
}
115+
116+
&._tooltip {
117+
background: transparent;
118+
margin: 0px 0px 8px 5px;
119+
120+
a {
121+
width: 21px;
122+
123+
&:hover {
124+
text-decoration: none;
125+
}
126+
127+
&:before {
128+
color: @color-tertiary;
129+
content: @icon-help__content;
130+
font-family: @icons__font-family;
131+
font-size: @component-indicator__size;
132+
}
133+
}
134+
}
115135
}
116136

117137
.col-manager-item-name {

setup/pub/magento/setup/extension-grid.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ angular.module('extension-grid', ['ngStorage'])
129129
version: $scope.availableUpdatePackages[extension.name]['latestVersion']
130130
}
131131
];
132-
titleService.setTitle('update', extension.moduleName ? extension.moduleName : extension.name);
132+
titleService.setTitle('update', extension);
133133
$state.go('root.readiness-check-update');
134134
};
135135

@@ -139,7 +139,7 @@ angular.module('extension-grid', ['ngStorage'])
139139
name: extension.name
140140
}
141141
];
142-
titleService.setTitle('uninstall', extension.moduleName ? extension.moduleName : extension.name);
142+
titleService.setTitle('uninstall', extension);
143143
$localStorage.componentType = extension.type;
144144
$state.go('root.readiness-check-uninstall');
145145
};

setup/pub/magento/setup/install-extension-grid.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -74,6 +74,7 @@ angular.module('install-extension-grid', ['ngStorage', 'clickOut'])
7474
}
7575
];
7676
$localStorage.moduleName = extension.name;
77+
$localStorage.packageTitle = extension.package_title;
7778
$scope.error = false;
7879
$scope.errorMessage = '';
7980
}

setup/pub/magento/setup/main.js

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -251,13 +251,19 @@ main.controller('navigationController',
251251
.service('titleService', ['$localStorage', '$rootScope',
252252
function ($localStorage, $rootScope) {
253253
return {
254-
setTitle: function(type, moduleName) {
255-
$localStorage.moduleName = moduleName;
254+
setTitle: function(type, component) {
255+
if (type === 'enable' || type === 'disable') {
256+
$localStorage.packageTitle = $localStorage.moduleName = component.moduleName;
257+
} else {
258+
$localStorage.moduleName = component.moduleName ? component.moduleName : component.name;
259+
$localStorage.packageTitle = component.package_title;
260+
}
261+
256262
if (typeof $localStorage.titles === 'undefined') {
257263
$localStorage.titles = [];
258264
}
259265
$localStorage.titles[type] = type.charAt(0).toUpperCase() + type.slice(1) + ' '
260-
+ $localStorage.moduleName;
266+
+ ($localStorage.packageTitle ? $localStorage.packageTitle : $localStorage.moduleName);
261267
$rootScope.titles = $localStorage.titles;
262268
}
263269
};

setup/pub/magento/setup/module-grid.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ angular.module('module-grid', ['ngStorage'])
6666
isComposerPackage: component.name !== 'unknown',
6767
}
6868
];
69-
titleService.setTitle(type, component.moduleName ? component.moduleName : component.name);
69+
titleService.setTitle(type, component);
7070
$localStorage.componentType = component.type;
7171
$state.go('root.readiness-check-'+type);
7272
};

setup/pub/magento/setup/readiness-check.js

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,7 @@ angular.module('readiness-check', ['remove-dialog'])
353353
$scope.wordingOfReadinessCheckAction = function() {
354354
var $actionString = 'We\'re making sure your server environment is ready for ';
355355
if ($localStorage.moduleName) {
356-
$actionString += $localStorage.moduleName;
356+
$actionString += $localStorage.packageTitle ? $localStorage.packageTitle : $localStorage.moduleName;
357357
} else {
358358
if($state.current.type === 'install' || $state.current.type === 'upgrade') {
359359
$actionString += 'Magento';
@@ -373,6 +373,14 @@ angular.module('readiness-check', ['remove-dialog'])
373373
} else {
374374
$actionString +='ed';
375375
}
376+
377+
if ($localStorage.moduleName
378+
&& $localStorage.packageTitle
379+
&& $localStorage.moduleName != $localStorage.packageTitle
380+
) {
381+
$actionString += ', which consists of the following packages:<br/>- ' + $localStorage.moduleName;
382+
}
383+
376384
return $actionString;
377385
};
378386

setup/pub/magento/setup/update-extension-grid.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ angular.module('update-extension-grid', ['ngStorage', 'clickOut'])
4848
version: extension.updateVersion
4949
}
5050
];
51-
titleService.setTitle('update', extension.name);
51+
titleService.setTitle('update', extension);
5252
$scope.nextState();
5353
};
5454
$scope.isHiddenSpinner = true;

setup/pub/styles/setup.css

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

setup/src/Magento/Setup/Controller/InstallExtensionGrid.php

Lines changed: 17 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
use Zend\View\Model\JsonModel;
1111
use Zend\View\Model\ViewModel;
1212
use Magento\Setup\Model\PackagesData;
13-
use \Magento\Setup\Model\Grid\TypeMapper;
1413

1514
/**
1615
* Controller for extensions grid tasks
@@ -22,21 +21,13 @@ class InstallExtensionGrid extends AbstractActionController
2221
*/
2322
private $packagesData;
2423

25-
/**
26-
* @var TypeMapper
27-
*/
28-
private $typeMapper;
29-
3024
/**
3125
* @param PackagesData $packagesData
32-
* @param TypeMapper $typeMapper
3326
*/
3427
public function __construct(
35-
PackagesData $packagesData,
36-
TypeMapper $typeMapper
28+
PackagesData $packagesData
3729
) {
3830
$this->packagesData = $packagesData;
39-
$this->typeMapper = $typeMapper;
4031
}
4132

4233
/**
@@ -60,10 +51,7 @@ public function extensionsAction()
6051
{
6152
$extensions = $this->packagesData->getPackagesForInstall();
6253
$packages = isset($extensions['packages']) ? $extensions['packages'] : [];
63-
array_walk($packages, function (&$package) {
64-
$package['vendor'] = ucfirst($package['vendor']);
65-
$package['type'] = $this->typeMapper->map($package['name'], $package['type']);
66-
});
54+
$packages = $this->formatPackageList($packages);
6755

6856
return new JsonModel(
6957
[
@@ -73,4 +61,19 @@ public function extensionsAction()
7361
]
7462
);
7563
}
64+
65+
/**
66+
* Format package list
67+
*
68+
* @param array $packages
69+
* @return array
70+
*/
71+
private function formatPackageList(array $packages)
72+
{
73+
array_walk($packages, function (&$package) {
74+
$package['vendor'] = ucfirst($package['vendor']);
75+
});
76+
77+
return $packages;
78+
}
7679
}

setup/src/Magento/Setup/Model/Grid/Extension.php

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,6 @@ class Extension
1818
*/
1919
private $composerInformation;
2020

21-
/**
22-
* @var TypeMapper
23-
*/
24-
private $typeMapper;
25-
2621
/**
2722
* @var PackagesData
2823
*/
@@ -31,16 +26,13 @@ class Extension
3126
/**
3227
* @param ComposerInformation $composerInformation
3328
* @param PackagesData $packagesData
34-
* @param TypeMapper $typeMapper
3529
*/
3630
public function __construct(
3731
ComposerInformation $composerInformation,
38-
PackagesData $packagesData,
39-
TypeMapper $typeMapper
32+
PackagesData $packagesData
4033
) {
4134
$this->composerInformation = $composerInformation;
4235
$this->packagesData = $packagesData;
43-
$this->typeMapper = $typeMapper;
4436
}
4537

4638
/**
@@ -89,7 +81,6 @@ private function formatExtensions(array $extensions)
8981
{
9082
foreach ($extensions as &$extension) {
9183
$extension['vendor'] = ucfirst(current(explode('/', $extension['name'])));
92-
$extension['type'] = $this->typeMapper->map($extension['name'], $extension['type']);
9384
}
9485
return array_values($extensions);
9586
}

0 commit comments

Comments
 (0)