Commit Graph

200 Commits

Author SHA1 Message Date
Serreau Jovann
0f7c752d9a test: ajout tests SetPasswordController, SonarBadgeController, StatusPageController, WebhookDocuSealController
tests/Controller/SetPasswordControllerTest.php (nouveau, 5 tests):
- testGetFormRendered: token valide, affiche le formulaire
- testTokenExpired: token invalide, affiche la page expired
- testPostPasswordTooShort: mot de passe < 8 caracteres, erreur
- testPostPasswordMismatch: confirmation differente, erreur
- testPostSuccess: mot de passe valide, flush + redirect 302

tests/Controller/SonarBadgeControllerTest.php (nouveau, 2 tests):
- testBadgeSuccess: metric valide, retourne SVG avec Content-Type image/svg+xml
- testBadgeInvalidMetric: metric invalide, retourne 404

tests/Controller/StatusPageControllerTest.php (reecrit, 2 tests):
- testIndexEmpty: aucune categorie, retourne 200
- testIndexWithServices: categorie avec service, QueryBuilder mocke
  pour les logs, retourne 200

tests/Controller/WebhookDocuSealControllerTest.php (nouveau, 9 tests):
- testUnauthorized: mauvais secret dans le header, retourne 401
- testInvalidPayload: JSON invalide, retourne 400
- testIgnoredDocType: doc_type autre que attestation, retourne ignored
- testEmptySecret: secret vide bypass la verification
- testFormViewedAttestationNotFound: attestation introuvable, retourne 404
- testFormViewedAttestationFound: attestation trouvee, retourne 200
- testFormStarted: evenement started, retourne 200
- testFormDeclined: evenement declined, retourne 200
- testUnknownEvent: evenement inconnu, retourne ignored

Resultat: 368 tests, 718 assertions, 0 failures, 0 notices

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-03 00:12:54 +02:00
Serreau Jovann
03d0ebbfba test: couverture 100% AnalyticsUniqId (29/29 methodes, 67/67 lignes)
tests/Entity/AnalyticsTest.php:
- 16 nouveaux tests pour les methodes statiques de parsing:
  - parseDeviceType: desktop (UA Windows), mobile (iPhone, Android),
    tablet (iPad)
  - parseOs: Windows (Windows NT), macOS (Macintosh), iOS (iPhone sans
    Mac OS X dans le UA), Android, Linux, unknown (retourne null)
  - parseBrowser: Chrome, Firefox, Edge (Edg/), Safari (sans Chrome),
    Opera (OPR/), unknown (retourne null)

AnalyticsUniqId passe de 89.66% (26/29) a 100% (29/29) methodes
et de 56.72% (38/67) a 100% (67/67) lignes

Resultat: 351 tests, 699 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-03 00:08:33 +02:00
Serreau Jovann
0f45ec4af0 test: ameliorer couverture AnalyticsUniqId avec verification fluent interface
tests/Entity/AnalyticsTest.php:
- testUniqIdSetters fusionne avec Screen et Optionals en un seul test
  testUniqIdSettersFluentAndGetters qui verifie que chaque setter
  retourne $this (fluent interface) puis que le getter retourne
  la bonne valeur pour les 10 proprietes: uid, hash, ipHash,
  userAgent, deviceType, screenWidth, screenHeight, language, os, browser
- testUniqIdUser: ajout verification fluent sur setUser() et test
  du set a null

Resultat: 336 tests, 683 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-03 00:05:57 +02:00
Serreau Jovann
25ddae58b2 test: ajout tests AnalyticsEvent, AnalyticsUniqId, AppLog, Attestation
tests/Entity/AnalyticsTest.php (nouveau, 8 tests):
- AnalyticsUniqId: constructor (id null, createdAt, events vide),
  setters (uid, hash, ipHash, userAgent, deviceType),
  screen (width, height), optionals (language, os, browser),
  user (nullable, set/get)
- AnalyticsEvent: constructor (id null, createdAt),
  setters (visitor, eventName, url, title, referrer),
  nullables (title, referrer null par defaut)

tests/Entity/AppLogTest.php (nouveau, 6 tests):
- testConstructor: method, url, route, action, user null, ip null,
  hmac non vide, createdAt
- testConstructorWithUser: user et ip passes au constructeur
- testVerifyHmacValid: meme secret retourne true
- testVerifyHmacInvalid: secret different retourne false
- testHmacDeterministic: hmac identique a chaque verification
- testDifferentDataProducesDifferentHmac: donnees differentes
  produisent des hmac differents

tests/Entity/AttestationTest.php (nouveau, 9 tests):
- testConstructor: type, ip, email, reference non vide, hmac non vide,
  status='pending', createdAt
- testReferencePrefix: 'access' contient ACC, 'deletion' contient DEL,
  'no_data' contient ABS dans la reference
- testPdfFiles: unsigned, signed, certificate get/set nullable
- testSubmitterId: get/set nullable
- testMarkAsSigned: status passe a 'signed'
- testMarkAsSent: status passe a 'sent'
- testVerifyValid: meme secret retourne true
- testVerifyInvalid: secret different retourne false
- testEmailTracking: null par defaut

Resultat: 338 tests, 670 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-03 00:03:53 +02:00
Serreau Jovann
48dd36e1ae test: ajout tests Customer, EmailTracking, MessengerLog, PriceAutomatic
tests/Entity/CustomerTest.php (nouveau, 11 tests):
- testConstructor: id null, user, state active, createdAt, updatedAt null
- testCodeComptable: get/set nullable
- testGenerateCodeComptable: genere un code non vide
- testNames: firstName, lastName, fullName
- testRaisonSociale: get/set
- testContact: email, phone
- testAddress: address, address2, zipCode, city
- testLegal: siret, rcs, numTva
- testStripe: stripeCustomerId get/set nullable
- testTypeCompany: set TYPE_SARL
- testState: setState suspended, isActive false

tests/Entity/EmailTrackingTest.php (nouveau, 4 tests):
- testConstructor: messageId, recipient, subject, htmlBody, attachments,
  state='sent', sentAt DateTimeImmutable, openedAt null
- testConstructorWithoutOptionals: htmlBody et attachments null
- testMarkAsOpened: state='opened', openedAt set
- testMarkAsOpenedOnlyOnce: deuxieme appel ne change pas openedAt

tests/Entity/MessengerLogTest.php (nouveau, 4 tests):
- testConstructor: tous les champs avec valeurs, status='failed',
  createdAt et failedAt DateTimeImmutable
- testConstructorMinimal: messageBody, stackTrace, transportName null,
  retryCount 0
- testSetStatus: change le status
- testMarkAsResolved: status passe a 'resolved'

tests/Entity/PriceAutomaticTest.php (nouveau, 2 tests):
- testGettersSetters: tous les champs avec valeurs
- testNullables: description, stripeId, stripeAbonnementId null

Resultat: 315 tests, 605 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-03 00:00:43 +02:00
Serreau Jovann
c0b31fb93d test: ajout tests User, Revendeur, Service et ServiceCategory
tests/Entity/UserExtendedTest.php (8 nouveaux tests):
- testSetIsEmailAuthEnabled: active l'auth email, verifie true sans
  keycloakId et false avec keycloakId
- testSetIsGoogleAuthEnabled: active Google Auth + secret
- testClearTempPassword: verifie clearTempPassword() met hasTempPassword a false
- testGenerateBackupCodes: genere 5 codes, verifie le count
- testSerializeUnserialize: serialise/deserialise, verifie l'email
- testCreatedAt: verifie le type DateTimeImmutable
- testUpdatedAt: null par defaut
- testEraseCredentials: appel sans erreur

