SemanticDiff fastapi
♻️ Refactor deciding if `embed` body fields, do not overwrite fields, compute once per router, refactor internals in preparation for Pydantic models in `Form`, `Query` and others
#12117
Merged

♻️ Refactor deciding if `embed` body fields, do not overwrite fields, compute once per router, refactor internals in preparation for Pydantic models in `Form`, `Query` and others #12117

tiangolo merged 33 commits into master from embed
tiangolo
tiangolo ♻️ Add _compat util to `get_model_fields()`
b8e87cc9
tiangolo ♻️ Make `Body()` have a default `embed=None` to be computed automatic…
0298aea0
tiangolo ♻️ Set embed for form fields in analyze_param()
b05e8331
tiangolo ♻️ Add `_should_embed_body_fields()` util
fc1d8712
tiangolo 🚚 Rename required_params to body_fields
3c51c993
tiangolo ♻️ Do not override ModelFields.field_info.embed, use a top-level sett…
7c7e763a
tiangolo 🚨 Add type ignore
6b73532a
tiangolo ♻️ Refactor `request_body_to_args()`
b6e24839
includeamin
includeamin commented on 2024-09-02
tiangolo 🔥 Remove unnecessary code
3a53c050
tiangolo ♻️ Refactor `_should_embed_body_fields()` with ideas from includeamin
783d1411
svlandeg svlandeg added refactor
tiangolo ♻️ Refactor `request_body_to_args()`, separate `_extract_form_body()`
299fd6f8
tiangolo ♻️ Refactor single model field validation in `_validate_value_with_mo…
fdbe6547
tiangolo ♻️ Refactor `request_body_to_args()` to handle single not embedded fi…
5adfe718
tiangolo 🚚 Move variables
0bdf69c7
tiangolo 🐛 Fix check for model field's model
db1ba434
tiangolo Merge branch 'master' into embed
cf22bb10
tiangolo 🚨 Fix type annotations
11ad3675
tiangolo 🔥 Remove code specific to supporting new features
ae610d6f
tiangolo 🔥 Remove feature specific utils
63c2e322
tiangolo 🔥 Remove unnecessary checks
5dd26d49
tiangolo 🔥 Refactor and remove `is_body_param()`
c841137f
tiangolo ♻️ Refactor arguments for `request_params_to_args()`
88dc5219
tiangolo 🔊 Raise on current invalid query params
a7e490f2
tiangolo ♻️ Move asserts inside of field block (not dependency)
bf62f9c2
tiangolo 🔥 Remove unnecessary embed set
45ffd5e5
tiangolo ✅ Add test for a single form field
9009632f
tiangolo ♻️ Refactor _compat util `field_annotation_is_sequence()` to support …
920e86a4
tiangolo ♻️ Refactor logic to `_get_multidict_value()` for form fields, header…
20b1a7ee
tiangolo 🔥 Remove unreachable code, add assert
0f41a28b
tiangolo 🚨 Fix type annotations
6a3b7d13
tiangolo ♻️ Add util to `get_model_fields()`
b806e4d8
tiangolo ✅ Add test for coverage
fe14197d
tiangolo tiangolo changed the title ♻️ Refactor deciding if `embed` body fields, do not overwrite fields, compute once per router ♻️ Refactor deciding if `embed` body fields, do not overwrite fields, compute once per router, refactor internals in preparation for Pydantic models in `Form`, `Query` and others 14 days ago
tiangolo Merge branch 'master' into embed
dccb2b64
tiangolo tiangolo marked this pull request as ready for review 14 days ago
tiangolo tiangolo merged aa21814a into master 14 days ago
tiangolo tiangolo deleted the embed branch 14 days ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
Milestone