Skip to content

Conversation

zangra-dev
Copy link
Owner

No description provided.

Maxime added 18 commits August 27, 2025 14:48
…dern style

- Added type hints and strict return types across all Order classes
- Replaced constants by public const for clarity and PSR-12 compliance
- Unified XML (toXML/createFromXML) signatures and logic
- Simplified inheritance for Address, Customer, Receiver, Sender, etc.
- Enforced validation exceptions (length, values) consistently
- Improved code readability and maintainability
- Ajout de strict_types=1 sur toutes les classes
- Typage strict des propriétés, méthodes et retours
- Harmonisation des méthodes createFromXML avec retour typé
- Correction des constantes et renommages cohérents
- Simplification de Address et factorisation dans
PugoAddress/ParcelsDepotAddress
- Validation des valeurs (length, value, weight) via exceptions dédiées
- Refactor des classes Order, Box, Label et ProductConfiguration
- Nettoyage du code obsolète, docblocks et cohérence générale

Dossier Bpost finalisé et prêt à l’usage
- Added full type declarations and return types across Common namespace
- Improved BasicAttribute and ValidatedValue with stricter validation
methods
- Cleaned up redundant phpdoc in favor of native type hints
- Implemented specific BasicAttribute subclasses:
  - EmailAddressCharacteristic
  - Language
  - PhoneNumber
- Implemented ValidatedValue subclass:
  - LabelFormat
- Simplified ComplexAttribute as empty abstract base class
- Updated IAttribute interface with consistent naming
(toXml/createFromXml)
- Refactored XmlHelper with stricter handling and validation
- Migration vers PHP 8.4 avec typage strict et propriétés nullables
- Ajout de constantes explicites pour les jours (Day)
- Amélioration de la validation des jours et gestion des exceptions
- Ajout de types pour les coordonnées et collections (Poi, Service)
- Simplification des méthodes de parsing XML avec cast explicite
- Classes finales pour éviter l’héritage non prévu (Service)
@zangra-dev zangra-dev merged commit 3968e89 into master Sep 11, 2025
0 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant