Create a Patch
Quick links:
So you want to fix a bug or add a new feature to TYPO3? Great!
Tip
If you should encounter any problems or have questions, talk to us on https://typo3.slack.com in the #typo3-cms-coredev channel (see Slack).
Step by Step Walkthrough
You should have a cloned Git repository with a working TYPO3 installation as described in setup (or via the Quick start guide). Especially the Git setup is required.
-
Create an Issue on Forge
More information: Report an Issue
Every patch must have a matching issue on Forge, so create an issue now or submit a patch for an existing issue.
-
Make your changes to the code, add documentation, tests
This part is pretty straightforward. But be warned, there are still a few dark places deep inside the TYPO3 core dating back to the medieval times of PHP4. Yes, TYPO3 has been around for quite some time now. And there is ancient code we didn't have to touch yet because it just works.
Make sure to look at How to deprecate classes, methods, arguments and hooks in the TYPO3 core in the Appendix for information about how to deprecate things if you need to make changes to the public API.
For new features, breaking changes and deprecations, it is necessary to add information to the changelog.
If you change SCSS, JavaScript or TypeScript files, you can build locally.
Add Unit Tests or Functional Tests for new functionality, refine existing tests if necessary. Tests are important because they ensure that TYPO3 will behave consistently now and in the future.
See Testing the core in TYPO3 Explained for more information about writing and running tests.
Once you have finalized your patch, check out the Common code review checks for a list of what kind of review checks people may perform on your contribution. Stay ahead of the game and address those yourself first.
-
Commit your changes
Please make sure that you read the Commit Message rules for TYPO3 CMS in the Appendix. Your code will not be merged if it does not follow the commit message rules.
Important
The section Commit Message rules for TYPO3 CMS is a must-read. Read it. Follow it.
For a bugfix, your commit message may look something like this:
[BUGFIX] Subject line of max 52 chars Some descriptions with line length of max. 72 characters Resolves: #12346 Releases: main, 13.4
Copied!Only create one commit. Do not create a branch. Work on main.
git commit -a
Copied!The commit-msg hook will do some sanity checks and add a line starting with
Change-
.Id: If you have activated the pre-commit hook it will loudly complain if something does not conform to the coding guidelines.
In that case, use the runTests.sh script to to fix CGL issues.
After you have created your commit, you can still make changes by amending to your commit:
git commit -a --amend
Copied!Tip
Keep in mind that you can commit with --amend as often as you want, just make sure you keep the
Change-
line intact.Id: -
Push to Gerrit
To submit the patch to Gerrit, issue the following command:
git push origin HEAD:refs/for/main
Copied!If you have setup the default as described in Setting up Your Remote it is sufficient to use:
git push
Copied!In case you want to push a "Work in progress", check out: Workflow - work in progress.
If Gerrit accepts your push, it responds with the following messages:
remote: SUCCESS remote: remote: https://review.typo3.org/c/Packages/TYPO3.CMS/+/<gerrit-id> [...] ... [NEW] remote: To ssh://review.typo3.org:29418/Packages/TYPO3.CMS.git * [new reference] main -> refs/for/main
Copied!If you see an error, check out the Git Troubleshooting section.
You can visit the link to https://review.typo3.org to see your patch in Gerrit.
The Continuous Integration service, running on GitLab Pipelines, will automatically be executed in the background and perform checks and tests on your patch. Failing tests will be linked to that instance, where jobs can also be retried (given sufficient permissions when being logged in to GitLab).
Advanced users / core team only: See cheat sheet: other branches for pushing to other branches.
- Optional: Advertise review on Slack channel
- Once your push to Gerrit goes through, you will receive a URL for your new
-
change. If you are on Slack you can now advertise your new change in the #typo3-cms-coredev channel. You can get a preformatted line of your change to post in the channel by clicking the copy button next to the title in Gerrit_:
This is not something, you will do for every review. As a first contributor it is recommended to mention that you are new to the process.
Now, it's time to sit back and await feedback on your changes. The review team process dozens of requests each day, so expect a succinct response that is short and to the point. You will get notified by email, if there is activity on your patch in Gerrit (e.g. votes, comments, new patchsets, merge etc.).
Check out the section Review a patch for more about this process, in which you can also be involved!
It is not unusual for a patch to get comments requesting changes. If that happens, please respond in a timely fashion and improve your review. If things are unclear, ask in the #typo3-cms-coredev channel on https://typo3.slack.com.
Helpful links
Next Steps
You will find some more information about the review process in the chapter Handle and Improve a Patch (Gerrit). The following pages are especially relevant for new contributors:
- Tips for new contributors
- Introduction to Gerrit describes the review tool Gerrit.
- Find a review on Gerrit is helpful if you don't know how to find your patch on Gerrit.
- Gerrit works with up- and downvoting patches. A patch must get a specific number of upvotes before it can be merged. Code Review gives an introduction to how this works.
- When you make additional changes to your patch, make sure you do not add another commit. Append to your original commit instead as described in Upload a new Patch Set.
- Before starting to work on a new, unrelated patch you need to run the Cleanup tasks.