Skip to content

Commit 2c2e286

Browse files
committed
requested changes
Signed-off-by: Jurj-Bogdan <[email protected]>
1 parent bd8baa9 commit 2c2e286

File tree

2 files changed

+2
-15
lines changed

2 files changed

+2
-15
lines changed

docs/book/v5/upgrading/UPGRADE-5.3.md

Lines changed: 1 addition & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
Dotkernel API 5.3 is a minor release. As such, no significant backward compatibility breaks are expected,
66
with minor backward compatibility breaks being prefixed in this document with `[BC BREAK]`.
7-
87
This document only covers upgrading from version 5.2.
98

109
## Table of Contents
@@ -34,9 +33,7 @@ Following PHPStan's introduction in version 5.2 for the reasons described on the
3433
### Update anonymization
3534

3635
By default, Dotkernel API uses "soft delete" for it's `User` entities in order to preserve the database entries.
37-
3836
Anonymization is used to make sure any sensitive information is scrubbed from the system, with the `User`'s `identity`, `email`, `firstName` and `lastName` properties being overwritten by a unique placeholder.
39-
4037
Version 5.3 is adding an optional suffix from a configuration file, from where it can be used anywhere in the application.
4138

4239
* Add the `userAnonymizeAppend` key to the returned array in `config/autoload/local.php`, as well as to the distributed`config/autoload/local.php.dist`
@@ -64,26 +61,21 @@ $user->setIdentity($placeholder . $this->config['userAnonymizeAppend']) //...
6461
### Update User status and remove isDeleted properties
6562

6663
Up to and including version 5.2, the `User` entity made use of the `UserStatusEnum` to mark the account status (`active` or `inactive`) and marked deleted accounts with the `isDeleted` property.
67-
6864
Starting from version 5.3 the `isDeleted` property has been removed because, by default, there is no use in having both it and the status property.
69-
7065
As such, a new `Deleted` case for `UserStatusEnum` is now used to mark a deleted account and remove the redundancy.
7166

7267
* [BC Break] Remove the `isDeleted` property from the `User` class, alongside all usages, as seen in the [pull request](https://github.com/dotkernel/api/pull/359/files)
7368
* Add a new "deleted" case to `UserStatusEnum`, which is to be used instead of the previous `isDeleted` property
74-
* Update the database and it's migrations to reflect the new structure
69+
* Update the database and its migrations to reflect the new structure
7570
> The use of "isDeleted" was redundant in the default application, and as such was removed
7671
>
7772
> All default methods are updated, but any custom functionality using "isDeleted" will require refactoring
7873
7974
### Update `dotkernel/dot-mail` to version 5.0
8075

8176
Dotkernel API uses `dotkernel/dot-mail` to handle the mailing service, which in versions older than 5.0 was based on `laminas/laminas-mail`.
82-
8377
Due to the deprecation of `laminas/laminas-mail`, a decision was made to switch `dot-mail` to using `symfony/mailer` starting from version 5.0.
84-
8578
To make the API more future-proof, the upgrade to the new version of `dot-mail` was necessary.
86-
8779
The default usage of the mailer remains unchanged, with the only required updates being to configuration, as described below:
8880

8981
* Bump `dotkernel/dot-mail` to "^5.0" in `composer.json`
@@ -109,7 +101,6 @@ Installing the API via `composer create-project` is not recommended, and because
109101
### Add post install script
110102

111103
To make installing the API less of a hassle, a new post installation script was added.
112-
113104
This script generates all the configuration files required by default, leaving the user to simply complete the relevant data.
114105

115106
> Note that the script will not overwrite existing configuration files, preserving any user data
@@ -168,7 +159,6 @@ cp config/autoload/mail.global.php.dist config/autoload/mail.global.php
168159
### Update security.txt
169160

170161
Updated the `security.txt` file to define the preferred language of the security team.
171-
172162
It is recommended that the `Expires` tag is also updated if necessary.
173163

174164
* Add the `Preferred-Languages` key to `public/.well-known/security.txt`
@@ -177,7 +167,6 @@ It is recommended that the `Expires` tag is also updated if necessary.
177167
### Update coding standards
178168

179169
Dotkernel API uses `laminas/laminas-coding-standard` as its baseline ruleset to ensure adherence to PSR-1 and PSR-12.
180-
181170
As this package had a major release, the minimum version the API uses was also bumped.
182171

183172
* Bump `laminas/laminas-coding-standard` to `^3.0` in `composer.json`
@@ -193,7 +182,6 @@ As this package had a major release, the minimum version the API uses was also b
193182
### Update Qodana configuration
194183

195184
The Qodana code quality workflow has changed its default PHP version to 8.4, which is unsupported by Dotkernel API, resulting in errors.
196-
197185
The issue was fixed by restricting Qodana to the supported PHP versions.
198186

199187
* Update `.github/workflows/qodana_code_quality.yml`, specifying the supported PHP versions by adding the `strategy` key:
@@ -223,7 +211,6 @@ with:
223211
### Remove laminas/laminas-http
224212

225213
Prior to version 5.3, `laminas/laminas-http` was only used in 2 test files to assert if correct status codes were returned.
226-
227214
This dependency was removed, as the usage in tests was replaced with the existing `StatusCodeInterface`.
228215

229216
* Remove `laminas/laminas-http` from `composer.json`

mkdocs.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ nav:
2323
- "Test the Installation": v5/installation/test-the-installation.md
2424
- "FAQ": v5/installation/faq.md
2525
- Upgrading:
26-
- "Upgrading": v5/upgrading/upgrading.md
26+
- "Upgrade procedure": v5/upgrading/upgrading.md
2727
- "Upgrading 5.2 to 5.3": v5/upgrading/UPGRADE-5.3.md
2828
- Flow:
2929
- "Middleware Flow": v5/flow/middleware-flow.md

0 commit comments

Comments
 (0)