tests/Entity/RevendeurTest.php (nouveau, 10 tests):
- testConstructor: id null, codeRevendeur, user, createdAt, isActive
- testRaisonSociale: get/set nullable
- testSiret: get/set
- testAddress: address, zipCode, city
- testContact: email, phone
- testStripeConnect: stripeConnectId, isUseStripe
- testStripeConnectState: state, statePayment, statePayout
- testContratPdf: unsigned, signed, audit, submitterId
- testActive: setIsActive false
- testUpdatedAt: get/set nullable

tests/Entity/ServiceTest.php (nouveau, 12 tests):
- testConstructor: tous les champs par defaut
- testSetName, testSetUrl, testSetStatus (avec message)
- testExternal: isExternal, externalType
- testPosition: get/set
- testStatusHistory: collection vide
- testCategory: verifie la relation
- testComputeUptimeRatioAllUp: 2 entries up = 100%
- testComputeUptimeRatioMixed: ratio entre 0 et 100
- testComputeUptimeRatioEmpty: pas d'historique = 100%

tests/Entity/ServiceCategoryTest.php (nouveau, 3 tests):
- testConstructor: name, slug, position 0, createdAt, services vide
- testSetName: modification du nom
- testSetPosition: modification de la position

Resultat: 294 tests, 534 assertions, 0 failures, 0 deprecations, 0 notices

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:57:05 +02:00
Serreau Jovann
a4eb9f6e2d fix: supprimer toutes les PHPUnit notices (40 → 0) et deprecations (9 → 0)
Probleme: PHPUnit 13 genere des notices quand createMock() est utilise
sans expects(), et des deprecations pour \$this->any() et ->with()
sans expects().

Corrections:
- tests/Service/AppLoggerServiceTest.php: suppression du setUp() partage,
  chaque test cree ses propres stubs/mocks selon ses besoins
  (bus createMock avec expects dans les tests log, createStub dans verify)
- tests/EventSubscriber/CsrfProtectionSubscriberTest.php: csrfTokenManager
  change de createMock a createStub (aucun expects utilise)
- tests/EventSubscriber/MessengerFailureSubscriberTest.php: em et mailer
  changes de createMock a createStub (aucun expects utilise)
- tests/EventListener/AdminLogListenerTest.php: testLogThrowsDoesNotBlock
  cree son propre stub local au lieu d'utiliser le mock du setUp,
  attribut #[AllowMockObjectsWithoutExpectations] ajoute pour le mock
  du setUp qui reste instancie mais non utilise dans ce test
- tests/Controller/SmallControllersTest.php: mocks sans expects remplaces
  par createStub via script automatise
- tests/Controller/MainControllersTest.php: idem
- tests/Controller/Admin/ClientsControllerTest.php: idem
- tests/MessageHandler/AnalyticsMessageHandlerTest.php: idem
- tests/EventListener/ExceptionListenerTest.php: idem

Resultat: 262 tests, 454 assertions, 0 failures, 0 deprecations, 0 notices

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:53:03 +02:00
Serreau Jovann
f6de3aa842 fix: supprimer toutes les deprecations PHPUnit (21 → 0)
Deprecation corrigee: "The any() invoked count expectation is deprecated"
- Remplacement de ->expects(\$this->any())->method() par ->method()
  sur les stubs dans 6 fichiers:
  tests/Controller/Admin/AdminControllersTest.php,
  tests/Controller/SmallControllersTest.php,
  tests/Controller/MainControllersTest.php,
  tests/EventListener/ExceptionListenerTest.php,
  tests/EventSubscriber/MessengerFailureSubscriberTest.php,
  tests/MessageHandler/AnalyticsMessageHandlerTest.php

Deprecation corrigee: "Using with() without expects() is deprecated"
- Suppression des ->with() sur les stubs qui n'ont pas de expects()
  dans SmallControllersTest, MessengerFailureSubscriberTest,
  AnalyticsMessageHandlerTest

