MapsterMapper / MapsterMapper/Mapster

Is there going to be a [MaxDepth] attribute?

Open
#550 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

good first issue improvement open for PR
Dominant language
C#
Stars
5.2k
Forks
410
Avg merge
2d 12h
Merged PRs (30d)
6

Description

Say I have a Class that has 3 Arrays of Classes on it, each has another 2 Arrays of classes on it and the final has 1 Array of class on it
And I want to get 3 deep on Class2, 2 deep on class3 and 1 deep on class4
i.e.

Class1 {
     [MaxDepth(3)]
     Class2[]
     [MaxDepth(2)]
     Class3[]
     [MaxDepth(1)]
     Class4[]
}

Class2 {
    Class5[]
    Class6[]
}

Class3 {
    Class5[]
    Class6[]
}

Class4 {
     Class5[]
     Class6[]
}

Class 5 {
    Class8[]
}

Class 6 {
    Class[9]
}

Class 7 {
    Class10[]
}

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source files or tests are named in the issue. Start by reviewing how Mapster currently handles nested mapping depth, then determine how per-property limits should behave for the Class2, Class3, and Class4 example and add tests that establish the expected result.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
tooling
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.