Overriding Varbase
First Rule in Overriding
Do not change the original code of Drupal Core, Varbase, Vartheme or any other Varbase components. In another words "Do not hack the original code without a trace for the change"
Use the standard Drupal overriding methods.
Custom Tweak Modules
Create custom modules and have your override in them.
- Basic hooks like hook alter.
Patching Bugs
In case of capturing a bug. Search if it was filed in an issue. Look if the a patch fix for the issue was uploaded.
Apply the patch using the Composer patching method using:
Do not use the manual patching method in projects.
This action will guarantee the direction of keeping track of changes over the code.
Example: In the
composer.json file for the project:1
{
2
"name": "vardot/varbase-project",
3
"description": "Project template for Varbase distribution.",
4
"type": "project",
5
"license": "GPL-2.0-or-later",
6
7
...
8
...
9
...
10
11
"require": {
12
"composer/installers": "~1.0 || ~2.0",
13
"oomphinc/composer-installers-extender": "~1.0 || ~2.0",
14
"cweagans/composer-patches": "~1.0",
15
"drupal/core-composer-scaffold": "^9",
16
"drupal/core-project-message": "^9",
17
"webflo/drupal-finder": "~1.0",
18
"webmozart/path-util": "~2.0",
19
"vardot/varbase": "~9.0",
20
"vardot/varbase-updater": "~2.0"
21
},
22
"extra": {
23
"enable-patching": true,
24
"composer-exit-on-patch-failure": true,
25
"patchLevel": {
26
"drupal/core": "-p2"
27
},
28
"patches": {
29
"drupal/core": {
30
"Issue 1543858: Add startup configuration for PHP server":
31
"https://www.drupal.org/files/issues/add_a_startup-1543858-30.patch"
32
}
33
}
34
}
35
}
Copied!
Please add the "Issue Number" and the URL for the used patch
Patch for a New Enhancements
An introductory patch for an enhancement over a Varbase component or any Drupal project is welcomed.
A patch may not be committed on the spot. It will need review, and testing for what affect it may do for websites.
A patch will work. But although a custom module is the better logic.
Ignoring Patches
There may be situations in which you want to ignore a patch supplied by a dependency. For example:
- You use a different more recent version of a dependency, and now a patch isn't applying.
- You have a more up to date patch than the dependency, and want to use yours instead of theirs.
- A dependency's patch adds a feature to a project that you don't need.
- Your patches conflict with a dependency's patches.
Example:
The Varbase Core module is requiring Drupal Core with
"drupal/core": "~9.0", . Having number of patches to fix issues or overriding Drupal core.Varbase Core's
composer.json file for the 9.0.7 versionLet imagine that the Drupal core release team had a new release, But they had committed the following issue:
The patch fix for the core issue was added when a fix was needed for the following issue:
Issue #3164237: Fix Varbase Welcome message not showing up after install compilation after adding and enabling moderation sidebar on the homepage
At the point of a new release for Drupal core and in case the patch was committed.
Ignore the patch being used as in the following example method:
1
{
2
"name": "vardot/varbase-project",
3
"description": "Project template for Varbase distribution.",
4
"type": "project",
5
"license": "GPL-2.0-or-later",
6
7
...
8
...
9
...
10
11
"require": {
12
"composer/installers": "~1.0 || ~2.0",
13
"oomphinc/composer-installers-extender": "~1.0 || ~2.0",
14
"cweagans/composer-patches": "~1.0",
15
"drupal/core-composer-scaffold": "^9",
16
"drupal/core-project-message": "^9",
17
"webflo/drupal-finder": "~1.0",
18
"webmozart/path-util": "~2.0",
19
"vardot/varbase": "~9.0",
20
"vardot/varbase-updater": "~2.0"
21
},
22
"extra": {
23
"enable-patching": true,
24
"composer-exit-on-patch-failure": true,
25
"patchLevel": {
26
"drupal/core": "-p2"
27
},
28
"patches": {
29
"drupal/core": {
30
"Issue 1543858: Add startup configuration for PHP server":
31
"https://www.drupal.org/files/issues/add_a_startup-1543858-30.patch"
32
}
33
},
34
"patches-ignore": {
35
"drupal/varbase_core": {
36
"drupal/core": {
37
"Issue #3165435: Fix tour <front> route as route name when a selected node had been set as the front page for the site":
38
"https://www.drupal.org/files/issues/2020-08-16/3165435-2.patch"
39
}
40
}
41
}
42
}
43
}
Copied!
Do not keep patches or patches-ignore for long in projects.
Keep track of them, and clean not needed patches.
- Report back if you'd spotted and committed patch. So it will be removed from the module.
- Report back if you'd spotted a better patch logic and standard. So it will be used instead of the old patch.
Last modified 10mo ago
Export as PDF
Copy link