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

PathWrapper orm #232

Open
CorvusYe opened this issue Sep 13, 2023 · 3 comments
Open

PathWrapper orm #232

CorvusYe opened this issue Sep 13, 2023 · 3 comments
Labels
Hacktoberfest help wanted Extra attention is needed

Comments

@CorvusYe
Copy link
Collaborator

in this case: https://github.com/nebula-contrib/ngbatis/blob/master/ngbatis-demo/src/test/java/ye/weicheng/ngbatis/demo/NebulaBasicDaoTests.java#L465

we need to add another condition branch to implement the orm of PathWrapper.

下面代码需要追加一个条件分支来实现 PathWrapper 的 orm

https://github.com/nebula-contrib/ngbatis/blob/master/src/main/java/org/nebula/contrib/ngbatis/handler/ObjectResultHandler.java#L67-L93

Such as:

 else if(v instanceof PathWrapper) {
    newResult = fillResultByPath( // fillResultByPath is not exists now.
      (PathWrapper) v,
      newResult,
      columnNames,
      resultType,
      columnName
    );
  } 
@CorvusYe CorvusYe added the help wanted Extra attention is needed label Sep 13, 2023
@wey-gu wey-gu moved this to Todo in hacktoberfest2023 Sep 27, 2023
@shbone
Copy link
Contributor

shbone commented Sep 27, 2023

@wey-gu I want to pr for this,Thank you

@shbone
Copy link
Contributor

shbone commented Sep 27, 2023

@CorvusYe what do I need to do? add another condition branch to https://github.com/nebula-contrib/ngbatis/blob/master/src/main/java/org/nebula/contrib/ngbatis/handler/ObjectResultHandler.java#L67-L93 ? solve the ORM problem?

@CorvusYe
Copy link
Collaborator Author

@shbone Your understanding is correct. A conditional branch needs to be added, but the corresponding method is not available at the moment. Need to design and implementation, how to establish the specific mapping relationship I have not thought well, you can also think about how to achieve the path of the entity mapping.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hacktoberfest help wanted Extra attention is needed
Projects
Status: No status
Status: Todo
Development

No branches or pull requests

3 participants