Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

InvalidPathException thrown when parsing typeAliases elements using package as sub-tag #1974

Closed
akamonk opened this issue Jul 9, 2020 · 4 comments · Fixed by #1975
Closed
Assignees
Labels
Milestone

Comments

@akamonk
Copy link

akamonk commented Jul 9, 2020

MyBatis version

3.5.4

Database vendor and version

MySQL v8.0.20

JDK version and OS

v9.0.1 & MacOS

Test case or example project

https://github.com/akamonk/mybatis-demo

Steps to reproduce

Use JDK9 as the running environment to build a named module. the module-info.java like as follows:

open module mybatis.demo {
    requires org.mybatis;
    requires java.sql;
}

When configuring, use package as a sub-tag of typeAliases tag.

<typeAliases>
      <package name="com.xxx.xxx.entity" />  
</typeAliases>

then, build the SqlSessionFactory. When parsing the typeAliases element, an InvalidPathException was thrown.

My attempt:

When I removed module-info.java from the module, I got the expected result.

Expected result

Successfully constructed SqlSessionFactory

Actual result

InvalidPathException thrown

@harawata
Copy link
Member

harawata commented Jul 9, 2020

Thank you so much for the report and the repro, @akamonk !
I could easily reproduce the exception and plan to fix it as soon as possible.

@harawata
Copy link
Member

@akamonk ,

It should be fixed in the latest 3.5.6-SNAPSHOT.
Please let me know if you experience any further issue.
Thank you again for your contribution!

@TakeshiYonezawa
Copy link

Hi harawata.
When will latest 3.5.6 be released?

@harawata
Copy link
Member

harawata commented Oct 6, 2020

@TakeshiYonezawa ,
3.5.6 has just been released. 🎊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants