Django-filter: FilterSet에 필드가 제공되지 않으면 예외 발생

@nip3o 네. 이것을 보게되어 기쁩니다.

또한 DRF에 따라 필드에 선언된 필터가 필요하도록 하고 싶습니다.

btw - 데이터 누출은 #451에 의해 부분적으로 해결되었습니다. 지금 당장은 정의되지 않은/변경되는 동작에 대한 사용 중단 경고 가 표시되어야 합니다. 1.0에서 새로운 동작은 Meta.fields 또는 Meta.fields = None 가 없으면 아무 것도 하지 않는다는 것입니다.

또한 필드에 DRF에 따라 선언된 필터가 필요하도록 하고 싶습니다.

이것이 Meta.fields 에 대한 dict 구문과 어떻게 작동하는지 잘 모르겠습니다. 예:

class UserFilter(filters.FilterSet):
    username = filters.CharFilter()

    class Meta:
        model = User
        fields = {
            'name': ['contains', 'startswith', 'endswith', ...'],
            ???
        }

이것이 dict 구문과 어떻게 작동하는지 잘 모르겠습니다.

물론.

나는 여전히 이것에 대해 일종의 :-1:입니다. Meta.fields dict 구문 w/ 선언된 필터를 해결하는 방법이 없습니다. 적어도 어색하지 않은 방식으로:

class UserFilter(filters.FilterSet):
    username = filters.CharFilter(name='username', lookup_expr='exact')
    username_like = filters.CharFilter(name='username', lookup_expr='icontains')
    relevance = filters.CharFilter(method='filter_relevance')

    class Meta:
        model = User
        fields = {
            'name': ['contains', 'startswith', 'endswith', ...'],
            'username': ['exact'],  # match w/ the lookup_expr? 
            'username_like': ['icontains'],
            'relevance': [???]  # What about method filters?
        }

또한 이 요구 사항은 FilterSet.FILTER_DEFAULTS 에 있는 필드 클래스를 상속하지 않는 사용자 지정 모델 필드에 대한 문제가 발생합니다. 사용자 정의 필드에는 다음이 필요합니다.

class NetworkSettingFilter(FilterSet):
    mask = MaskFilter(...)

    class Meta:
        model = NetworkSetting
        fields = ['mask']
        filter_overrides = {
            SubnetMaskField: {'filter_class': MaskFilter},
        }

# vs.

class NetworkSettingFilter(FilterSet):
    mask = MaskFilter(...)

    class Meta:
        model = NetworkSetting

참고 사항: 이 나중 요점은 우리가 _고칠 수 있는_ 것입니다. Django의 모델 형식 대신 DRF의 직렬 변환기처럼 작동하도록 자동 생성을 다시 작성해야 합니다. 어느 쪽이든, 내 주요 관심사는 구문 충돌입니다.

좋아요. 일단 놔두자.

나는 원래 제안을 좋아한다. 범위를 _ fields 가 목록인 경우..._로 범위를 지정하는 경향이 있습니다. 하지만 조금 더 생각해야 합니다.

약간 앞뒤가 맞지만 원래 문제에 대해서만 다시 이정표를 작성하겠습니다.

fields 또는 exclude 를 선언해야 합니다. fields = '__all__' 는 최소값이어야 합니다.

(목록 일치를 강제하지 않는 것은 다시 돌아올 수 있는 문제입니다.)

관련 DRF 코드는 다음 과 같습니다.

        assert not (fields is None and exclude is None), (
            "Creating a ModelSerializer without either the 'fields' attribute "
            "or the 'exclude' attribute has been deprecated since 3.3.0, "
            "and is now disallowed. Add an explicit fields = '__all__' to the "
            "{serializer_class} serializer.".format(
                serializer_class=self.__class__.__name__
            ),
        )

filters_for_model 에 비슷한 추가 가 필요합니다.

    # Setting exclude with no fields implies all other fields.
    if exclude is not None and fields is None:
        fields = ALL_FIELDS

시작하려면 더 이상 사용되지 않아야 합니다.

fields 또는 exclude 를 선언해야 합니다. fields = '__all__' 는 최소값이어야 합니다.

나는 확실히 그 뒤에 갈 수 있습니다. 현재 PR 작업을 하고 있습니다.

멋있는.

감가 상각에 대해 걱정하지 마십시오. 주요 버전 변경에 대한 작은 변경입니다. 변경 노트에 명시해 두십시오. 👍🏽

감가 상각에 대해 걱정하지 마십시오. 주요 버전 변경에 대한 작은 변경입니다. 변경 노트에 명시해 두십시오.

의도는 약간 다르지만 실제로 여기 에서 다루었습니다.

또한 현재 filters_for_model 를 리팩토링하여 선언된 필터에 대한 필터 생성을 시도하지 않습니다. (이것은 fields = ['mask'] 가 불필요한 예외를 발생시키는 서브넷/마스크 예와 관련이 있습니다.

폐쇄 #550