Skip to content

refactor(version): namespaceFromMajor, restes de la relecture de #112 - #114

Merged
PrestaEdit merged 5 commits into
devfrom
chore/version-namespace-helper
Oct 5, 2026
Merged

PrestaEdit merged 5 commits into
devfrom
chore/version-namespace-helper

Conversation

@PrestaEdit

Copy link
Copy Markdown
Contributor

Restes de la relecture finale de #112 (version par instance). Aucun changement de comportement.

  • Version::namespaceFromMajor() (statique) devient la seule règle « majeure → namespace » : '1.7' → '7', '1.6' → '6', '8' et '9' inchangés. getMajorVersion(namespace: true) et ImportPage::pageNamespaceVersion() l'utilisent ; le doublon substr($major, 2) disparaît.
  • Translations::getCatalog : la condition morte getMajorVersion() !== null est retirée. getMajorVersion() renvoie toujours une chaîne ou lève une exception.
  • VisualCheckpointTagTest : makePage() construit la page avec son patchVersion au lieu de forcer une majeure sur une page en 8.1.0. Les assertions de tag sont inchangées.

Tests : 543 verts sous PHP 8.4 et 8.1 (un test ajouté pour namespaceFromMajor).

L'app ignore les méthodes statiques dans l'éditeur (SuiteMetadata::methodsForClass) : la nouvelle méthode n'y apparaît pas.

🤖 Generated with Claude Code

PrestaEdit and others added 5 commits October 5, 2026 15:24
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e ; test mort retiré de getCatalog

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…rcer la majeure

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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