Skip to content

Feature ETP-3674: Hibernate 6 and Jakarta EE migration core fixes - #41

Open
RomanMagnoli wants to merge 3 commits into
epic/ETP-2587-Y27from
feature/ETP-3674-Y27
Open

Feature ETP-3674: Hibernate 6 and Jakarta EE migration core fixes#41
RomanMagnoli wants to merge 3 commits into
epic/ETP-2587-Y27from
feature/ETP-3674-Y27

Conversation

@RomanMagnoli

Copy link
Copy Markdown
Contributor

Summary

  • Migrate from Hibernate 5 to Hibernate 6
  • Migrate from javax to Jakarta EE

Test plan

  • Verify module compiles correctly with Hibernate 6
  • Verify javax imports replaced with jakarta
  • Run integration tests

🤖 Generated with Claude Code

Comment thread pipelines/Jenkinsfile
Comment thread pipelines/Jenkinsfile
Comment thread pipelines/Jenkinsfile
@sonarscanetendo

Copy link
Copy Markdown

Passed Quality Gate passed

Issues

Measures

Project ID: etendosoftware_com.etendoerp.docker_AZDFudzPGnpq8rkD_xyH

View in SonarQube

<!--36D2DDA4D6CE411C8A6096D0C13C7A7F--> <NAME><![CDATA[Docker Help Utils]]></NAME>
<!--36D2DDA4D6CE411C8A6096D0C13C7A7F--> <VERSION><![CDATA[3.0.2]]></VERSION>
<!--36D2DDA4D6CE411C8A6096D0C13C7A7F--> <VERSION><![CDATA[3.0.2-SNAPSHOT]]></VERSION>
<!--36D2DDA4D6CE411C8A6096D0C13C7A7F--> <DESCRIPTION><![CDATA[This module provides the infrastructure to distribute and manage Docker containers within the Etendo Classic environment, using a standardized method and Gradle tasks]]></DESCRIPTION>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warning

⚠️ Blocking Issue

Remove the '-SNAPSHOT' suffix and use a purely numeric version (e.g., 3.0.3 or 3.0.2). If you need to mark a development version, consider using the 'In Development' flag or managing the snapshot status via the Gradle build configuration instead of the database metadata.

<!--36D2DDA4D6CE411C8A6096D0C13C7A7F-->  <VERSION><![CDATA[3.0.3]]></VERSION>

Rationale: The Etendo framework's version parsing logic (e.g., OpenbravoVersion class) splits the version string by dots and attempts to convert each part to an integer using Integer.valueOf() or Integer.parseInt(). Including a non-numeric suffix like '-SNAPSHOT' will trigger a NumberFormatException, leading to runtime failures in module management and dependency resolution.

@etendobot etendobot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes requested by agent. Please resolve blocking issues.

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.

2 participants