AdminControllersTest::testStatusIndex:
- EntityManagerInterface change de createMock a createStub
  (pas d'expects() sur getRepository)

Resultat: 262 tests, 454 assertions, 0 failures, 0 deprecations

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:48:48 +02:00
Serreau Jovann
c3d3962b99 test: ajout tests ServiceStatusHistory, User (extended) et AdminLogListener
tests/Entity/ServiceStatusHistoryTest.php (nouveau, 2 tests):
- testConstructor: verifie id null, service, status, message et createdAt
- testConstructorWithoutMessage: message nullable a null par defaut

tests/Entity/UserExtendedTest.php (nouveau, 9 tests):
- testKeycloakId: get/set keycloakId nullable
- testTempPassword: get/set/has tempPassword nullable
- testGoogleAuthenticator: isEnabled (false par defaut, true quand
  secret set + isGoogleAuthEnabled true), get/set secret, getUsername
- testEmailAuth: isEnabled (false par defaut), getRecipient, get/set code
- testBackupCodes: get/set codes, isBackupCode, invalidateBackupCode
  (supprime du tableau)
- testAvatar: get/set avatar nullable
- testFullName: retourne "FirstName LastName"
- testRoles: contient toujours ROLE_USER, set/get custom roles
- testUserIdentifier: retourne l'email

tests/EventListener/AdminLogListenerTest.php (nouveau, 7 tests):
- testLogsAdminRoute: appelle log() pour une route app_admin_*
- testLogsAdminRouteWithUser: passe le User depuis le TokenStorage
- testIgnoresNonAdminRoute: ne log pas les routes non admin
- testIgnoresSubRequest: ne log pas les sous-requetes
- testIgnoresAjaxSearch: ne log pas les recherches AJAX
- testLogThrowsDoesNotBlock: exception dans log() ne bloque pas
  la requete (catch silencieux)
- testNoTokenReturnsNullUser: token null passe null comme user

Resultat global: 262 tests, 463 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:41:44 +02:00
Serreau Jovann
f068456308 test: ajout 6 tests KeycloakAdminService pour listGroups, createGroup, ensureRequiredGroups
tests/Service/KeycloakAdminServiceTest.php:
- testListGroups: liste les groupes, verifie 2 resultats et le nom du premier
- testCreateGroup: creation reussie retourne true (status 201)
- testCreateGroupFailure: creation echouee retourne false (status 409)
- testGetRequiredGroups: verifie que la liste contient siteconseil_admin,
  siteconseil_member, esy-web, esy-mail et fait 15 elements au total
- testEnsureRequiredGroupsAllExist: tous les groupes existent deja,
  retourne un tableau vide (aucune creation)
- testEnsureRequiredGroupsCreatesMissing: seul siteconseil_admin existe,
  les 14 autres sont crees, le resultat ne contient pas siteconseil_admin

Resultat global: 244 tests, 420 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:36:59 +02:00
Serreau Jovann
51bea93dbd test: ajout tests ClientsController (7 tests) + MeilisearchService price + AppLoggerService
tests/Controller/Admin/ClientsControllerTest.php (nouveau, 7 tests):
- testIndex: liste des clients avec repo vide
- testCreateGet: affichage du formulaire de creation
- testCreatePostInvalidData: soumission avec champs vides,
  UserManagementService lance InvalidArgumentException
- testCreatePostThrowsGenericError: soumission qui lance RuntimeException
- testSearchEmpty: recherche avec query vide retourne []
- testSearchWithQuery: recherche retourne les resultats Meilisearch
- testToggle: bascule actif/suspendu d'un client, verifie flush + redirect

Helper createController() avec RequestStack pour supporter les flash
messages et le router pour les redirections

Resultat global: 238 tests, 408 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:35:54 +02:00
Serreau Jovann
58f648a55b test: ajout tests pour MeilisearchService (price) et AppLoggerService
tests/Service/MeilisearchServiceTest.php:
- testIndexPriceSuccess: indexe un PriceAutomatic sans erreur
- testIndexPriceThrows: indexation echoue, log l'erreur sans throw
- testRemovePriceSuccess: supprime un price de l'index
- testRemovePriceThrows: suppression echoue, log sans throw
- testSearchPricesSuccess: recherche retourne les hits avec type/title
- testSearchPricesThrows: recherche echoue, retourne tableau vide
Total: 20 tests (6 nouveaux)

tests/Service/AppLoggerServiceTest.php (nouveau):
- testLogDispatchesMessage: verifie que log() dispatch un AppLogMessage
  via le bus Messenger avec method, url, route, action corrects
- testLogWithUser: log avec utilisateur, userId passe au message
- testLogPostAddsSubmission: les POST ajoutent "(soumission)" a l'action
- testLogUnknownRoute: route inconnue genere "Acces a {route}"
- testLogDirectPersistsImmediately: logDirect() appelle persist+flush
- testLogDirectWithUser: logDirect avec user et IP
- testVerifyLogValid: HMAC valide avec le meme secret
- testVerifyLogInvalid: HMAC invalide avec un secret different
Total: 8 tests

Resultat global: 231 tests, 398 assertions, 0 failures

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:33:36 +02:00
Serreau Jovann
f396b759f9 fix: corriger les 18 tests en echec apres le refactoring
tests/TestUserProvider.php (nouveau):
- Implementation de UserProviderInterface pour l'environnement test
- loadUserByIdentifier(), refreshUser(), supportsClass()
- Le service etait reference dans security.yaml when@test mais
  n'existait pas

config/services_test.yaml (nouveau):
- Enregistrement de App\Tests\TestUserProvider comme service public
  pour que le container test puisse le resoudre

tests/Controller/LegalControllerTest.php:
- Selecteurs CSS mis a jour: .border-red-600 remplace par .border-red-300
  et .border-green-600 par .border-green-300 (glassmorphism)

tests/Controller/Admin/AdminControllersTest.php:
- testSyncIndex(): ajout de PriceAutomaticRepository et
  StripeWebhookSecretRepository dans les arguments de
  SyncController::index() (4 arguments au lieu de 2)

tests/Controller/MainControllersTest.php:
- testForgotPasswordFullFlow(): sendEmail attendu 2 fois au lieu de 1
  (step 2 envoie le code, step 3 envoie la confirmation de changement)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:31:13 +02:00
Serreau Jovann
63c558e955 feat: passer les logs et le sync Meilisearch en asynchrone via Messenger
src/Message/AppLogMessage.php (nouveau):
- Message serialisable avec method, url, route, action, userId (int
  nullable au lieu de l'entite User), ip
- Dispatche via le bus Messenger pour traitement asynchrone par Redis

src/MessageHandler/AppLogMessageHandler.php (nouveau):
- Recharge le User par ID depuis le repository
- Cree l'AppLog avec le HMAC et persiste en BDD
- Execute en arriere-plan sans bloquer la requete HTTP

src/Service/AppLoggerService.php:
- log(): dispatch maintenant un AppLogMessage via le bus Messenger
  au lieu de persister directement (asynchrone)
- logDirect(): reste synchrone pour les suppressions de logs qui
  doivent etre tracees immediatement avant la reponse HTTP
- Injection de MessageBusInterface en plus de EntityManager

src/Message/MeilisearchSyncMessage.php (nouveau):
- Message avec type (customer/revendeur/price), entityId et action
  (index ou remove)
- Constantes TYPE_CUSTOMER, TYPE_REVENDEUR, TYPE_PRICE

src/MessageHandler/MeilisearchSyncMessageHandler.php (nouveau):
- Recharge l'entite par ID selon le type
- Appelle indexCustomer/indexRevendeur/indexPrice ou les methodes
  remove correspondantes sur MeilisearchService
- Execute en arriere-plan via Redis

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:26:30 +02:00
Serreau Jovann
33bd89e617 feat: page de verification en ligne des logs + QR code dans le PDF
src/Controller/LogVerifyController.php (nouveau):
- Route GET /admin/logs/verif/{id}/{hmac} accessible publiquement
- Le hmac dans l'URL est les 16 premiers caracteres du HMAC complet
  (suffisant pour identifier le log sans exposer la signature entiere)
- Verifie que le log existe et que le hmac partiel correspond
- Affiche la page de verification avec statut integrite

src/Controller/Admin/LogsController.php - pdf():
- Generation du QR code via Endroid\QrCode pointant vers l'URL
  de verification /admin/logs/verif/{id}/{hmac16}
- QR code encode en base64 et passe au template PDF

templates/admin/logs/verify.html.twig (nouveau):
- Page glassmorphism style attestation:
  - Log introuvable: bandeau rouge avec croix
  - Integrite verifiee: bandeau vert avec checkmark et message
    "La signature HMAC-SHA256 a ete verifiee avec succes"
  - Integrite compromise: bandeau rouge avec message d'alerte
- Tableau des details: ID, date, utilisateur, methode (badge colore),
  action, URL, route, IP
- Signature HMAC-SHA256 affichee en bas

templates/admin/logs/pdf.html.twig:
- Ajout du bloc verification avec QR code (72x72px) et lien URL
  identique au style des attestations RGPD (verify-box avec
  bordure indigo, QR a gauche, texte a droite)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:21:46 +02:00
Serreau Jovann
b2c6f0194d feat: suppression individuelle de logs + trace obligatoire des suppressions
src/Controller/Admin/LogsController.php:
- purge(): compte les logs avant suppression, supprime tous les logs,
  puis cree un nouveau log via logDirect() avec le message
  "Suppression de tous les logs (X entrees supprimees)" pour garder
  une trace de la purge meme apres suppression
- delete(): nouvelle route POST /admin/logs/{id}/delete, supprime un
  log individuel puis cree un log de trace avec le message
  "Suppression du log #X (action du date)" pour conserver l'historique

src/Service/AppLoggerService.php:
- logDirect(): nouvelle methode qui cree un AppLog avec une action
  personnalisee sans passer par le dictionnaire ROUTE_LABELS
  (utilisee pour les traces de suppression)

templates/admin/logs/index.html.twig:
- Bouton supprimer (croix rouge) ajoute a cote du bouton PDF
  sur chaque ligne du tableau, avec confirmation data-confirm

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:18:11 +02:00
Serreau Jovann
e093dae3ef feat: bouton purge des logs pour ROLE_ROOT
src/Controller/Admin/LogsController.php:
- Nouvelle route POST /admin/logs/purge: supprime tous les AppLog
  via requete DQL DELETE, accessible uniquement ROLE_ROOT
  (le controller entier est deja protege par ROLE_ROOT)

templates/admin/logs/index.html.twig:
- Bouton "Supprimer tous les logs" en haut a droite, rouge,
  visible uniquement pour ROLE_ROOT
- Confirmation data-confirm avant suppression

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:15:54 +02:00
Serreau Jovann
d3e76f00de fix: corriger HMAC des logs + PDF style attestation + pagination glassmorphism + purge logs
src/Entity/AppLog.php:
- createdAt initialise avec date('Y-m-d H:i:s') au lieu de
  new DateTimeImmutable() pour tronquer les microsecondes
  (PostgreSQL arrondit les microsecondes differemment de PHP,
  ce qui causait des HMAC invalides a la relecture)
- generateHmac(): format Y-m-d\TH:i:s sans microsecondes

templates/admin/logs/pdf.html.twig (reecrit):
- Meme style que les attestations RGPD (templates/pdf/rgpd_*.html.twig):
  banniere gold avec logo, doc-type badge indigo, titre italic uppercase,
  info-grid avec cellules bordure indigo, tableaux data avec header dark,
  bloc HMAC avec encadre vert/rouge, footer SARL SITECONSEIL
- Logo passe au template via base64

src/Controller/Admin/LogsController.php:
- pdf(): injection de kernel.project_dir, chargement du logo en base64
  et passage au template

src/Command/PurgeEmailTrackingCommand.php:
- Ajout de la purge des AppLog de plus de 90 jours (meme seuil
  que EmailTracking), affiche le nombre de logs supprimes

templates/components/pagination/glass.html.twig (nouveau):
- Template de pagination KnpPaginator style glassmorphism:
  boutons glass avec hover, page active en glass-gold,
  fleches precedent/suivant

config/packages/knp_paginator.yaml (nouveau):
- Configuration KnpPaginator pour utiliser le template glass

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:15:00 +02:00
Serreau Jovann
9c1ea29505 feat: systeme de logs d'activite admin avec HMAC + export PDF
src/Entity/AppLog.php (nouveau):
- id, user (ManyToOne nullable, SET NULL on delete), method (GET/POST/etc),
  url (500 chars), route (nom de la route Symfony), action (description
  lisible de l'action), ip (nullable), hmac (SHA-256), createdAt
- Index sur created_at pour les requetes paginées
- HMAC genere dans le constructeur avec payload:
  method|url|route|action|ip|userId|createdAt (microsecondes)
- verifyHmac(): verification timing-safe avec hash_equals
- Aucun setter sur les champs (immutable apres creation)

src/Repository/AppLogRepository.php (nouveau):
- createPaginatedQueryBuilder(): ORDER BY createdAt DESC avec jointure user

src/Service/AppLoggerService.php (nouveau):
- Dictionnaire ROUTE_LABELS: 30+ routes admin avec descriptions
  lisibles (ex: app_admin_clients_create → "Creation d'un client")
- log(): cree un AppLog avec l'action lisible, persiste et flush
- verifyLog(): verifie le HMAC d'un log
- Si la route n'est pas dans le dictionnaire, utilise "Acces a {route}"
- Ajoute "(soumission)" pour les POST

src/EventListener/AdminLogListener.php (nouveau):
- Ecoute KernelEvents::CONTROLLER avec priorite -10
- Intercepte toutes les requetes dont la route commence par app_admin_
- Ignore les requetes AJAX de recherche (evite le spam)
- Recupere l'utilisateur connecte via TokenStorage
- Appelle AppLoggerService::log() dans un try/catch
  (ne bloque jamais la requete si le logging echoue)

src/Controller/Admin/LogsController.php (nouveau):
- Route /admin/logs, ROLE_ROOT
- index(): pagination KnpPaginator (20 par page), verifie le HMAC
  de chaque log affiche
- pdf(): genere un PDF Dompdf avec toutes les infos du log
  + verification HMAC (CONFORME vert / ALTEREES rouge)

templates/admin/logs/index.html.twig (nouveau):
- Tableau glassmorphism: date, utilisateur, methode (badge colore),
  action, URL (tronquee), IP, colonne HMAC (rond vert/rouge),
  bouton PDF par ligne
- Pagination KnpPaginator en bas

templates/admin/logs/pdf.html.twig (nouveau):
- PDF A4 avec tableau d'informations du log
- Bloc HMAC avec fond vert "INTEGRITE VERIFIEE" ou rouge
  "INTEGRITE COMPROMISE" + signature HMAC complete
- Footer avec mention SARL SITECONSEIL

templates/admin/_layout.html.twig:
- Ajout lien "Logs" dans la sidebar Super Admin avec icone document

migrations/Version20260402211054.php:
- Table app_log avec FK user_id, index sur created_at

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:11:34 +02:00
Serreau Jovann
ecc9ec82b7 feat: ajout champs Stripe Connect state sur Revendeur
src/Entity/Revendeur.php:
- stripeConnectState: string(30) default 'not_started', etat global
  de l'onboarding Stripe Connect du revendeur
- stripeConnectStatePayment: string(30) nullable, etat de la capacite
  de reception de paiements (enabled/disabled/pending)
- stripeConnectStatePayout: string(30) nullable, etat de la capacite
  de versement des fonds (enabled/disabled/pending)
- Getters/setters fluent pour les 3 champs

migrations/Version20260402210431.php:
- Ajout colonnes stripe_connect_state DEFAULT 'not_started',
  stripe_connect_state_payment nullable,
  stripe_connect_state_payout nullable sur la table revendeur

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:04:41 +02:00
Serreau Jovann
ae560b1957 feat: affichage dynamique des compteurs sync/non sync sur la page admin/sync
src/Controller/Admin/SyncController.php - index():
- Calcul du nombre de tarifs synchronises avec Stripe (stripeId non vide)
  et non synchronises, passes au template
- Chargement des StripeWebhookSecret depuis la BDD pour afficher
  le nombre de webhooks configures

templates/admin/sync/index.html.twig:
- Bloc Tarifs Stripe: affiche "X sync" (vert) + "Y non sync" (rouge si > 0)
  + "/ Z total" (gris) au lieu du texte statique
- Bloc Webhooks Stripe: affiche "X/4 configure(s)" en vert si 4/4,
  orange si partiel, rouge si 0

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:02:57 +02:00
Serreau Jovann
0a7eb50e37 feat: ajout extension bcmath dans les Dockerfiles PHP
docker/php/dev/Dockerfile:
- Ajout de bcmath dans docker-php-ext-install (calculs decimaux precis
  pour les montants Stripe, TVA, totaux factures)

docker/php/prod/Dockerfile:
- Meme ajout de bcmath

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:01:33 +02:00
Serreau Jovann
7a9c9f3edf fix: remplacer bcmul/bccomp par round/float dans StripePriceService
src/Service/StripePriceService.php:
- bcmul() remplace par (int) round((float) $amountHt * 100) pour
  convertir le montant HT en centimes (bcmath non installe)
- bccomp() remplace par (float) cast pour comparer monthPrice a 0

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 23:01:08 +02:00
Serreau Jovann
bec008bdc1 refactor: stocker les secrets webhook Stripe en BDD au lieu de .env.local
src/Entity/StripeWebhookSecret.php (nouveau):
- Constantes TYPE_MAIN_LIGHT, TYPE_MAIN_INSTANT, TYPE_CONNECT_LIGHT,
  TYPE_CONNECT_INSTANT pour les 4 types de webhook
- type: string(30) unique, identifie le webhook (main_light, etc.)
- secret: string(255), le signing secret retourne par Stripe (whsec_xxx)
- endpointId: string nullable, l'ID de l'endpoint Stripe (we_xxx)
- createdAt: DateTimeImmutable

src/Repository/StripeWebhookSecretRepository.php (nouveau):
- findByType(): trouve un secret par type
- getSecret(): retourne directement la valeur du secret ou null

src/Controller/WebhookStripeController.php (reecrit):
- Les 4 routes lisent le secret depuis la BDD via
  StripeWebhookSecretRepository::getSecret() au lieu de variables d'env
- Retourne HTTP 503 si le secret n'est pas encore configure
- Plus besoin des variables STRIPE_WH_*_SECRET dans .env

src/Controller/Admin/SyncController.php:
- syncStripeWebhooks(): sauvegarde les secrets en BDD
  (cree ou met a jour StripeWebhookSecret par type)
- Suppression de saveSecretsToEnvLocal() (plus de modification .env.local)
- URL de base lue depuis WEBHOOK_BASE_URL (env)

.env:
- Suppression des 4 variables STRIPE_WH_*_SECRET (stockees en BDD)
- Ajout WEBHOOK_BASE_URL (vide par defaut)

docker/ngrok/sync.sh:
- Ecrit aussi WEBHOOK_BASE_URL en plus de OUTSIDE_URL

ansible/env.local.j2:
- WEBHOOK_BASE_URL=https://stripe.siteconseil.fr pour la prod

migrations/Version20260402205935.php:
- Table stripe_webhook_secret avec type unique, secret, endpoint_id

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:59:51 +02:00
Serreau Jovann
0ab2c8d0aa feat: creation automatique des webhooks Stripe + controllers de reception
src/Service/StripeWebhookService.php (nouveau):
- createAllWebhooks(baseUrl): cree 4 webhook endpoints dans Stripe:
  - /webhooks/stripe/main/light: customer.created/updated/deleted,
    product.created/updated, price.created/updated, invoice.created/
    finalized/payment_succeeded/payment_failed, subscription.created/
    updated/deleted
  - /webhooks/stripe/main/instant: checkout.session.completed/expired,
    payment_intent.succeeded/payment_failed, charge.succeeded/failed/
    refunded/dispute.created, invoice.paid/payment_failed,
    customer.subscription.trial_will_end/deleted
  - /webhooks/stripe/connect/light: account.updated/application.
    authorized/deauthorized, transfer.created/updated, payout.created/
    paid/failed
  - /webhooks/stripe/connect/instant: payment_intent.succeeded/
    payment_failed, charge.succeeded/failed/refunded,
    checkout.session.completed
- Verification si le webhook existe deja par URL avant creation
  (pas de doublon)
- listWebhooks(): liste tous les webhooks du compte
- deleteWebhook(): supprime un webhook par ID

src/Controller/WebhookStripeController.php (nouveau):
- 4 routes POST sans authentification (firewall webhooks):
  /webhooks/stripe/main/light, /webhooks/stripe/main/instant,
  /webhooks/stripe/connect/light, /webhooks/stripe/connect/instant
- Verification de signature Stripe via Stripe\Webhook::constructEvent()
- Log de chaque evenement avec channel, type et event_id
- TODO pour dispatcher vers les handlers specifiques

src/Controller/Admin/SyncController.php:
- Nouvelle route POST /admin/sync/stripe/webhooks: prend une URL
  de base en parametre, appelle createAllWebhooks(), affiche les
  resultats (cree/existe deja) en flash messages

templates/admin/sync/index.html.twig:
- Nouveau bloc "Webhooks Stripe" (bleu) avec champ URL de base
  (pre-rempli avec l'URL courante), bouton "Creer les webhooks",
  et liste des 4 endpoints avec leurs evenements

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:53:29 +02:00
Serreau Jovann
62718b5942 feat: sync automatique Stripe pour les tarifs + boutons sync admin
src/Service/StripePriceService.php (nouveau):
- Utilise Stripe SDK v20 (StripeClient) avec STRIPE_SK
- syncPrice(): pour chaque PriceAutomatic, cree ou retrouve le produit
  Stripe via metadata price_auto_type, puis cree le Stripe Price
  (unique et/ou recurrent selon monthPrice)
- ensureProduct(): cherche un produit existant par metadata, le cree
  sinon, met a jour nom/description si modifies
- createStripePrice(): cree un prix Stripe en centimes, avec
  tax_behavior=exclusive, recurring si monthPrice > 0 avec
  interval=month (ou year si period >= 12)
- updateStripePriceIfNeeded(): si le montant a change, archive l'ancien
  prix Stripe et en cree un nouveau (Stripe ne permet pas de modifier
  le montant d'un prix existant)
- syncAll(): synchronise tous les tarifs, retourne synced + errors

src/Service/TarificationService.php:
- Injection optionnelle de StripePriceService
- ensureDefaultPrices(): apres creation des tarifs, sync automatique
  avec Stripe (cree produits + prix) en plus de Meilisearch

src/Controller/Admin/TarificationController.php:
- edit(): apres mise a jour d'un tarif, sync automatique avec Stripe
  (cree/archive/recree les prix si montant change) et Meilisearch
- Flash d'erreur si Stripe echoue, les modifs locales sont sauvegardees

src/Controller/Admin/SyncController.php:
- Nouvelle route POST /admin/sync/stripe/prices: synchronise tous
  les tarifs avec Stripe via StripePriceService::syncAll()

templates/admin/sync/index.html.twig:
- Section "Stripe" avec bouton "Synchroniser Stripe" (violet)
  pour les tarifs, avec confirmation avant execution
- Section Meilisearch tarifs renommee "Tarifs - Meilisearch"

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:50:27 +02:00
Serreau Jovann
d2bf0279bd fix: cacher les champs Stripe Price ID de la page tarification
templates/admin/tarification/index.html.twig:
- Champs Stripe Price ID (unique) et Stripe Price ID (abonnement)
  remplaces par des input hidden pour conserver les valeurs existantes
  sans les afficher dans le formulaire

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:47:18 +02:00
Serreau Jovann
78c79a911c fix: mettre a jour le texte de synchronisation complete
templates/admin/sync/index.html.twig:
- Description changee de "Reindexe tous les clients et revendeurs"
  vers "Reindexe tous les clients, revendeurs et tarifs dans Meilisearch"

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:46:06 +02:00
Serreau Jovann
49d4cb702d feat: index Meilisearch price_auto + bouton sync tarifs + statuts Stripe
src/Service/MeilisearchService.php:
- indexPrice(): indexe un PriceAutomatic dans l'index price_auto
- removePrice(): supprime un tarif de l'index
- searchPrices(): recherche dans les tarifs
- setupIndexes(): ajout de l'index price_auto avec searchableAttributes
  (type, title, description) et filterableAttributes (type, period)
- serializePrice(): serialise id, type, title, description, priceHt,
  monthPrice, period, stripeId, stripeAbonnementId

src/Service/TarificationService.php:
- Injection optionnelle de MeilisearchService dans le constructeur
- ensureDefaultPrices(): apres flush des nouveaux tarifs, les indexe
  automatiquement dans Meilisearch

src/Controller/Admin/SyncController.php:
- Injection de PriceAutomaticRepository dans index() pour compter les tarifs
- Nouvelle route POST /admin/sync/prices: synchronise tous les tarifs
  dans l'index price_auto de Meilisearch
- syncAll(): inclut maintenant les tarifs dans la synchronisation complete

templates/admin/sync/index.html.twig:
- Nouveau bloc "Tarifs" avec index price_auto, compteur en base,
  bouton "Synchroniser" vert

templates/admin/tarification/index.html.twig:
- Header de chaque tarif: badges de statut Stripe
  - "Stripe OK" (vert) si stripeId renseigne, "Non sync" (rouge) sinon
  - "Abo OK" (vert) si stripeAbonnementId renseigne, "Abo non sync"
    (rouge) sinon (affiche uniquement si monthPrice != 0.00)
- Champ Stripe Price ID abonnement masque si monthPrice == 0.00
  (pas d'abonnement pour les paiements uniques)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:45:52 +02:00
Serreau Jovann
32aa5b0d78 feat: page admin tarification + TarificationService + champs PriceAutomatic
src/Entity/PriceAutomatic.php:
- type: ajout contrainte unique pour eviter les doublons
- monthPrice: decimal(10,2) default 0.00, prix mensuel recurrent
- period: smallint default 1, duree de la periode en mois
  (1=mensuel, 3=trimestriel, 12=annuel)
- stripeId: string nullable, ID du Stripe Price pour le paiement unique
- stripeAbonnementId: string nullable, ID du Stripe Price pour l'abonnement

src/Service/TarificationService.php (nouveau):
- Constante DEFAULT_PRICES avec 16 tarifs par defaut:
  esyweb_business (500€ + 100€/mois), esyweb_premium (3200€ + 100€/mois),
  ecommerce_business (999€ + 150€/mois), ecommerce_premium (5110€ + 150€/mois),
  esymail (50€ + 30€/mois), esymailer (50€ + 30€/mois),
  esydefender_pro (50€ + 60€/mois periode 3), esymeet (50€ + 30€/mois),
  esytchat (50€ + 15€/mois), esycreator (500€ + 100€/mois periode 3),
  ndd_depot (20€), ndd_renouvellement (20€/an), ndd_gestion (30€/an),
  ndd_reactivation (50€), formation_pack10h (500€), formation_heure (70€)
- ensureDefaultPrices(): verifie les tarifs existants, cree ceux manquants
- getAll(), getByType(), getDefaultTypes()

src/Controller/Admin/TarificationController.php (nouveau):
- Route /admin/tarification, ROLE_ROOT
- index(): appelle ensureDefaultPrices() pour creer les tarifs manquants
  automatiquement a chaque visite, affiche tous les tarifs editables
- edit(): met a jour titre, description, prixHt, monthPrice, period,
  stripeId, stripeAbonnementId via formulaire POST

templates/admin/tarification/index.html.twig (nouveau):
- Liste de tous les tarifs sous forme de cards glassmorphism
- Header dark avec titre, type (badge) et prix
- Formulaire d'edition inline: titre, prix unique, prix mensuel,
  periode (select 1/2/3/6/12 mois), Stripe Price ID unique,
  Stripe Price ID abonnement, description (textarea)
- Bouton enregistrer par tarif

templates/admin/_layout.html.twig:
- Ajout lien "Tarification" dans la sidebar Super Admin avec icone dollar

migrations/Version20260402204223.php:
- Ajout colonnes month_price, period, stripe_id, stripe_abonnement_id
  sur price_automatic + index unique sur type

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:42:43 +02:00
Serreau Jovann
8136475356 feat: ajout state et raisonMessage sur Devis
src/Entity/Devis.php:
- Constantes STATE_CREATED, STATE_SEND, STATE_ACCEPTED, STATE_REFUSED,
  STATE_CANCEL pour les 5 etats possibles du devis
- state: string(20) default 'created', cycle de vie du devis
  (created → send → accepted/refused/cancel)
- raisonMessage: text nullable, motif de refus ou annulation

migrations/Version20260402203711.php:
- Ajout colonnes state VARCHAR(20) DEFAULT 'created' et
  raison_message TEXT nullable sur la table devis

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:37:20 +02:00
Serreau Jovann
42fe3257a1 feat: ajout totalHt, totalTva, totalTtc sur Devis
src/Entity/Devis.php:
- totalHt: decimal(10,2) default 0.00, montant hors taxes du devis
- totalTva: decimal(10,2) default 0.00, montant de la TVA
- totalTtc: decimal(10,2) default 0.00, montant toutes taxes comprises
- Getters/setters pour les 3 champs

migrations/Version20260402203631.php:
- Ajout colonnes total_ht, total_tva, total_ttc sur la table devis

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:36:38 +02:00
Serreau Jovann
1c82da99f3 fix: utiliser /uploads/devis comme URL pour les PDFs de devis
src/Controller/DevisPdfController.php:
- Route changee de /devis/pdf/{id}/{type} vers /uploads/devis/{id}/{type}
  pour garder une URL coherente avec le dossier uploads

config/packages/vich_uploader.yaml:
- uri_prefix change de /devis/pdf vers /uploads/devis

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:35:11 +02:00
Serreau Jovann
88053611a4 feat: controller securise pour servir les PDFs de devis + stockage prive
src/Controller/DevisPdfController.php (nouveau):
- Route /devis/pdf/{id}/{type} avec type = unsigned|signed|audit
- Requiert ROLE_USER minimum
- checkAccess(): les ROLE_EMPLOYE ont toujours acces,
  pour les clients un TODO est prepare pour verifier que le
  client connecte est bien lie au devis (a implementer quand
  la relation Customer sera ajoutee sur Devis)
- Sert le fichier via BinaryFileResponse en inline (affichage
  dans le navigateur) avec nom de telechargement propre
  (ex: signed-04-2026-00001.pdf)

config/packages/vich_uploader.yaml:
- Mapping devis_pdf: stockage deplace de public/uploads/devis
  vers var/uploads/devis (hors du dossier public, inaccessible
  directement par URL)
- uri_prefix change en /devis/pdf (pointe vers le controller)

config/packages/security.yaml:
- Suppression de la regle access_control sur /uploads/devis
  (remplacee par le controller avec verification plus fine)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:34:54 +02:00
Serreau Jovann
09148b5b33 feat: ajout champs submitter + PDFs Vich sur Devis + protection uploads
src/Entity/Devis.php:
- submitterSiteconseilId (int nullable): ID du soumetteur cote SITECONSEIL
  dans DocuSeal apres signature
- submitterCustomerId (int nullable): ID du soumetteur cote client
  dans DocuSeal apres signature
- unsignedPdf (string nullable) + unsignedPdfFile (Vich): PDF non signe
- signedPdf (string nullable) + signedPdfFile (Vich): PDF signe
- auditPdf (string nullable) + auditPdfFile (Vich): certificat d'audit
- updatedAt (DateTimeImmutable nullable): mis a jour automatiquement
  a chaque upload de fichier via les setters *File()
- Annotation #[Vich\Uploadable] sur la classe
- Les 3 champs fichier utilisent le mapping 'devis_pdf'

config/packages/vich_uploader.yaml:
- Nouveau mapping devis_pdf: stockage dans public/uploads/devis
  avec SmartUniqueNamer pour eviter les collisions de noms

config/packages/security.yaml:
- Nouvelle regle access_control: /uploads/devis requiert ROLE_USER
  (empeche l'acces aux PDF de devis sans etre connecte)

migrations/Version20260402203334.php:
- Ajout colonnes submitter_siteconseil_id, submitter_customer_id,
  unsigned_pdf, signed_pdf, audit_pdf, updated_at sur la table devis

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:33:47 +02:00
Serreau Jovann
cdd5c656a9 feat: ajout signature HMAC SHA-256 sur Devis, Advert et Facture
src/Entity/Devis.php:
- Nouveau champ hmac (string 128), genere dans le constructeur
- Constructeur prend maintenant le hmacSecret en 2eme parametre
- generateHmac(): payload = "devis|numOrder|createdAt" signe avec APP_SECRET
- verifyHmac(): verification par hash_equals (timing-safe)

src/Entity/Advert.php:
- Nouveau champ hmac (string 128), genere dans le constructeur
- Constructeur prend maintenant le hmacSecret en 2eme parametre
- generateHmac(): payload = "advert|numOrder|createdAt" signe avec APP_SECRET
- verifyHmac(): verification par hash_equals

src/Entity/Facture.php:
- Nouveau champ hmac (string 128), genere dans le constructeur
- Constructeur prend maintenant le hmacSecret en 2eme parametre
- generateHmac(): payload = "facture|numOrder|splitIndex|createdAt"
  signe avec APP_SECRET (inclut splitIndex pour differencier les splits)
- verifyHmac(): verification par hash_equals

src/Service/DevisService.php, AdvertService.php, FactureService.php:
- Injection de APP_SECRET via #[Autowire('%env(APP_SECRET)%')]
- Passage du hmacSecret aux constructeurs des entites

migrations/Version20260402203207.php:
- Ajout colonne hmac VARCHAR(128) NOT NULL sur devis, advert, facture

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:32:18 +02:00
Serreau Jovann
5b0e4707f7 feat: ajout des services DevisService, AdvertService et FactureService
src/Service/DevisService.php (nouveau):
- create(): genere un OrderNumber via OrderNumberService::generateAndUse(),
  cree le Devis, persiste et flush. Retourne le Devis.

src/Service/AdvertService.php (nouveau):
- create(?Devis): si un devis est passe, reutilise son OrderNumber
  (meme numero pour devis et advert) et lie l'advert au devis.
  Si pas de devis, genere un nouveau OrderNumber.
- createFromDevis(Devis): raccourci qui appelle create() avec le devis.

src/Service/FactureService.php (nouveau):
- create(?Advert): si un advert est passe, appelle createFromAdvert().
  Si pas d'advert, genere un nouveau OrderNumber (facture seule).
- createFromAdvert(Advert): reutilise le OrderNumber de l'advert
  (meme numero). Gere le splitIndex automatiquement:
  - 1ere facture sur un advert: splitIndex=0 (pas de suffixe)
  - 2eme facture: la 1ere passe a splitIndex=1, la nouvelle a splitIndex=2
    (ex: 04/2026-00001-1 et 04/2026-00001-2)
  - 3eme facture: splitIndex=3 (04/2026-00001-3)

Exemples d'utilisation:
- Facture seule: $factureService->create()
- Devis seul: $devisService->create()
- Advert seul: $advertService->create()
- Devis → Advert: $advertService->createFromDevis($devis)
- Advert → Facture: $factureService->createFromAdvert($advert)
- Advert → 3 Factures: 3x createFromAdvert() → -1, -2, -3

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:29:54 +02:00
Serreau Jovann
da7f46f7e9 refactor: renommer Order en Facture + meme OrderNumber partage entre Devis/Advert/Facture
Changement de modele:
- Le meme OrderNumber est partage entre Devis, Advert et Facture
  (ex: 04/2026-00001 pour les 3)
- Les relations OrderNumber passent de OneToOne a ManyToOne pour
  permettre le partage du meme numero

src/Entity/Order.php supprime, remplace par:
src/Entity/Facture.php (nouveau):
- orderNumber: ManyToOne vers OrderNumber (meme numero que l'Advert parent)
- advert: ManyToOne vers Advert (nullable)
- splitIndex: smallint, suffixe pour factures multiples sur un meme advert
  (0 = pas de suffixe, 1 = -1, 2 = -2, etc.)
- getInvoiceNumber(): retourne le numero complet avec suffixe si splitIndex > 0
  (ex: 04/2026-00001 ou 04/2026-00001-2)

src/Entity/Devis.php:
- orderNumber: OneToOne remplace par ManyToOne vers OrderNumber
- adverts: OneToMany vers Advert (inchange)

src/Entity/Advert.php:
- orderNumber: OneToOne remplace par ManyToOne vers OrderNumber
- orders: renomme en factures, OneToMany vers Facture

src/Repository/OrderRepository.php supprime, remplace par:
src/Repository/FactureRepository.php (nouveau)

migrations/Version20260402202809.php:
- Suppression table `order`, creation table facture
- Modification des contraintes unique sur devis et advert
  (unique index supprime car ManyToOne)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:28:30 +02:00
Serreau Jovann
a6e529e643 feat: ajout entity PriceAutomatic pour les tarifs automatiques
src/Entity/PriceAutomatic.php (nouveau):
- id: int auto-increment
- type: string(50), categorie du tarif (ex: esy-web, esy-mail, ndd)
- title: string(255), intitule du tarif
- description: text nullable, detail du tarif
- priceHt: decimal(10,2), prix hors taxes

src/Repository/PriceAutomaticRepository.php (nouveau)

migrations/Version20260402202703.php:
- Table price_automatic avec colonnes type, title, description, price_ht

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:27:12 +02:00
Serreau Jovann
3bda43c72f feat: ajout entities Devis, Advert et Order liees a OrderNumber
Relations:
- Devis → OrderNumber (OneToOne): chaque devis a un numero unique
- Devis → Advert (OneToMany): un devis peut generer plusieurs factures pro forma
- Advert → OrderNumber (OneToOne): chaque facture pro forma a un numero unique
- Advert → Devis (ManyToOne, nullable): un advert peut exister sans devis
- Advert → Order (OneToMany): un advert peut generer plusieurs factures
- Order → OrderNumber (OneToOne): chaque facture a un numero unique
- Order → Advert (ManyToOne, nullable): une facture peut exister sans advert

Chaine: Devis → Advert → Order
Si un advert genere plusieurs factures, le champ splitIndex (smallint)
ajoute un suffixe -X au numero (ex: 04/2026-00001-1, 04/2026-00001-2).
La methode getInvoiceNumber() retourne le numero complet avec suffixe.

src/Entity/Devis.php: id, orderNumber (OneToOne), createdAt, adverts (OneToMany)
src/Entity/Advert.php: id, orderNumber (OneToOne), devis (ManyToOne nullable),
  createdAt, orders (OneToMany)
src/Entity/Order.php: id, orderNumber (OneToOne), advert (ManyToOne nullable),
  splitIndex (smallint default 0), createdAt, getInvoiceNumber()
  Table nommee `order` (mot reserve SQL, echappee avec backticks)

src/Repository/DevisRepository.php, AdvertRepository.php, OrderRepository.php

migrations/Version20260402202554.php: tables devis, advert, `order` avec
  foreign keys vers order_number et relations entre elles

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:26:15 +02:00
Serreau Jovann
cacd3ac66c feat: page admin de gestion de la numerotation des commandes
src/Controller/Admin/OrderNumberController.php (nouveau):
- Route /admin/numerotation, accessible ROLE_ROOT uniquement
- index(): affiche le prochain numero via OrderNumberService::preview()
  et les 20 derniers numeros generes (ORDER BY id DESC)
- update(): modifie le prochain numero en creant une entree placeholder
  avec le numero precedent (N-1) marque comme utilise, pour que le
  prochain generate() retourne le numero souhaite
  - Validation du format MM/YYYY-XXXXX via regex
  - Verification que le numero n'existe pas deja
  - Verification que le numero est au minimum 00001

templates/admin/order_number/index.html.twig (nouveau):
- Section "Prochain numero" : affiche le prochain numero en gros (gold)
  avec formulaire pour le modifier (input avec pattern regex,
  placeholder MM/YYYY-XXXXX, explication de l'utilite)
- Section "Derniers numeros generes" : tableau avec numero (font-mono),
  date de creation, statut (Utilise vert / Reserve gris)
- Design glassmorphism (glass, input-glass, btn-gold, glass-dark header)

templates/admin/_layout.html.twig:
- Ajout du lien "Numerotation" dans la sidebar Super Admin avec
  icone hash (#), route app_admin_order_number, style active-danger

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:23:07 +02:00
Serreau Jovann
85220c6200 feat: ajout OrderNumberService pour generer les numeros de commande
src/Service/OrderNumberService.php (nouveau):
- Format: MM/YYYY-XXXXX (ex: 04/2026-00001, 04/2026-00002, etc.)
- Le compteur XXXXX est par mois: reset a 00001 a chaque nouveau mois
- generate(): cherche le dernier numOrder du mois courant via LIKE
  'MM/YYYY-%' ORDER BY DESC, extrait le compteur, incremente de 1,
  pad sur 5 chiffres, persiste et flush le nouvel OrderNumber
- generateAndUse(): genere + markAsUsed() en une operation
- preview(): retourne le prochain numero sans le creer en BDD

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:20:45 +02:00
Serreau Jovann
423ee779e0 feat: ajout entity OrderNumber pour la gestion des numeros de commande
src/Entity/OrderNumber.php (nouveau):
- id: int auto-increment
- numOrder: string(50) unique, le numero de commande
- createdAt: DateTimeImmutable, date de creation (auto dans le constructeur)
- isUsed: bool, false par defaut, marque via markAsUsed()

src/Repository/OrderNumberRepository.php (nouveau):
- Repository Doctrine standard pour OrderNumber

migrations/Version20260402201935.php:
- Table order_number avec index unique sur num_order
- created_at TIMESTAMP, is_used BOOLEAN

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:19:54 +02:00
Serreau Jovann
a9057374d4 fix: deplacer le pool dns_infra_cache dans le bon fichier cache.yaml
config/packages/cache.yaml:
- Ajout du pool dns_infra_cache avec adapter cache.app et TTL 3600s
- Le pool etait dans config/packages/packages/cache.yaml qui est
  surcharge par config/packages/cache.yaml, donc le service
  dns_infra_cache n'existait pas

config/packages/packages/cache.yaml:
- Suppression du pool dns_infra_cache (doublon)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:17:29 +02:00
Serreau Jovann
28b84f09d4 feat: cache DNS report + purge EmailTracking + crons mis a jour
src/Controller/DnsReportController.php:
- Injection du pool cache dns_infra_cache via #[Autowire]
- Les resultats des checks sont caches avec la cle dns_infra_check_{token}
  pendant 1 heure (3600s) pour eviter de rappeler toutes les APIs
  (Cloudflare, AWS SES, Mailcow, RDAP, dig) a chaque rechargement
- La date du rapport est stockee dans le cache au format ISO 8601

config/packages/packages/cache.yaml:
- Nouveau pool dns_infra_cache sur Redis avec default_lifetime 3600s

src/Command/PurgeEmailTrackingCommand.php (nouveau):
- Commande app:email-tracking:purge qui supprime les EmailTracking
  dont sentAt est anterieur au seuil (90 jours par defaut)
- Option --days pour changer la retention (ex: --days=30)
- Utilise une requete DQL DELETE pour performance

ansible/deploy.yml.disabled:
- Nouveau cron "crm-siteconseil email-tracking purge": tous les jours
  a 5h du matin, supprime les EmailTracking de plus de 90 jours

docker/cron/entrypoint.sh:
- Liste complete des taches cron mise a jour avec:
  - */5 min: expire-pending, infra:snapshot
  - */15 min: services:check
  - toutes les heures: monitor:messenger
  - toutes les 2h: dns:check
  - toutes les 6h: stripe:sync
  - 3h: meilisearch consistency
  - 4h: attestations clean
  - 5h: email-tracking purge
  - 6h: cloudflare clean

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:16:50 +02:00
Serreau Jovann
c7a1c1b39f fix: corriger le serveur RDAP pour les domaines .dev
src/Service/DnsCheckService.php:
- RDAP_SERVERS['dev']: URL corrigee de rdap.nic.google (n'existe pas)
  vers pubapi.registry.google/rdap/domain/ (endpoint officiel Google
  Registry trouve via le bootstrap IANA data.iana.org/rdap/dns.json)
- esy-web.dev retourne maintenant: expiration 2028-01-08,
  NS candy.ns.cloudflare.com + nero.ns.cloudflare.com

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:13:34 +02:00
Serreau Jovann
a02e84eb26 feat: ajout verification WHOIS (nameservers Cloudflare + expiration domaine)
src/Service/DnsCheckService.php:
- Nouvelle methode checkWhois() qui verifie pour chaque domaine:
  - Nameservers via dig @1.1.1.1 NS: verifie que les NS contiennent
    *.ns.cloudflare.com, erreur si non Cloudflare
  - Expiration via RDAP (protocole standard ICANN remplacant whois):
    erreur si expire ou moins de 30 jours restants,
    warning si moins de 90 jours, OK sinon
- Constante RDAP_SERVERS avec les serveurs RDAP par TLD
  (.fr → rdap.nic.fr, .com → rdap.verisign.com, .dev → rdap.nic.google,
  .org/.eu → rdap.org, fallback rdap.org)
- Methode queryRdap() qui interroge le serveur RDAP du TLD avec
  Accept: application/rdap+json et timeout 10s
- Methode getNsRecords() qui parse la sortie dig NS
- Ajout du type NS dans le fallback dns_get_record

src/Command/CheckDnsCommand.php:
- Appel de checkWhois() apres checkBounce() pour chaque domaine

src/Controller/DnsReportController.php:
- Meme appel checkWhois() dans le rapport web

Seuils d'alerte:
- < 0 jours : erreur "EXPIRE"
- < 30 jours : erreur "X jours restants"
- < 90 jours : warning "X jours restants"
- > 90 jours : OK "X jours restants"

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:11:32 +02:00
Serreau Jovann
f3ebcdaa85 fix: supprimer la ligne Mailcow DKIM du rapport DNS
src/Command/CheckDnsCommand.php:
- Suppression du check 'Mailcow DKIM: Ignore (DKIM via AWS SES)'
  et du succes associe (le DKIM est deja verifie par les 3 checks
  AWS SES DKIM CNAME, cette ligne etait redondante)

src/Controller/DnsReportController.php:
- Meme suppression dans le controller du rapport web

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:08:13 +02:00
Serreau Jovann
08f18d42fb fix: afficher les vraies valeurs dig et Cloudflare pour tous les sous-domaines
src/Command/CheckDnsCommand.php:
- Nouvelle methode getActualDnsValue() qui retourne la vraie valeur DNS
  selon le type (MX: liste des targets, CNAME: target, TXT: contenu,
  SRV: target:port) au lieu de "Trouve"/"Non trouve"
- Methodes getFirstTxtValue() et getSrvValue() pour recuperer les
  valeurs reelles TXT et SRV via dig @1.1.1.1
- Les checks Mailcow DNS utilisent maintenant getActualDnsValue()
  dans la colonne dig: mail.esy-web.dev, v=STSv1;id=..., etc.

src/Controller/DnsReportController.php (reecrit completement):
- Memes methodes enrichWithCloudflare() et enrichLastCheck() que la commande
- checkAwsSes(): enrichissement CF sur chaque DKIM CNAME
  ({token}._domainkey.{domain}), MAIL FROM MX et MAIL FROM TXT
- checkMailcow(): enrichissement CF sur chaque record attendu
  (autodiscover, autoconfig, SRV, _mta-sts, mta-sts)
- getActualDnsValue(), getMxValues(), getFirstTxtValue(), getSrvValue()
  pour afficher les vraies valeurs dig dans la colonne Dig
- Bounce: enrichissement CF sur bounce.{domain} MX
- La page web /email/configuration/{token} affiche maintenant les
  3 colonnes correctement remplies pour tous les enregistrements

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:07:24 +02:00
Serreau Jovann
cf85c6b25a feat: enrichir la colonne Cloudflare pour tous les sous-domaines DNS
src/Command/CheckDnsCommand.php:
- Nouvelle methode enrichLastCheck() qui cherche un record dans les
  records Cloudflare par nom+type et remplit cloudflare/cf_status
  sur le dernier check ajoute au tableau
- Bounce: enrichissement CF sur bounce.{domain} type MX
- AWS SES DKIM CNAME: enrichissement CF sur {token}._domainkey.{domain}
  type CNAME pour chacun des 3 tokens DKIM
- AWS SES MAIL FROM MX: enrichissement CF sur bounce.{domain} type MX
- AWS SES MAIL FROM TXT: enrichissement CF sur bounce.{domain} type TXT
- Mailcow DNS: enrichissement CF pour chaque record attendu
  (autodiscover CNAME, autoconfig CNAME, _autodiscover._tcp SRV,
  _mta-sts TXT, mta-sts CNAME) avec le nom et type exacts
- checkAwsSes() et checkMailcow() recoivent maintenant $cfRecords
  en parametre pour effectuer les enrichissements

La colonne Cloudflare du rapport web et email affiche maintenant
la valeur presente dans la zone CF pour bounce, DKIM CNAME,
MAIL FROM, autodiscover, autoconfig, mta-sts au lieu de "Non trouve"

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-02 22:04:43 +02:00