FasterXML / FasterXML/jackson-annotations

Make `@JsonIgnoreProperties` and `@JsonPropertyOrder` merge (optionally), not just override

Aberta
#82 3 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Java
Estrelas
1.1k
Forks
342
Merge médio
8h 53min
PRs com merge (30d)
2

Descrição

A small subset of array-valued annotations would benefit from ability to append/merge their values.
List includes at least:
- `@JsonIgnoreProperties` (default: simple union/merge)
- `@JsonPropertyOrder` (default: simple append?)

In these cases, current behavior of full replacement is probably not what is usually the expected behavior.
But additionally we should probably also allow alternate choice of "override"/"overwrite".

One obvious challenge is that since `Annotation` types can not be instantiated, we may have to figure out whether we could use value object approach (see `@JacksonAnnotationValue` for details) to help here.

The main challenge here is to combine two important aspects:
1. Try to keep annotation-merging/inheritance-handling/resolution code annotation agnostic and fully generic (that is, logic need not know semantics of actual annotation tpes)
2. Fully handle merging so that un-merged set of annotations need not be exposed to calling code.

So: big part of this is to see if merging is practical, and how.

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.