dansmaculotte / dansmaculotte/monetico-php

Fatal error après mise à jour en PHP 8.3

Open
#35 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
PHP
Stars
14
Forks
19
PR merge metrics
No merged PRs in 30d

Description

Bien le bonjour à vous !

Tout d'abord merci pour le travail réalisé 😊

Petite requête, car je me casse les dents pour mettre les fichiers à jour, mais rien à faire, je me retrouve avec une signature MAC qui n'est pas correcte.
Dans un premier temps, je me retrouve avec un message :

```
Fatal error: Uncaught DansMaCulotte\Monetico\Exceptions\Exception: dateTime value is not a DateTime object in /var/www/public/trouble-clic.com_v2/shop/monetico/dansmaculotte/monetico-php/src/Exceptions/Exception.php:30
Stack trace:
#0 /var/www/public/trouble-clic.com_v2/shop/monetico/dansmaculotte/monetico-php/src/Requests/PurchaseRequest.php(137): DansMaCulotte\Monetico\Exceptions\Exception::invalidDatetime()
#1 /var/www/public/trouble-clic.com_v2/shop/monetico/dansmaculotte/monetico-php/src/Requests/PurchaseRequest.php(120): DansMaCulotte\Monetico\Requests\PurchaseRequest->validate()
#2 /var/www/public/trouble-clic.com_v2/shop/paiements.php(411): DansMaCulotte\Monetico\Requests\PurchaseRequest->__construct()
#3 {main}
thrown in /var/www/public/trouble-clic.com_v2/shop/monetico/dansmaculotte/monetico-php/src/Exceptions/Exception.php on line 30

```

J'ai pu outrepasser cette erreur en commentant cette ces lignes :

```
if (!$this->dateTime instanceof DateTime) {
throw Exception::invalidDatetime();
}
```

Et en modifiant celle ci :
`'date' => $this->dateTime->format(self::DATETIME_FORMAT),`
Par
`'date' => date('d/m/Y:H:i:s'),`

Mais mainetnant je me retrouve avec un MAC incorrect
![Capture d'écran 2024-01-17 221038](https://github.com/dansmaculotte/monetico-php/assets/156960136/c4f9954e-8158-44d2-9d71-1776286a4fc0)

Auriez vous une solution ou correction à apporter ?
Merci en tous cas pour le temps que vous y consacrerez 😘

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.