Skip to content
This repository has been archived by the owner on Jun 22, 2020. It is now read-only.

Issue #3040605: Order of optional thunder modules install is wrong #669

Open
wants to merge 8 commits into
base: 8.x-2.x
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions modules/thunder_demo/thunder_demo.info.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,5 +6,3 @@ core: 8.x
dependencies:
- better_normalizers:better_normalizers
- default_content:default_content
- drupal:tour
- drupal:help
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ langcode: en
status: true
dependencies:
module:
- thunder_demo
- thunder_help
id: content-add
label: 'Add new content'
module: thunder_demo
module: thunder_help
routes:
-
route_name: node.add
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ langcode: en
status: true
dependencies:
module:
- thunder_demo
- thunder_help
id: content-list
label: 'Content listing'
module: thunder_demo
module: thunder_help
routes:
-
route_name: view.content.page_1
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ langcode: en
status: true
dependencies:
module:
- thunder_demo
- thunder_help
id: content-paragraphs
label: 'Introduction to paragraphs'
module: thunder_demo
module: thunder_help
routes:
-
route_name: entity.node.edit_form
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ langcode: en
status: true
dependencies:
module:
- thunder_demo
- thunder_help
id: homepage
label: Homepage
module: thunder_demo
module: thunder_help
routes:
-
route_name: view.frontpage.page_1
Expand Down
8 changes: 8 additions & 0 deletions modules/thunder_help/thunder_help.info.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
name: Thunder Help
type: module
description: Provides a tour and some help.
package: Thunder
core: 8.x
dependencies:
- drupal:tour
- drupal:help
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ use Drupal\Core\Entity\EntityInterface;
* Tour depends a lot on used theme. Because of that whitelist is provided, to
* ensure that tour is loaded only if proper theme is used.
*/
function thunder_demo_tour_tips_alter(array &$tour_tips, EntityInterface $entity) {
function thunder_help_tour_tips_alter(array &$tour_tips, EntityInterface $entity) {
// Whitelist of tours for frontend and backend theme.
$tourThemeMapping = [
'homepage' => ['thunder_base'],
Expand Down
2 changes: 1 addition & 1 deletion scripts/travis/04-install-thunder.sh
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
install_thunder() {
cd $1

/usr/bin/env PHP_OPTIONS="-d sendmail_path=`which true`" drush si thunder --db-url=mysql://[email protected]/drupal -y thunder_module_configure_form.install_modules_thunder_demo
/usr/bin/env PHP_OPTIONS="-d sendmail_path=`which true`" drush si thunder --db-url=mysql://[email protected]/drupal -y install_configure_form.demo_content
drush en simpletest -y

drush -y sql-dump --result-file=${DEPLOYMENT_DUMP_FILE}
Expand Down
46 changes: 0 additions & 46 deletions src/Annotation/ThunderOptionalModule.php

This file was deleted.

Loading